From eabc356187abaa975da2ca6c66179c71423c4a79 Mon Sep 17 00:00:00 2001 From: Yunhee Seo Date: Thu, 19 Jan 2023 15:24:23 +0900 Subject: [PATCH] touchscreen: clean-up uninitialized variable warning by static analysis Add return code to avoid reaching uninitialized enum variable "state". Change-Id: Idce5e24bc2c979c350629b8e413494dd62003cf5 Signed-off-by: Yunhee Seo --- src/touchscreen/touchscreen.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/touchscreen/touchscreen.c b/src/touchscreen/touchscreen.c index 183a407..d7d0c18 100644 --- a/src/touchscreen/touchscreen.c +++ b/src/touchscreen/touchscreen.c @@ -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"); -- 2.7.4