[igt-dev] [PATCH i-g-t 1/2] lib/igt_eld: fix potential read from undefined value

Simon Ser simon.ser at intel.com
Tue Jun 18 12:55:42 UTC 2019


If the ELD doesn't contain a monitor_present line (this shouldn't happen), then
the value is garbage. Define the variable as a safety net.

Signed-off-by: Simon Ser <simon.ser at intel.com>
---
 lib/igt_eld.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/igt_eld.c b/lib/igt_eld.c
index 15dd8ac95189..3d7fd4dd0a82 100644
--- a/lib/igt_eld.c
+++ b/lib/igt_eld.c
@@ -152,7 +152,7 @@ static bool eld_parse_entry(const char *path, struct eld_entry *eld)
 	char buf[1024];
 	char *key, *value, *sad_key;
 	size_t len;
-	bool monitor_present;
+	bool monitor_present = false;
 	int sad_index;
 
 	memset(eld, 0, sizeof(*eld));
-- 
2.22.0



More information about the igt-dev mailing list