[Gstreamer-bugs] [Bug 119456] Changed - [0.6.3/patch] assertion `GST_IS_SCHEDULER (sched)' failed

[bugzilla-daemon at widget.gnome.org]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=119456

Changed by ds at schleef.org.

--- shadow/119456	Fri Aug  8 17:37:54 2003
+++ shadow/119456.tmp.31851	Thu Aug 21 04:42:08 2003
@@ -8,17 +8,24 @@
 Severity: normal
 Priority: Normal
 Component: gstreamer (core)
 AssignedTo: ds at schleef.org                            
 ReportedBy: ds at schleef.org               
 QAContact: gstreamer-maint at bugzilla.gnome.org
-TargetMilestone: HEAD
+TargetMilestone: 0.6.3
 URL: 
-Summary: assertion `GST_IS_SCHEDULER (sched)' failed
+Summary: [0.6.3/patch] assertion `GST_IS_SCHEDULER (sched)' failed
 
 rhythmbox HEAD causes this assertion failure when changing from one song to
 the next.
 
 (rhythmbox:11290): GStreamer-CRITICAL **: file gstscheduler.c: line 179
 (gst_scheduler_pad_unlink): assertion `GST_IS_SCHEDULER (sched)' failed
 
 This bug is a reminder for me to look into it.
+
+------- Additional Comments From ds at schleef.org  2003-08-21 04:42 -------
+This is caused by gst_bin_unset_element_sched() being called with
+sched==NULL.  Presumably, since there is another check for sched==NULL
+in the function, NULL is a valid value for sched.  However, I don't
+really understand the code.  The following patch adds another check in
+the correct place.