diff options
| author | wm4 <wm4@nowhere> | 2013-06-05 02:04:21 +0200 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2013-06-09 22:06:03 +0200 |
| commit | 7b16d4b3b2a46ea3314d0a2f9140f3359750770f (patch) | |
| tree | 1628d42fc60e572ba161a396e7d58fe67d677fe2 /stream/stream_vcd.c | |
| parent | 780d62f19eedc32814c17de6943196d1aab0f26e (diff) | |
| download | mpv-7b16d4b3b2a46ea3314d0a2f9140f3359750770f.tar.bz2 mpv-7b16d4b3b2a46ea3314d0a2f9140f3359750770f.tar.xz | |
stream_cdda, stream_vcd: check read buffer size
These assumed that the buffer provided with fill_buffer() was at least
sector sized, instead of checking the size parameter.
This is just a cleanup, since every caller made sure to align everything
on sector sizes, if a stream has the sector size set.
Diffstat (limited to 'stream/stream_vcd.c')
| -rw-r--r-- | stream/stream_vcd.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/stream/stream_vcd.c b/stream/stream_vcd.c index 1ef9a337de..d8ef552aa5 100644 --- a/stream/stream_vcd.c +++ b/stream/stream_vcd.c @@ -79,6 +79,8 @@ static const struct m_struct_st stream_opts = { static int fill_buffer(stream_t *s, char* buffer, int max_len){ if(s->pos > s->end_pos) /// don't past end of current track return 0; + if (max_len < VCD_SECTOR_DATA) + return -1; return vcd_read(s->priv,buffer); } |