From 3c18a7b18807de81566381a1bcbe9f6103c0296b Mon Sep 17 00:00:00 2001 From: Luca Barbato Date: Sat, 24 Jan 2015 13:06:19 +0100 Subject: avio: Do not consider the end-of-buffer position valid Trigger a refill if the seek action moves the pointer at the end of the buffer. Before this patch the read action following the seek would trigger the refill, while write action would write outside the buffer. In the Libav codebase few muxers seek forward outside of what already has been written so it is quite unlikely to experience the problem with the default buffer size. CC: libav-stable@libav.org --- libavformat/aviobuf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'libavformat/aviobuf.c') diff --git a/libavformat/aviobuf.c b/libavformat/aviobuf.c index 6923b78ea1..5f848b20f9 100644 --- a/libavformat/aviobuf.c +++ b/libavformat/aviobuf.c @@ -206,7 +206,7 @@ int64_t avio_seek(AVIOContext *s, int64_t offset, int whence) } offset1 = offset - pos; if (!s->must_flush && - offset1 >= 0 && offset1 <= (s->buf_end - s->buffer)) { + offset1 >= 0 && offset1 < (s->buf_end - s->buffer)) { /* can do the seek inside the buffer */ s->buf_ptr = s->buffer + offset1; } else if ((!s->seekable || -- cgit v1.2.3