From: Alex Hung <alex.h...@amd.com>

[WHY]
fe_clk_en has size of 5 but sizeof(fe_clk_en) has byte size 20 which is
lager than the array size.

[HOW]
Divide byte size 20 by its element size.

This fixes 2 OVERRUN issues reported by Coverity.

Reviewed-by: Harry Wentland <harry.wentl...@amd.com>
Acked-by: Tom Chung <chiahsuan.ch...@amd.com>
Signed-off-by: Alex Hung <alex.h...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/dccg/dcn35/dcn35_dccg.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/dc/dccg/dcn35/dcn35_dccg.c 
b/drivers/gpu/drm/amd/display/dc/dccg/dcn35/dcn35_dccg.c
index 02ec16bf381f..1fc3aa2b507b 100644
--- a/drivers/gpu/drm/amd/display/dc/dccg/dcn35/dcn35_dccg.c
+++ b/drivers/gpu/drm/amd/display/dc/dccg/dcn35/dcn35_dccg.c
@@ -904,7 +904,7 @@ static uint8_t dccg35_get_other_enabled_symclk_fe(struct 
dccg *dccg, uint32_t st
        /* for DPMST, this backend could be used by multiple front end.
        only disable the backend if this stream_enc_ins is the last active 
stream enc connected to this back_end*/
                uint8_t i;
-               for (i = 0; i != link_enc_inst && i < sizeof(fe_clk_en); i++) {
+               for (i = 0; i != link_enc_inst && i < ARRAY_SIZE(fe_clk_en); 
i++) {
                        if (fe_clk_en[i] && be_clk_sel[i] == link_enc_inst)
                                num_enabled_symclk_fe++;
                }
-- 
2.34.1

Reply via email to