diff options
| author | Martin Storsjö <martin@martin.st> | 2013-01-17 16:03:36 +0200 |
|---|---|---|
| committer | Martin Storsjö <martin@martin.st> | 2013-01-17 19:06:56 +0200 |
| commit | e33db35b4a91ad543d9dde3a981a89118ba68937 (patch) | |
| tree | 9e056fd0cc4b13349cc8bdf1c98173f2a5e2358d | |
| parent | b805c725a31f6cb05c488e485eb6ab370a851633 (diff) | |
| download | ffmpeg-e33db35b4a91ad543d9dde3a981a89118ba68937.tar.gz | |
theora: Skip zero-sized headers
This fixes a regression since d9cf5f51 with theora over RTP
(possibly with other variants of theora as well).
In theora over RTP, the second of the 3 headers turns out to be
0 bytes long, which prior to d9cf5f51 worked just fine. After
d9cf5f51, reading from the bitstream reader fails (since the reader
wasn't initialized but returned an error if initialized with 0 bits).
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>
| -rw-r--r-- | libavcodec/vp3.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/libavcodec/vp3.c b/libavcodec/vp3.c index bdd4289a49..0340c22bb2 100644 --- a/libavcodec/vp3.c +++ b/libavcodec/vp3.c @@ -2339,6 +2339,8 @@ static av_cold int theora_decode_init(AVCodecContext *avctx) } for(i=0;i<3;i++) { + if (header_len[i] <= 0) + continue; init_get_bits(&gb, header_start[i], header_len[i] * 8); ptype = get_bits(&gb, 8); |