Commit ac8cdc8c authored by Alex Converse's avatar Alex Converse

misc_fixes: Don't use uninitialized probability data.

(cherry picked from commit 7a6cb59d)

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=2

Change-Id: I17b76fcf0d8c191850350d5aa50dcc007b8b0cdc
parent ee9ac321
...@@ -276,12 +276,11 @@ void av1_choose_segmap_coding_method(AV1_COMMON *cm, MACROBLOCKD *xd) { ...@@ -276,12 +276,11 @@ void av1_choose_segmap_coding_method(AV1_COMMON *cm, MACROBLOCKD *xd) {
const int count0 = temporal_predictor_count[i][0]; const int count0 = temporal_predictor_count[i][0];
const int count1 = temporal_predictor_count[i][1]; const int count1 = temporal_predictor_count[i][1];
t_nopred_prob[i] = get_binary_prob(count0, count1);
#if CONFIG_MISC_FIXES #if CONFIG_MISC_FIXES
av1_prob_diff_update_savings_search(temporal_predictor_count[i], av1_prob_diff_update_savings_search(temporal_predictor_count[i],
segp->pred_probs[i], segp->pred_probs[i],
&t_nopred_prob[i], DIFF_UPDATE_PROB); &t_nopred_prob[i], DIFF_UPDATE_PROB);
#else
t_nopred_prob[i] = get_binary_prob(count0, count1);
#endif #endif
// Add in the predictor signaling cost // Add in the predictor signaling cost
......
...@@ -123,10 +123,7 @@ TEST_P(ActiveMapRefreshTest, Test) { ...@@ -123,10 +123,7 @@ TEST_P(ActiveMapRefreshTest, Test) {
ASSERT_NO_FATAL_FAILURE(RunLoop(&video)); ASSERT_NO_FATAL_FAILURE(RunLoop(&video));
} }
INSTANTIATE_TEST_CASE_P( AV1_INSTANTIATE_TEST_CASE(ActiveMapRefreshTest,
DISABLED_AV1, ActiveMapRefreshTest, ::testing::Values(::libaom_test::kRealTime),
::testing::Combine( ::testing::Range(5, 6));
::testing::Values(
static_cast<const libaom_test::CodecFactory *>(&libaom_test::kAV1)),
::testing::Values(::libaom_test::kRealTime), ::testing::Range(5, 6)));
} // namespace } // namespace
...@@ -81,10 +81,7 @@ TEST_P(ActiveMapTest, Test) { ...@@ -81,10 +81,7 @@ TEST_P(ActiveMapTest, Test) {
ASSERT_NO_FATAL_FAILURE(RunLoop(&video)); ASSERT_NO_FATAL_FAILURE(RunLoop(&video));
} }
INSTANTIATE_TEST_CASE_P( AV1_INSTANTIATE_TEST_CASE(ActiveMapTest,
DISABLED_AV1, ActiveMapTest, ::testing::Values(::libaom_test::kRealTime),
::testing::Combine( ::testing::Range(0, 6));
::testing::Values(
static_cast<const libaom_test::CodecFactory *>(&libaom_test::kAV1)),
::testing::Values(::libaom_test::kRealTime), ::testing::Range(0, 6)));
} // namespace } // namespace
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment