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 753365 - basesrc: Test Assertion 'segment->rate == 0.5' failed
basesrc: Test Assertion 'segment->rate == 0.5' failed
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-08-07 18:33 UTC by Nicolas Dufresne (ndufresne)
Modified: 2018-11-03 12:29 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
basesrc-test: Fix race testing segment update (2.36 KB, patch)
2015-08-07 19:44 UTC, Nicolas Dufresne (ndufresne)
none Details | Review

Description Nicolas Dufresne (ndufresne) 2015-08-07 18:33:27 UTC
I was running make check over core, and got this one time failure. After running:

  make -C tests/check libs/basesrc.forever

I ended up with:

  87%: Checks: 8, Failures: 1, Errors: 0
  libs/basesrc.c:598:F:general:basesrc_seek_events_rate_update:0: Assertion 'segment->rate == 0.5' failed
  make: Leaving directory '/home/nicolas/Sources/gstreamer-master/gstreamer/tests/check'
Comment 1 Nicolas Dufresne (ndufresne) 2015-08-07 19:44:35 UTC
Created attachment 308910 [details] [review]
basesrc-test: Fix race testing segment update

As this test is using a short sleep (GST_USECOND, which is 10ms
in microsecond), sometimes that EOS event is received before the
loop in basesrc have run _do_seek() and pushed the update segment.
To solve this issue, we wait for the initial segment (and flush it)
then we wait for the second segment before sending EOS.
Comment 2 Nicolas Dufresne (ndufresne) 2015-08-07 19:45:24 UTC
Note that all the tests wrongly pretends waiting for 1 second, but in fact wait for 10ms. There is as many possible race as there is sleeps in that test imho.
Comment 3 Nicolas Dufresne (ndufresne) 2015-08-22 16:11:40 UTC
Leaving open, as we could try and fix the other test later, this one was the only one I could manage to get failing though.

Attachment 308910 [details] pushed as 900110c - basesrc-test: Fix race testing segment update
Comment 4 GStreamer system administrator 2018-11-03 12:29:30 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gstreamer/issues/127.