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

JULIAN GARDNER joolzg at btinternet.com
Sat Aug 16 10:00:35 CEST 2014








>________________________________
> From: Michael Niedermayer <michaelni at gmx.at>
>To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org> 
>Sent: Saturday, 16 August 2014, 1:42
>Subject: Re: [FFmpeg-devel] Small patch for someone to push please
> 
>
>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
>

Thanks

Did you manage to have a quick look at my image loading code? would love to get this fixed.

joolz



More information about the ffmpeg-devel mailing list