On 24/11/15 15:21, Vittorio Giovara wrote: > 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 >
Amended. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
