commit ce179a6eef0fc8a9488e6689e502836559891061
author Yingbin Wang <yingbinwang@tencent.com> Fri May 20 17:02:40 2022 +0000
committer Urvang Joshi <urvang@google.com> Fri May 20 17:02:40 2022 +0000
tree 0fdc9163f7991bfa08b6cfe89e468ff363ffed6a
parent 839f141ab70106949652edcb1f4c5d268e821da6

Fix the mismatch for IntraBC when turning off SDP

When SDP is turned off, there is a bug in the is_bv_valid function that may allow the use of illegal block vectors for chroma blocks. This MR tries to fix the mismatch for IntraBC by fixing this bug.

av1/common/mvref_common.h

diff --git a/av1/common/mvref_common.h b/av1/common/mvref_common.h
index 8546251..a424509 100644
--- a/av1/common/mvref_common.h
+++ b/av1/common/mvref_common.h
@@ -618,7 +618,7 @@
       int tmp_col = mi_col;
       int tmp_bh = bh;
       int tmp_bw = bw;
-      if (!frame_is_intra_only(cm)) {
+      if (!cm->seq_params.enable_sdp || !frame_is_intra_only(cm)) {
         if (xd->is_chroma_ref && av1_num_planes(cm) > 1) {
           const struct macroblockd_plane *const pd = &xd->plane[1];
           if ((bw < 8 && pd->subsampling_x) && (bh < 8 && pd->subsampling_y)) {