[FFmpeg-cvslog] r18406 - in trunk/libavcodec: h264.c h264.h h264_parser.c

Reimar Döffinger Reimar.Doeffinger
Fri Apr 10 13:31:13 CEST 2009


On Fri, Apr 10, 2009 at 01:16:33PM +0200, Diego Biurrun wrote:
> On Fri, Apr 10, 2009 at 09:01:56AM +0200, reimar wrote:
> > 
> > Log:
> > Add a ff_h264_free_context function and call it from the H.264 parser.
> > This ensures that the parser will no longer leak memory for all SPS/PPS it encounters.
> > 
> > --- trunk/libavcodec/h264.c	Fri Apr 10 02:09:07 2009	(r18405)
> > +++ trunk/libavcodec/h264.c	Fri Apr 10 09:01:55 2009	(r18406)
> > @@ -8083,10 +8083,8 @@ int main(void){
> >  
> > -static av_cold int decode_end(AVCodecContext *avctx)
> > +av_cold ff_h264_free_context(H264Context *h)
> >  
> > --- trunk/libavcodec/h264.h	Fri Apr 10 02:09:07 2009	(r18405)
> > +++ trunk/libavcodec/h264.h	Fri Apr 10 09:01:55 2009	(r18406)
> > @@ -569,4 +569,9 @@ const uint8_t *ff_h264_decode_nal(H264Co
> >  
> > +/**
> > + * frees any data that may have been allocated in the H264 context like SPS, PPS etc.
> > + */
> > +av_cold ff_h264_free_context(H264Context *h);
> 
> Ahem...
> 
> libavcodec/h264.h:575: warning: data definition has no type or storage class
> libavcodec/h264.h:575: warning: type defaults to ?int? in declaration of ?ff_h264_free_context?

Crap. Is there a compiler flag we could add that makes this an error?



More information about the ffmpeg-cvslog mailing list