[FFmpeg-devel] [PATCH] delogo filter: new "uglarm" interpolation mode added

Carl Eugen Hoyos ceffmpeg at gmail.com
Thu Dec 27 21:25:35 EET 2018


2018-12-27 13:00 GMT+01:00, Uwe Freese <uwe.freese at gmx.de>:

>  > The "10" should be a #define, [...]
>
> I have now added as error handling:
>
> av_log(inlink->src, AV_LOG_ERROR, "More planes in frame than expected.\n");
> return AVERROR(ENOMEM);
>
> Is this ok, or how should this be implemented instead?

Not sure I understand: How can plane get >= MAX_PLANES?
If this is impossible (as I believe), please use av_assert0().

Please use av_freep() instead of av_free() for non-local
variables (the parameter needs an additional "&").

Carl Eugen


More information about the ffmpeg-devel mailing list