On Wed, Nov 09, 2011 at 05:33:01PM -0500, Justin Ruggles wrote:
> The user could provide a larger buffer, which is already checked separately
> before writing output.
> ---
>  libavcodec/vorbisdec.c |    6 ------
>  1 files changed, 0 insertions(+), 6 deletions(-)
> 
> diff --git a/libavcodec/vorbisdec.c b/libavcodec/vorbisdec.c
> index e79feb9..b202249 100644
> --- a/libavcodec/vorbisdec.c
> +++ b/libavcodec/vorbisdec.c
> @@ -936,12 +936,6 @@ static int vorbis_parse_id_hdr(vorbis_context *vc)
>          av_log(vc->avccontext, AV_LOG_ERROR, " Vorbis id header packet 
> corrupt (illegal blocksize). \n");
>          return AVERROR_INVALIDDATA;
>      }
> -    // output format int16
> -    if (vc->blocksize[1] / 2 * vc->audio_channels * 2 > 
> AVCODEC_MAX_AUDIO_FRAME_SIZE) {
> -        av_log(vc->avccontext, AV_LOG_ERROR, "Vorbis channel count makes "
> -               "output packets too large.\n");
> -        return AVERROR_PATCHWELCOME;
> -    }
>      vc->win[0] = ff_vorbis_vwin[bl0 - 6];
>      vc->win[1] = ff_vorbis_vwin[bl1 - 6];
>  
> -- 

since we rely on *data_size, OK
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to