qtmux: Remove the pad from our internal list before calling collectpads
authorEdward Hervey <bilboed@bilboed.com>
Mon, 21 Jun 2010 17:39:54 +0000 (19:39 +0200)
committerTim-Philipp Müller <tim.muller@collabora.co.uk>
Tue, 12 Apr 2011 19:32:17 +0000 (20:32 +0100)
commit0fa380b19996cde70af50ba247474779e746b833
tree763aff2dd69ed46ebdab32930169c83371089439
parent4bb9ffa8fb7dbacb320964729abf7bf7ad4ff2cf
qtmux: Remove the pad from our internal list before calling collectpads

Previously we would end up with the collectpaddata structure already freed.
This would result in a bogus iteration of mux->sinkpads (all the
GstQTPad being freed) and it wouldn't be removed from that list.

Finally, due to it not being removed from that list, we would end up
calling a bogus gst_qt_mux_pad_reset on those structures => SEGFAULT
gst/quicktime/gstqtmux.c