commit f436b12e24a1e10fc612ece052adfb9dc8206ff0
author Luc Trudeau <luc@trud.ca> Wed May 10 15:03:23 2017 -0400
committer Luc Trudeau <luc@trud.ca> Fri May 12 00:27:11 2017 +0000
tree c00656393d419a8f1bf81d7c36d07ae5918585e2
parent 367da993a3fd7d1125486de6aa74fbedb06a0a87

[CFL] Check for uninitialized signs

To avoid writting out uninitialized signs for CfL alpha, checks are
added in the write_cfl_alphas function.

Change-Id: I832875e496a6318132c82bc1e5406dc95ae9c055

av1/encoder/bitstream.c

diff --git a/av1/encoder/bitstream.c b/av1/encoder/bitstream.c
index 7f4c739..fb21bcf 100644
--- a/av1/encoder/bitstream.c
+++ b/av1/encoder/bitstream.c
@@ -1703,6 +1703,12 @@
     assert(signs[CFL_PRED_U] == CFL_SIGN_POS);
     assert(signs[CFL_PRED_V] == CFL_SIGN_POS);
   } else {
+    // Check for uninitialized signs
+    if (cfl_alpha_codes[ind][CFL_PRED_U] == 0)
+      assert(signs[CFL_PRED_U] == CFL_SIGN_POS);
+    if (cfl_alpha_codes[ind][CFL_PRED_V] == 0)
+      assert(signs[CFL_PRED_V] == CFL_SIGN_POS);
+
     // Write a symbol representing a combination of alpha Cb and alpha Cr.
     aom_write_symbol(w, ind, frame_ctx->cfl_alpha_cdf, CFL_ALPHABET_SIZE);