touchscreen: clean-up uninitialized variable warning by static analysis 20/287220/1
authorYunhee Seo <yuni.seo@samsung.com>
Thu, 19 Jan 2023 06:24:23 +0000 (15:24 +0900)
committeryunhee seo <yuni.seo@samsung.com>
Wed, 25 Jan 2023 03:02:09 +0000 (03:02 +0000)
Add return code to avoid reaching uninitialized enum variable "state".

Change-Id: Idce5e24bc2c979c350629b8e413494dd62003cf5
Signed-off-by: Yunhee Seo <yuni.seo@samsung.com>
(cherry picked from commit 5d024fdb8bd0ab5d493ddaeeccd0fff5bc9472f8)

src/touchscreen/touchscreen.c

index 183a407..d7d0c18 100644 (file)
@@ -242,8 +242,10 @@ static int touchscreen_dump(FILE *fp, int mode, void *dump_data)
        if (!touchscreen_dev_available || (ret == -ENODEV))
                return 0;
 
-       if (ret < 0)
+       if (ret < 0) {
                _E("Failed to get touchscreen state: %d", ret);
+               return ret;
+       }
 
        if (state == TOUCHSCREEN_ON)
                LOG_DUMP(fp, "Touchscreen is enabled\n");