arrfab / rpms / libreport

Forked from rpms/libreport 5 years ago
Clone

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

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