<artem.ga...@gmail.com> 于2020年5月19日周二 上午3:26写道:
>
> From: Artem Galin <artem.ga...@intel.com>
>
> Signed-off-by: Artem Galin <artem.ga...@intel.com>
> ---
>  fftools/ffmpeg_opt.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c
> index 60bb437ea7..83f1b53e9d 100644
> --- a/fftools/ffmpeg_opt.c
> +++ b/fftools/ffmpeg_opt.c
> @@ -558,7 +558,11 @@ static int opt_init_hw_device(void *optctx, const char 
> *opt, const char *arg)
>          printf("\n");
>          exit_program(0);
>      } else {
> -        return hw_device_init_from_string(arg, NULL);
> +        int err;
> +        if (!arg)
> +            return AVERROR(ENOMEM);

There is no memory allocation, so return EINVAL is better IMHO.

> +        err = hw_device_init_from_string(arg, NULL);
> +        return err;
>      }
>  }

The rest look good, but prefer to remove "qsv" in the patch subject
since this is not a "qsv" limited change .
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to