From: Ronald S. Bultje <rbultje@google.com>
Date: Thu, 16 Aug 2012 05:20:24 +0000 (-0700)
Subject: Fix stack overrun in calculating segmentation map coding method.
X-Git-Tag: v1.3.0~1217^2~298
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=0393e558a4710b31836ead8f9832f6b270daf57c;p=libvpx

Fix stack overrun in calculating segmentation map coding method.

This caused spurious crashes on encodes, frustratingly often only on
high-definition material.

Change-Id: Ia8c1dfc420d598ca92108bd9cc2df7450e8d10a2
---

diff --git a/vp8/encoder/segmentation.c b/vp8/encoder/segmentation.c
index 83b27a6e9..e9d02cdd4 100644
--- a/vp8/encoder/segmentation.c
+++ b/vp8/encoder/segmentation.c
@@ -111,7 +111,8 @@ static void calc_segtree_probs(MACROBLOCKD *xd,
   int i;
 
   // Blank the strtucture to start with
-  vpx_memset(segment_tree_probs, 0, sizeof(segment_tree_probs));
+  vpx_memset(segment_tree_probs, 0,
+             MB_FEATURE_TREE_PROBS * sizeof(*segment_tree_probs));
 
   // Total count for all segments
   count1 = segcounts[0] + segcounts[1];