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 747028 - dashdemux: crash in multiple period scenario
dashdemux: crash in multiple period scenario
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
1.4.5
Other All
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-03-30 14:33 UTC by xixi
Modified: 2015-08-16 11:14 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (3.08 KB, patch)
2015-03-30 14:48 UTC, xixi 		none Details | Review

Description xixi 2015-03-30 14:33:36 UTC 
In multiple period scenario, "gst_dash_demux_advance_period" will cleanup/unref the running task itself, this will cause undefined strange behaviours.
 
In master branch, it seems that it's been fixed in adaptivedemux though I did not test it. 
I will try it later on master branch.
Comment 1 xixi 2015-03-30 14:48:03 UTC 
Created attachment 300599 [details] [review]
patch

a candidate patch is attached.

previous period's gsttask will be cleanup/unref-ed in next period's gsttask loop.
Comment 2 xixi 2015-04-07 13:23:09 UTC 
I tested multiple period case on master branch.
It does NOT crash, yet it does NOT work well.
I reported the bug and also submitted a patch for that.
Please refer to bug 747449 .
 
 
For this crash and undefined behaviour in 1.4.5 branch, this bug and patch are still open and waiting for review.
Comment 3 xixi 2015-05-14 03:25:18 UTC 
ping... any comments will be appreciated
Comment 4 Sebastian Dröge (slomo) 2015-08-16 11:14:11 UTC 
Closing as this is fixed in GIT master apparently