On Thu, 2008-08-07 at 09:36 -0400, Brandon Jenkins wrote:
> On Wed, Aug 6, 2008 at 8:55 PM, Andy Walls <[EMAIL PROTECTED]> wrote:
> > On Wed, 2008-08-06 at 06:57 -0400, Brandon Jenkins wrote:
> >> On Tue, Aug 5, 2008 at 9:29 PM, Andy Walls <[EMAIL PROTECTED]> wrote:
> >> > On Tue, 2008-08-05 at 21:04 -0400, Brandon Jenkins wrote:

> >
> > Nope.  The problem I have has to do with per stream queue and buffer
> > accounting being slightly but you'll only notice when it's being freed.
> >
> > I suspect you have the same problem, but I can't tell for sure as you
> > system is compiling the code differently than mine.
> >
> > Could you please send the output of
> >
> > $ cd v4l-dvb
> > $ objdump -D v4l/cx18-queue.o
> >
> > from the offending build to me.  That way I can see the assembled
> > machine code and verify where in the function the NULL dereference is
> > happening.
> >
> > If you have the exact same problem as me, I can give you a "band-aid"
> > patch which will lessen the problem in short order.  It'll be a band aid
> > because it won't fix the accounting problem though.  I need to do more
> > extensive test and debug to find out where the accounting of buffers is
> > getting screwed up.
> >
> > Regards,
> > Andy
> 
> Andy,
> 
> Reposting with the file hosted in my dropbox instead. I didn't realize
> there was a size limit on the devel list.

Brandon,

I got the file anyway.  You have the exact same problem I had in the
oops.  Ubuntu's compilation options appear to be a little more
aggressive than Fedora's, but it's the same NULL pointer.

The problem is when moving all the buffers for a stream back to the
q_free queue, it appears the source queue claims to have more bytes to
move back to q_free than it actually does.  The old general purpose
code, copied from ivtv I think, then tries to steal buffers from another
queue to satisfy the request to move all the remaining bytes that the
first queue lied about having. But there is no queue to steal from, so
you deref a NULL pointer.

I'll have a quick fix sometime tomorrow.  It's been a long day and
experience tells me I'll only code up bugs tonight when I'm tired. :)

Regards,
Andy



> Brandon
> 
> https://dl.getdropbox.com/u/4976/cx18-queue.o.objdump.tar.gz
> 

--
video4linux-list mailing list
Unsubscribe mailto:[EMAIL PROTECTED]
https://www.redhat.com/mailman/listinfo/video4linux-list


_______________________________________________
ivtv-devel mailing list
[email protected]
http://ivtvdriver.org/mailman/listinfo/ivtv-devel

Reply via email to