aviobuf: Write new data at s->buf_end in fill_buffer
authorMartin Storsjö <martin@martin.st>
Sat, 26 Feb 2011 23:02:32 +0000 (01:02 +0200)
committerLuca Barbato <lu_zero@gentoo.org>
Wed, 2 Mar 2011 10:16:17 +0000 (11:16 +0100)
commite360ada2d13af36ab7afd9ebcd2bd236d23d9b96
tree2aaa116d938feda59f5ab74a16a60d22af7d7f48
parent06ed4873e6e6aed8ec7cc24285d610ef4060880e
aviobuf: Write new data at s->buf_end in fill_buffer

In most cases, s->buf_ptr will be equal to s->buf_end when
fill_buffer is called, but this may not always be the case, if
we're seeking forward by reading (permitted by the short seek
threshold).

If fill_buffer is writing to s->buf_ptr instead of s->buf_end (when
they aren't equal and s->buf_ptr is ahead of s->buffer), the data
between s->buf_ptr and s->buf_end is overwritten, leading to
inconsistent buffer content. This could return incorrect data if
later seeking back into the area before the current s->buf_ptr.

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
libavformat/aviobuf.c