[FFmpeg-devel] Small patch for someone to push please

[Michael Niedermayer]

On Fri, Aug 15, 2014 at 11:26:40AM +0100, JULIAN GARDNER wrote:
> In libavfilter/lavfutils.c around line 77 the code is
> 
>     if (ret < 0 || !frame_decoded) {
>         av_log(log_ctx, AV_LOG_ERROR, "Failed to decode image from file\n");
>         goto end;
>     }
>     ret = 0;
> 
> Now this causes a problem if ret>=0 and frame_decoded==0 as you get dropped out of the routine with a failed decode and no width or height.
> 
> my changes
> 
>     if (ret < 0 || !frame_decoded) {
>         av_log(log_ctx, AV_LOG_ERROR, "Failed to decode image from file\n");
> +        ret = !frame_decoded ? -1:ret;
>         goto end;
>     }
>     ret = 0;
> 

applied similar solution

thanks

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

No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140816/dbe67b3e/attachment.asc>