From 999267d07240f0a9d8a85c0521b311331081b605 Mon Sep 17 00:00:00 2001 From: Colin Ian King Date: Wed, 7 Apr 2021 15:48:31 +0200 Subject: [PATCH] media: venus: core,pm: fix potential infinite loop The for-loop iterates with a u8 loop counter i and compares this with the loop upper limit of res->resets_num which is an unsigned int type. There is a potential infinite loop if res->resets_num is larger than the u8 loop counter i. Fix this by making the loop counter the same type as res->resets_num. Addresses-Coverity: ("Infinite loop") Fixes: 3bca43585e22 ("media: venus: core,pm: Add handling for resets") Signed-off-by: Colin Ian King Signed-off-by: Stanimir Varbanov Signed-off-by: Mauro Carvalho Chehab --- drivers/media/platform/qcom/venus/pm_helpers.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/qcom/venus/pm_helpers.c b/drivers/media/platform/qcom/venus/pm_helpers.c index dfe3ee84eeb6..5c0a9aa23e83 100644 --- a/drivers/media/platform/qcom/venus/pm_helpers.c +++ b/drivers/media/platform/qcom/venus/pm_helpers.c @@ -868,7 +868,7 @@ skip_pmdomains: static int core_resets_reset(struct venus_core *core) { const struct venus_resources *res = core->res; - unsigned char i; + unsigned int i; int ret; if (!res->resets_num) @@ -893,7 +893,7 @@ static int core_resets_get(struct venus_core *core) { struct device *dev = core->dev; const struct venus_resources *res = core->res; - unsigned char i; + unsigned int i; int ret; if (!res->resets_num) -- 2.11.0