On 12/10/2012 07:24 PM, Justin Ruggles wrote: > On 12/10/2012 01:17 PM, Luca Barbato wrote: >> On 12/10/2012 05:49 PM, Justin Ruggles wrote: >>> From: Michael Niedermayer <[email protected]> >>> >>> This avoids a potential infinite loop due to not consuming any packet data. >>> >>> Signed-off-by: Michael Niedermayer <[email protected]> >>> Signed-off-by: Justin Ruggles <[email protected]> >>> >>> CC:[email protected] >>> --- >>> libavcodec/wmadec.c | 5 +++-- >>> 1 files changed, 3 insertions(+), 2 deletions(-) >>> >>> diff --git a/libavcodec/wmadec.c b/libavcodec/wmadec.c >>> index e2803bb..8d82ec4 100644 >>> --- a/libavcodec/wmadec.c >>> +++ b/libavcodec/wmadec.c >>> @@ -808,7 +808,8 @@ static int wma_decode_superframe(AVCodecContext *avctx, >>> void *data, >>> buf_size, avctx->block_align); >>> return AVERROR_INVALIDDATA; >>> } >>> - buf_size = avctx->block_align; >>> + if (avctx->block_align) >>> + buf_size = avctx->block_align; >> >> why block_align should be 0 ? > > It's from the container.
So we should bail on init in that case? _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
