[FFmpeg-devel] [PATCH]Do not free non-allocated memory

Michael Niedermayer michaelni
Sun Oct 18 22:55:53 CEST 2009


On Sun, Oct 18, 2009 at 09:54:03AM +0200, Reimar D?ffinger wrote:
> On Sat, Oct 17, 2009 at 10:11:23PM +0200, Michael Niedermayer wrote:
> > On Fri, Oct 16, 2009 at 05:40:47PM +0200, Reimar D?ffinger wrote:
> > > On Fri, Oct 16, 2009 at 06:14:11PM +0300, Kostya wrote:
> > > > On Fri, Oct 16, 2009 at 05:00:53PM +0200, Carl Eugen Hoyos wrote:
> > > > > Hi!
> > > > >
> > > > > As spotted by Alex Converse in a patch I sent recently. I believe his  
> > > > > comment is also true for these files.
> > > > >
> > > > > Please comment, Carl Eugen
> > > > 
> > > > Indeed, seems so.
> > > > Please apply.
> > > 
> > > Maybe someone could be convinced to write a test program that just calls
> > > init() and close() or whatever it is called on all "things" (codecs,
> > > muxers,...) that would allow to easily spot such things with valgrind?
> > 
> > and/or run the regression tests or fate under valgrind
> 
> - will take really, really long, there is no way you can get people to run
>   it before every commit, and running FATE like this would make it slow down
>   enough that e.g. the AVR32 targets would be able to test far fewer revisions
>   and with far more delay

i obviously did not mean to suggest that this is run for every revission in
fate or before every commit on the developers machine

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091018/752dca57/attachment.pgp>



More information about the ffmpeg-devel mailing list