[linux-dvb] Re: Oops in dvbdmx_release_ts_feed (2.6.0-test9)

[Ralph Metzler <rjkm@metzlerbros.de>]

Andreas Oberritter writes:
 > [1  <text/plain>]
 > On Sun, 2003-11-02 at 14:05, Ralph Metzler wrote:
 > > Pekka Pietikainen writes:
 > >  > On Sun, Nov 02, 2003 at 01:22:35PM +0200, Pekka Pietikainen wrote:
 > >  > 
 > >  > > Hopefully this is enough to debug the problem, I'll try to figure it out
 > >  > > myself as well, but since I'm quite unfamiliar with the code it might take
 > >  > > some time :/
 > >  > Is it just me, or shouldn't dmx_pid_set get demux->mutex before deleting 
 > >  > entries from feed->list?
 > > 
 > > No, it only is called from functions which already got the mutex.
 > > But it should disable irq/bh (see dmx_pid_set in dvb_sw_demux.c in our
 > > driver).
 > 
 > Thank you, Ralph. Can anyone please test the attached patch for the
 > dvb-kernel tree to see if it fixes the problem?
 > [2 dvb_demux.diff <text/x-patch; ISO-8859-15 (7bit)>]


The possible problem I referred to and which your patch
fixes only could lead to segfaults during interrupt or bottom half
caused by possibly deleted but not yet unlinked list entries or other
structures. 
The original problem was in dvbdmx_release_ts_feed() when called
from user space!? The bug causing this probably is still there.


Ralph





-- 
Info:
To unsubscribe send a mail to ecartis@linuxtv.org with "unsubscribe linux-dvb" as subject.