[FFmpeg-devel] [PATCH] H.264 : lowres isn't supported currently so treat a non-zero value as an error.

[Carl Eugen Hoyos]

Jai Menon <jmenon86 <at> gmail.com> writes:

> diff --git a/libavcodec/h264.c b/libavcodec/h264.c
> index 1054286..5d9ef89 100644
> --- a/libavcodec/h264.c
> +++ b/libavcodec/h264.c
> @@ -903,6 +903,11 @@ av_cold int ff_h264_decode_init(AVCodecContext *avctx){
>      H264Context *h= avctx->priv_data;
>      MpegEncContext * const s = &h->s;
> 
> +    if (avctx->lowres) {

Since this affects many decoders (and for the remaining ones, there usually is a
limit, and they crash if it is exceeded), could this be solved more generally.

My naive idea was to add the maximum value to struct AVCodec, and assume 0 as
default value.

Carl Eugen