After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 738845 - audiomixer: critical error for blocksize, timeout min/max values
audiomixer: critical error for blocksize, timeout min/max values
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
unspecified
Other Linux
: Normal normal
: 1.5.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-10-20 07:49 UTC by Vineeth
Modified: 2014-10-24 14:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch (1.84 KB, patch)
2014-10-20 07:50 UTC, Vineeth
needs-work Details | Review
audiomixer critical error (1.95 KB, patch)
2014-10-20 12:56 UTC, Vineeth
committed Details | Review

Description Vineeth 2014-10-20 07:49:17 UTC
Audiomixer blocksize, cant be 0, hence adjusting the minimum value to 1
timeout value of aggregator is defined with MAX of MAXINT64,
but it cannot cross G_MAXLONG * GST_SECOND - 1
Hence changed the max value of the same
Comment 1 Vineeth 2014-10-20 07:50:30 UTC
Created attachment 288886 [details] [review]
proposed patch
Comment 2 Sebastian Dröge (slomo) 2014-10-20 08:07:29 UTC
Review of attachment 288886 [details] [review]:

::: gst-libs/gst/base/gstaggregator.c
@@ +1421,3 @@
           "Number of nanoseconds to wait for a buffer to arrive on a sink pad"
           "before the pad is deemed unresponsive (-1 unlimited)", -1,
+          G_MAXLONG * GST_SECOND - 1, DEFAULT_TIMEOUT,

This will overflow on 64 bit systems where long is 64 bits big. Just keep it as is :)
Comment 3 Vineeth 2014-10-20 08:17:14 UTC
is there any other way to resolve this error?

any value above G_MAXLONG * GST_SECOND - 1 will result in the below error.


GStreamer:ERROR:gstpoll.c:1242:gst_poll_wait: assertion failed: ("Value of time
" "timeout" " is out of timespec's range" && ((timeout) / GST_SECOND) <
G_MAXLONG)
Comment 4 Sebastian Dröge (slomo) 2014-10-20 08:53:06 UTC
Just take the value that is used inside gstpoll.c:1242
Comment 5 Vineeth 2014-10-20 12:56:40 UTC
Created attachment 288935 [details] [review]
audiomixer critical error
Comment 6 Sebastian Dröge (slomo) 2014-10-21 08:59:01 UTC
commit aec1f2a001cde486c58c42b76feec62bcefab870
Author: Vineeth T M <vineeth.tm@samsung.com>
Date:   Mon Oct 20 18:25:08 2014 +0530

    audiomixer: critical error for blocksize, timeout min/max values
    
    Audiomixer blocksize, cant be 0, hence adjusting the minimum value to 1
    timeout value of aggregator is defined with MAX of MAXINT64,
    but it cannot cross G_MAXLONG * GST_SECOND - 1
    Hence changed the max value of the same
    
    https://bugzilla.gnome.org/show_bug.cgi?id=738845
Comment 7 Luis de Bethencourt 2014-10-24 14:28:00 UTC
Thanks for the review Sebastian :)