On Wed, 8 Jan 2014 03:25:32 +0100, Luca Barbato <[email protected]> wrote:
> Overly large files are to be expected.
>
> Reported-by: Jean-Baptiste Kempf <[email protected]>
> Sample: http://streams.videolan.org/misc/mxf.zip
IIUC the samples are not stored there indefinitely, so there's not much point in
mentioning it in a commit message (especially since it has such a generic name)
> ---
> libavformat/mxfdec.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> index 0dd3f75..8f17fa1 100644
> --- a/libavformat/mxfdec.c
> +++ b/libavformat/mxfdec.c
> @@ -2237,8 +2237,10 @@ static int mxf_read_seek(AVFormatContext *s, int
> stream_index, int64_t sample_ti
> sample_time = 0;
> seconds = av_rescale(sample_time, st->time_base.num, st->time_base.den);
>
> - if ((ret = avio_seek(s->pb, (s->bit_rate * seconds) >> 3, SEEK_SET)) < 0)
> - return ret;
> + seekpos = avio_seek(s->pb, (s->bit_rate * seconds) >> 3, SEEK_SET);
> + if (seekpos < 0)
> + return seekpos;
> +
> ff_update_cur_dts(s, st, sample_time);
> mxf->current_edit_unit = sample_time;
> } else {
> --
> 1.8.5.1
>
Patch itself looks ok.
--
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
