pass: thermal: Fix possible null deference in thermal_free() 55/296955/2 accepted/tizen_unified_riscv accepted/tizen/unified/20230808.170202 accepted/tizen/unified/riscv/20230809.013248
authorSeung-Woo Kim <sw0312.kim@samsung.com>
Tue, 8 Aug 2023 02:39:33 +0000 (11:39 +0900)
committerSeung-Woo Kim <sw0312.kim@samsung.com>
Tue, 8 Aug 2023 02:49:41 +0000 (11:49 +0900)
When there is no thermal scenario, thermal can not be initialized
and g_thermal is null. Exiting pass from the status, in
thermal_free(), there is null deference. Fix the possible null
deference.

Change-Id: I69ab200ea3d182adc628913d89306db6fe3ebcff
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
src/thermal/thermal.c

index 40773c2..7ad7c0a 100644 (file)
@@ -91,10 +91,12 @@ static void thermal_free(void)
        if (ret < 0)
                _E("failed to put Thermal Monitor scenario\n");
 
-       g_ptr_array_free(g_thermal->resources, TRUE);
+       if (g_thermal) {
+               g_ptr_array_free(g_thermal->resources, TRUE);
 
-       free(g_thermal);
-       g_thermal = NULL;
+               free(g_thermal);
+               g_thermal = NULL;
+       }
 }
 
 /**