On Tue, Sep 17, 2013 at 12:13:49PM +0300, Martin Storsjö wrote:
> On Mon, 16 Sep 2013, Diego Biurrun wrote:
>
> >On Mon, Sep 16, 2013 at 10:06:18PM +0300, Martin Storsjö wrote:
> >>--- a/libavcodec/rv34.c
> >>+++ b/libavcodec/rv34.c
> >>@@ -1495,8 +1495,10 @@ av_cold int ff_rv34_decode_init(AVCodecContext
> >>*avctx)
> >>
> >>- if ((ret = rv34_decoder_alloc(r)) < 0)
> >>+ if ((ret = rv34_decoder_alloc(r)) < 0) {
> >>+ ff_MPV_common_end(&r->s);
> >> return ret;
> >>+ }
> >
> >Don't you have to do the same thing on line 1520?
>
> Possibly - I'm not sure how the error handling/cleanup is handled
> for the threading case off-hand (the sample I have doesn't get that
> far). Janne or Kostya?
No idea either.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
