On Mon, Nov 23, 2015 at 7:25 PM, Luca Barbato <[email protected]> wrote:
> ---
>  libavformat/nutdec.c | 15 ++++++++++-----
>  1 file changed, 10 insertions(+), 5 deletions(-)
>
> diff --git a/libavformat/nutdec.c b/libavformat/nutdec.c
> index bf1ab7b..1778f42 100644
> --- a/libavformat/nutdec.c
> +++ b/libavformat/nutdec.c
> @@ -238,7 +238,9 @@ static int decode_main_header(NUTContext *nut)
>          GET_V(nut->time_base[i].num, tmp > 0 && tmp < (1ULL << 31));
>          GET_V(nut->time_base[i].den, tmp > 0 && tmp < (1ULL << 31));
>          if (av_gcd(nut->time_base[i].num, nut->time_base[i].den) != 1) {
> -            av_log(s, AV_LOG_ERROR, "time base invalid\n");
> +            av_log(s, AV_LOG_ERROR, "invalid time base %d/%d\n",
> +                   nut->time_base[i].num,
> +                   nut->time_base[i].den);
>              return AVERROR_INVALIDDATA;
>          }
>      }
> @@ -281,7 +283,8 @@ static int decode_main_header(NUTContext *nut)
>              return AVERROR_INVALIDDATA;
>          }
>          if (tmp_stream >= stream_count) {
> -            av_log(s, AV_LOG_ERROR, "illegal stream number\n");
> +            av_log(s, AV_LOG_ERROR, "illegal stream number %d expected %d\n",
> +                   tmp_stream, stream_count);
>              return AVERROR_INVALIDDATA;
>          }

here you might get tmp_stream == stream_count which would print a
weird error message
I suggest "illegal stream number (%d >= %d)\n"

the rest looks ok
-- 
Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to