avio: Do not consider the end-of-buffer position valid
authorLuca Barbato <lu_zero@gentoo.org>
Sat, 24 Jan 2015 12:06:19 +0000 (13:06 +0100)
committerLuca Barbato <lu_zero@gentoo.org>
Wed, 28 Jan 2015 15:28:10 +0000 (16:28 +0100)
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

index 6923b78..5f848b2 100644 (file)
@@ -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 ||