On Wed, Nov 28, 2012 at 1:32 PM, Janne Grunau <[email protected]> wrote:
>
> Fixes mov_svq3_aac__t_starcraft2_teasecinv2_h264.mov.SIGSEGV.d8a.664.
>
> CC: [email protected]
> ---
>  libavcodec/svq3.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/svq3.c b/libavcodec/svq3.c
> index ac8d9c1..c905276 100644
> --- a/libavcodec/svq3.c
> +++ b/libavcodec/svq3.c
> @@ -786,7 +786,7 @@ static int svq3_decode_slice_header(AVCodecContext *avctx)
>          skip_bits_long(&s->gb, 0);
>      }
>
> -    if ((i = svq3_get_ue_golomb(&s->gb)) == INVALID_VLC || i >= 3) {
> +    if ((i = svq3_get_ue_golomb(&s->gb)) == INVALID_VLC || i >= 3 || i < 0) {

Doesn't this make the INVALID_VLC comparison redundant?
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to