On Tue, Apr 28, 2015 at 08:58:21PM +0200, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com> [...] > @@ -806,7 +811,10 @@ static int nut_read_header(AVFormatContext *s) > > ff_metadata_conv_ctx(s, NULL, ff_nut_metadata_conv); > > - return 0; > +end: > + if (ret < 0) > + nut_read_close(s);
> + return FFMIN(ret, 0); btw, the FFMIN could be removed unless iam missing something not really important though [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many things microsoft did are stupid, but not doing something just because microsoft did it is even more stupid. If everything ms did were stupid they would be bankrupt already.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
