[FFmpeg-devel] [PATCH] avdevice/x11grab: fix error handling in pixel formats

Benoit Fouet benoit.fouet at free.fr
Thu Sep 4 10:27:34 CEST 2014


Hi,

----- Mail original -----
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> ---
>  libavdevice/x11grab.c |    5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/libavdevice/x11grab.c b/libavdevice/x11grab.c
> index 120f754..e0d1dfa 100644
> --- a/libavdevice/x11grab.c
> +++ b/libavdevice/x11grab.c
> @@ -181,6 +181,8 @@ static int pixfmt_from_image(AVFormatContext *s,
> XImage *image, int *pix_fmt)
>             image->blue_mask,
>             image->bits_per_pixel);
>  
> +    *pix_fmt = AV_PIX_FMT_NONE;
> +
>      switch (image->bits_per_pixel) {
>      case 8:
>          *pix_fmt =  AV_PIX_FMT_PAL8;
> @@ -214,7 +216,8 @@ static int pixfmt_from_image(AVFormatContext *s,
> XImage *image, int *pix_fmt)
>              *pix_fmt = AV_PIX_FMT_0RGB32;
>          }
>          break;
> -    default:
> +    }
> +    if (*pix_fmt == AV_PIX_FMT_NONE) {
>          av_log(s, AV_LOG_ERROR,
>                 "XImages with RGB mask 0x%.6lx 0x%.6lx 0x%.6lx and
>                 depth %i "
>                 "are currently not supported.\n",

LGTM

-- 
Ben


More information about the ffmpeg-devel mailing list