bytestream2_* will not cause buffer overflow, but on the other hand, it should be checked whether overflows have been prevented. --- libavcodec/wavpackenc.c | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/libavcodec/wavpackenc.c b/libavcodec/wavpackenc.c index 5b8973c..46c69a3 100644 --- a/libavcodec/wavpackenc.c +++ b/libavcodec/wavpackenc.c @@ -2813,6 +2813,11 @@ static int wavpack_encode_block(WavPackEncodeContext *s, block_size = bytestream2_tell_p(&pb); AV_WL32(out + 4, block_size - 8); + if (put_bits_left(&s->pb) <= 0) { + av_log(s->avctx, AV_LOG_ERROR, "Packet allocated too small\n"); + return AVERROR_INVALIDDATA; + } + return block_size; } -- 1.9.2.msysgit.0 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel