h264: move 444 border xchg under if (deblock_top)
authorMichael Niedermayer <michaelni@gmx.at>
Mon, 1 Jul 2013 23:41:19 +0000 (01:41 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Tue, 2 Jul 2013 01:27:04 +0000 (03:27 +0200)
Fixes out of array access
Fixes Ticket2668

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
libavcodec/h264.c

index edd34ae..c436fdb 100644 (file)
@@ -2230,7 +2230,6 @@ static av_always_inline void xchg_mb_border(H264Context *h, uint8_t *src_y,
             XCHG(h->top_borders[top_idx][h->mb_x + 1],
                  src_y + (17 << pixel_shift), 1);
         }
-    }
     if (simple || !CONFIG_GRAY || !(h->flags & CODEC_FLAG_GRAY)) {
         if (chroma444) {
             if (deblock_topleft) {
@@ -2246,16 +2245,15 @@ static av_always_inline void xchg_mb_border(H264Context *h, uint8_t *src_y,
                 XCHG(h->top_borders[top_idx][h->mb_x + 1] + (32 << pixel_shift), src_cr + (17 << pixel_shift), 1);
             }
         } else {
-            if (deblock_top) {
                 if (deblock_topleft) {
                     XCHG(top_border_m1 + (16 << pixel_shift), src_cb - (7 << pixel_shift), 1);
                     XCHG(top_border_m1 + (24 << pixel_shift), src_cr - (7 << pixel_shift), 1);
                 }
                 XCHG(top_border + (16 << pixel_shift), src_cb + 1 + pixel_shift, 1);
                 XCHG(top_border + (24 << pixel_shift), src_cr + 1 + pixel_shift, 1);
-            }
         }
     }
+    }
 }
 
 static av_always_inline int dctcoef_get(int16_t *mb, int high_bit_depth,