[FFmpeg-devel] [PATCH 4/4] avfilter/vf_paletteuse: Forward error codes

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Tue Mar 24 22:14:36 EET 2020


Paul B Mahol:
> lgtm
> 
> On 1/27/20, Andreas Rheinhardt <andreas.rheinhardt at gmail.com> wrote:
>> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
>> ---
>>  libavfilter/vf_paletteuse.c | 8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/libavfilter/vf_paletteuse.c b/libavfilter/vf_paletteuse.c
>> index 255c9d79e3..408cff786e 100644
>> --- a/libavfilter/vf_paletteuse.c
>> +++ b/libavfilter/vf_paletteuse.c
>> @@ -912,12 +912,12 @@ static int apply_palette(AVFilterLink *inlink, AVFrame
>> *in, AVFrame **outf)
>>                            s->last_out, out, &x, &y, &w, &h);
>>      av_frame_unref(s->last_in);
>>      av_frame_unref(s->last_out);
>> -    if (av_frame_ref(s->last_in, in) < 0 ||
>> -        av_frame_ref(s->last_out, out) < 0 ||
>> -        av_frame_make_writable(s->last_in) < 0) {
>> +    if ((ret = av_frame_ref(s->last_in, in))       < 0 ||
>> +        (ret = av_frame_ref(s->last_out, out))     < 0 ||
>> +        (ret = av_frame_make_writable(s->last_in)) < 0) {
>>          av_frame_free(&out);
>>          *outf = NULL;
>> -        return AVERROR(ENOMEM);
>> +        return ret;
>>      }
>>
>>      ff_dlog(ctx, "%dx%d rect: (%d;%d) -> (%d,%d) [area:%dx%d]\n",
>> --
>> 2.20.1
>>
Applied, thanks.

- Andreas



More information about the ffmpeg-devel mailing list