On 01/14/2012 11:47 AM, Paul B Mahol wrote:

> ---
>  Changelog                |    1 +
>  doc/general.texi         |    2 +-
>  libavformat/Makefile     |    1 +
>  libavformat/allformats.c |    2 +-
>  libavformat/smjpegenc.c  |  149 
> ++++++++++++++++++++++++++++++++++++++++++++++
>  libavformat/version.h    |    2 +-
>  6 files changed, 154 insertions(+), 3 deletions(-)
>  create mode 100644 libavformat/smjpegenc.c

> +            tag = ff_codec_get_tag(ff_codec_smjpeg_audio_tags, 
> codec->codec_id);
> +            if (!tag) {
> +                av_log(s, AV_LOG_ERROR, "unsupported audio codec\n");
> +                return AVERROR_INVALIDDATA;


return AVERROR(EINVAL);
since the invalid value is from the user.

> +            tag = ff_codec_get_tag(ff_codec_smjpeg_video_tags, 
> codec->codec_id);
> +            if (!tag) {
> +                av_log(s, AV_LOG_ERROR, "unsupported video codec\n");
> +                return AVERROR_INVALIDDATA;


ditto.

Thanks,
Justin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to