[mythtv] Re: [mythtv-commits] mythtv commit: r6959 by danielk

[David Engel]

On Fri, Jul 29, 2005 at 02:08:06PM -0400, Anduin Withers wrote:
> ./mythtv/libs/libmythtv/avformatdecoder.cpp line 143 isn't C++, no new
> (short int)[size] (looks like a placement but isn't), the parens around
> short int should be removed.

And where is audioSamples freed?

> > I discovered that the problem is that something in the libav or
> > avformatdecoder overwrites the avformatdecoder class with audio
> > data. I haven't figured out where this is happening, but by
> > allocating the audio buffer on the heap, instead of as an
> > array in avformatdecoder, the SEGFAULT doesn't happen.

So now it might overwrite something else on the heap that is totally
unrelated and make it even harder to find the real problem?  Doesn't
sound like a win to me.

David
-- 
David Engel
gigem at comcast.net