drm/amd/display: Fix Coverity INTEGER_OVERFLOW within dal_gpio_service_create
authorHersen Wu <hersenxs.wu@amd.com>
Fri, 26 Apr 2024 15:58:11 +0000 (11:58 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 2 May 2024 20:18:17 +0000 (16:18 -0400)
[Why]
For subtraction, coverity reports integer overflow
warning message when variable type is uint32_t.

[How]
Change variable type to int32_t.

Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Acked-by: Tom Chung <chiahsuan.chung@amd.com>
Signed-off-by: Hersen Wu <hersenxs.wu@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/gpio/gpio_service.c

index d19d5c1..f344478 100644 (file)
@@ -56,7 +56,7 @@ struct gpio_service *dal_gpio_service_create(
        struct dc_context *ctx)
 {
        struct gpio_service *service;
-       uint32_t index_of_id;
+       int32_t index_of_id;
 
        service = kzalloc(sizeof(struct gpio_service), GFP_KERNEL);
 
@@ -112,7 +112,7 @@ struct gpio_service *dal_gpio_service_create(
        return service;
 
 failure_2:
-       while (index_of_id) {
+       while (index_of_id > 0) {
                --index_of_id;
                kfree(service->busyness[index_of_id]);
        }