Merge commit '28eddef689f2b4843a84f7d05fd9614246f92cc4'
authorMichael Niedermayer <michaelni@gmx.at>
Thu, 23 Apr 2015 19:57:12 +0000 (21:57 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Thu, 23 Apr 2015 19:57:16 +0000 (21:57 +0200)
* commit '28eddef689f2b4843a84f7d05fd9614246f92cc4':
  hq_hqa: Validate get_vlc2 return value

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavcodec/hq_hqa.c

diff --combined libavcodec/hq_hqa.c
@@@ -1,20 -1,20 +1,20 @@@
  /*
   * Canopus HQ/HQA decoder
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
@@@ -75,7 -75,10 +75,10 @@@ static int hq_decode_block(HQContext *c
      }
  
      for (;;) {
-         val  = get_vlc2(gb, c->hq_ac_vlc.table, 9, 2);
+         val = get_vlc2(gb, c->hq_ac_vlc.table, 9, 2);
+         if (val < 0)
+             return AVERROR_INVALIDDATA;
          pos += ff_hq_ac_skips[val];
          if (pos >= 64)
              break;
@@@ -134,8 -137,10 +137,8 @@@ static int hq_decode_frame(HQContext *c
      ctx->avctx->pix_fmt             = AV_PIX_FMT_YUV422P;
  
      ret = ff_get_buffer(ctx->avctx, pic, 0);
 -    if (ret < 0) {
 -        av_log(ctx->avctx, AV_LOG_ERROR, "Could not allocate buffer.\n");
 +    if (ret < 0)
          return ret;
 -    }
  
      /* Offsets are stored from CUV position, so adjust them accordingly. */
      for (i = 0; i < profile->num_slices + 1; i++)
@@@ -262,8 -267,10 +265,8 @@@ static int hqa_decode_frame(HQContext *
      }
  
      ret = ff_get_buffer(ctx->avctx, pic, 0);
 -    if (ret < 0) {
 -        av_log(ctx->avctx, AV_LOG_ERROR, "Could not allocate buffer.\n");
 +    if (ret < 0)
          return ret;
 -    }
  
      /* Offsets are stored from HQA1 position, so adjust them accordingly. */
      for (i = 0; i < num_slices + 1; i++)