On 9/25/11 11:03 PM, Justin Ruggles wrote:
This also prevents NULL packets from being sent to the decoder unless CODEC_CAP_DELAY is set. --- libavcodec/utils.c | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-)diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 8459e5f..85c9d76 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -747,6 +747,11 @@ int attribute_align_arg avcodec_decode_audio3(AVCodecContext *avctx, int16_t *sa avctx->pkt = avpkt; + if (!avpkt->data&& avpkt->size) { + av_log(avctx, AV_LOG_ERROR, "invalid packet: NULL data, size> 0\n"); + return AVERROR(EINVAL); + } + if((avctx->codec->capabilities& CODEC_CAP_DELAY) || avpkt->size){ //FIXME remove the check below _after_ ensuring that all audio check that the available space is enough if(*frame_size_ptr< AVCODEC_MAX_AUDIO_FRAME_SIZE){
Seems ok as patch, I'm wondering about the reference to CODEC_CAP_DELAY though.
lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
