re: drm/bridge: it6505: fix HDCP CTS compare V matching

["Colin King (gmail)" <colin.i.king@gmail.com>]

Hi,

static analysis on Linux-next has found a potential issue with the 
following commit:

commit 0989c02c7a5c887c70deafen80c64d0291624e1a7
Author: Hermes Wu <hermes.wu@ite.com.tw>
Date:   Mon Dec 30 18:51:26 2024 +0800

     drm/bridge: it6505: fix HDCP CTS compare V matching


The issue is as follows:

Source: drivers/gpu/drm/bridge/ite-it6505.c, function: 
it6505_hdcp_part2_ksvlist_check:

             for (i = 0; i < 5; i++) {
                    if (bv[i][3] != av[i][0] || bv[i][2] != av[i][1] ||
                        av[i][1] != av[i][2] || bv[i][0] != av[i][3])
                             break;

                     DRM_DEV_DEBUG_DRIVER(dev, "V' all match!! %d, %d", 
retry, i);
                     return true;
             }


in the above for-loop, only iteration with i = 0 is performed, either 
the if statement breaks out of the loop if the condition is true or
we reach the return true statement. Both conditions means the loop is 
never iterated, yet the loop is expected to iterate for 5 times. This 
looks incorrect.

Colin