diff options
| author | Martin Storsjö <martin@martin.st> | 2013-09-16 20:17:47 +0300 |
|---|---|---|
| committer | Martin Storsjö <martin@martin.st> | 2013-09-17 11:33:37 +0300 |
| commit | 7f8d41eb097e8d4223c9caf97dd332a2fdb29d52 (patch) | |
| tree | f259e33ddd108bcc303fa2c48d9533dc53f8f09d /libavformat/mov.c | |
| parent | 19b9659f3174599e8685d329c4330b1ea8c4c6db (diff) | |
| download | ffmpeg-7f8d41eb097e8d4223c9caf97dd332a2fdb29d52.tar.gz | |
mov: Don't use a negative duration for setting other fields
Some files have the duration set to -1 in the mdhd atom, more
or less legitimately. (We produce such files ourselves, for the
initial duration in fragmented mp4 files.)
Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>
Diffstat (limited to 'libavformat/mov.c')
| -rw-r--r-- | libavformat/mov.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libavformat/mov.c b/libavformat/mov.c index 9c72426539..77ba4441fb 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -2130,7 +2130,7 @@ static int mov_read_trak(MOVContext *c, AVIOContext *pb, MOVAtom atom) ((double)st->codec->width * sc->height), INT_MAX); } - if (st->duration != AV_NOPTS_VALUE) + if (st->duration != AV_NOPTS_VALUE && st->duration > 0) av_reduce(&st->avg_frame_rate.num, &st->avg_frame_rate.den, sc->time_scale*st->nb_frames, st->duration, INT_MAX); } @@ -2863,7 +2863,7 @@ static int mov_read_header(AVFormatContext *s) for (i = 0; i < s->nb_streams; i++) { AVStream *st = s->streams[i]; MOVStreamContext *sc = st->priv_data; - if (st->duration) + if (st->duration > 0) st->codec->bit_rate = sc->data_size * 8 * sc->time_scale / st->duration; } } |