arrfab / rpms / libreport

Forked from rpms/libreport 5 years ago
Clone

Blame SOURCES/0006-coverity-fix-def7.patch

91b162
From 6ef9f702af8df7acc69887656752a24cfd5a64c8 Mon Sep 17 00:00:00 2001
91b162
From: Matej Habrnal <mhabrnal@redhat.com>
91b162
Date: Wed, 18 Jul 2018 17:10:17 +0200
91b162
Subject: [PATCH] coverity fix [#def7]
91b162
91b162
libreport-2.9.5/src/gui-wizard-gtk/wizard.c:1028: dereference: Dereferencing a
91b162
null pointer "event_name_end".
91b162
91b162
https://cov01.lab.eng.brq.redhat.com/el8-results/el8/libreport-2.9.5-2.el8+7/scan-results.html#def7
91b162
91b162
Signed-off-by: Matej Habrnal <mhabrnal@redhat.com>
91b162
---
91b162
 src/gui-wizard-gtk/wizard.c | 3 +++
91b162
 1 file changed, 3 insertions(+)
91b162
91b162
diff --git a/src/gui-wizard-gtk/wizard.c b/src/gui-wizard-gtk/wizard.c
91b162
index cb9ac5bb..af4c255f 100644
91b162
--- a/src/gui-wizard-gtk/wizard.c
91b162
+++ b/src/gui-wizard-gtk/wizard.c
91b162
@@ -1024,6 +1024,9 @@ static event_gui_data_t *add_event_buttons(GtkBox *box,
91b162
             break;
91b162
 
91b162
         char *event_name_end = strchr(event_name, '\n');
91b162
+        if (event_name_end == NULL)
91b162
+            break;
91b162
+
91b162
         *event_name_end = '\0';
91b162
 
91b162
         event_config_t *cfg = get_event_config(event_name);
91b162
-- 
91b162
2.17.1
91b162