]> granicus.if.org Git - libvpx/commitdiff
Return error on wrong sync code
authorYaowu Xu <yaowu@google.com>
Mon, 5 Jan 2015 18:49:06 +0000 (10:49 -0800)
committerYaowu Xu <yaowu@google.com>
Tue, 6 Jan 2015 01:19:32 +0000 (17:19 -0800)
Not initialize codec stream info

This fixes vp8 fuzzing issue reported at build #56

Change-Id: I307ac839b4f4bda3a02a551019c0d688dba19c86

vp8/vp8_dx_iface.c

index 6810644bad878a31a2e8225bc571b900bac3d4ce..67a0fef64af9bd6b3fd8d60bef2e732bbfa0768a 100644 (file)
@@ -188,7 +188,7 @@ static vpx_codec_err_t vp8_peek_si_internal(const uint8_t *data,
 
             /* vet via sync code */
             if (clear[3] != 0x9d || clear[4] != 0x01 || clear[5] != 0x2a)
-                res = VPX_CODEC_UNSUP_BITSTREAM;
+                return VPX_CODEC_UNSUP_BITSTREAM;
 
             si->w = (clear[6] | (clear[7] << 8)) & 0x3fff;
             si->h = (clear[8] | (clear[9] << 8)) & 0x3fff;
@@ -402,7 +402,7 @@ static vpx_codec_err_t vp8_decode(vpx_codec_alg_priv_t  *ctx,
     if (!res)
     {
         VP8D_COMP *pbi = ctx->yv12_frame_buffers.pbi[0];
-        if(resolution_change)
+        if (resolution_change)
         {
             VP8_COMMON *const pc = & pbi->common;
             MACROBLOCKD *const xd  = & pbi->mb;