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-sta...@libav.org
---
The return value of the bitstream reader initialization should
perhaps also be checked, but there's not much point in trying
to initialize it and trying to read something, if we know that the
header is 0 bytes long.
---
libavcodec/vp3.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/libavcodec/vp3.c b/libavcodec/vp3.c
index bdd4289..0340c22 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);
--
1.7.9.4
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
