On Sun, Jul 06, 2014 at 10:24:14AM +0530, Nidhi Makhijani wrote: > --- > changed commit tag and added condition ret=0 > libavformat/cdg.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-)
You changed the commit, but you did not amend the log message to match. > --- a/libavformat/cdg.c > +++ b/libavformat/cdg.c > @@ -42,9 +42,10 @@ static int read_header(AVFormatContext *s) > ret = avio_size(s->pb); > - if (ret > 0) > - vst->duration = (ret * vst->time_base.den) / (CDG_PACKET_SIZE * 300); > + if (ret <= 0) > + return AVERROR_INVALIDDATA; > > + vst->duration = (ret * vst->time_base.den) / (CDG_PACKET_SIZE * 300); This no longer just changes the return value, you also change the condition. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
