|
Jakub Filak |
395b2d |
From 54be469b8b91272e1f9852670a2f49e6bbecd0d8 Mon Sep 17 00:00:00 2001
|
|
Jakub Filak |
395b2d |
From: Jakub Filak <jfilak@redhat.com>
|
|
Jakub Filak |
395b2d |
Date: Wed, 22 Jan 2014 15:33:50 +0100
|
|
Jakub Filak |
395b2d |
Subject: [PATCH 1/2] Fix a pair of defects uncovered by coverity
|
|
Jakub Filak |
395b2d |
|
|
Jakub Filak |
395b2d |
---
|
|
Jakub Filak |
395b2d |
src/abrt-checker.c | 8 ++++++--
|
|
Jakub Filak |
395b2d |
src/jthread_map.c | 1 +
|
|
Jakub Filak |
395b2d |
2 files changed, 7 insertions(+), 2 deletions(-)
|
|
Jakub Filak |
395b2d |
|
|
Jakub Filak |
395b2d |
diff --git a/src/abrt-checker.c b/src/abrt-checker.c
|
|
Jakub Filak |
395b2d |
index 91485e0..713053c 100644
|
|
Jakub Filak |
395b2d |
--- a/src/abrt-checker.c
|
|
Jakub Filak |
395b2d |
+++ b/src/abrt-checker.c
|
|
Jakub Filak |
395b2d |
@@ -2886,7 +2886,11 @@ void parse_commandline_options(char *options)
|
|
Jakub Filak |
395b2d |
}
|
|
Jakub Filak |
395b2d |
else if (strcmp("executable", key) == 0)
|
|
Jakub Filak |
395b2d |
{
|
|
Jakub Filak |
395b2d |
- if (strcmp("threadclass", value) == 0)
|
|
Jakub Filak |
395b2d |
+ if (NULL == value || '\0' == value[0])
|
|
Jakub Filak |
395b2d |
+ {
|
|
Jakub Filak |
395b2d |
+ fprintf(stderr, "A value of '%s' option cannot be empty\n", key);
|
|
Jakub Filak |
395b2d |
+ }
|
|
Jakub Filak |
395b2d |
+ else if (strcmp("threadclass", value) == 0)
|
|
Jakub Filak |
395b2d |
{
|
|
Jakub Filak |
395b2d |
VERBOSE_PRINT("Use a thread class for 'executable'\n");
|
|
Jakub Filak |
395b2d |
executableFlags |= ABRT_EXECUTABLE_THREAD;
|
|
Jakub Filak |
395b2d |
@@ -2899,7 +2903,7 @@ void parse_commandline_options(char *options)
|
|
Jakub Filak |
395b2d |
}
|
|
Jakub Filak |
395b2d |
else
|
|
Jakub Filak |
395b2d |
{
|
|
Jakub Filak |
395b2d |
- fprintf(stderr, "Unknown 'executable' option's value '%s'\n", key);
|
|
Jakub Filak |
395b2d |
+ fprintf(stderr, "Unknown '%s' option's value '%s'\n", key, value);
|
|
Jakub Filak |
395b2d |
}
|
|
Jakub Filak |
395b2d |
}
|
|
Jakub Filak |
395b2d |
else
|
|
Jakub Filak |
395b2d |
diff --git a/src/jthread_map.c b/src/jthread_map.c
|
|
Jakub Filak |
395b2d |
index 4517398..4cb417b 100644
|
|
Jakub Filak |
395b2d |
--- a/src/jthread_map.c
|
|
Jakub Filak |
395b2d |
+++ b/src/jthread_map.c
|
|
Jakub Filak |
395b2d |
@@ -55,6 +55,7 @@ T_jthreadMap *jthread_map_new()
|
|
Jakub Filak |
395b2d |
if (NULL == map)
|
|
Jakub Filak |
395b2d |
{
|
|
Jakub Filak |
395b2d |
fprintf(stderr, __FILE__ ":" STRINGIZE(__LINE__) ": calloc() error\n");
|
|
Jakub Filak |
395b2d |
+ return NULL;
|
|
Jakub Filak |
395b2d |
}
|
|
Jakub Filak |
395b2d |
|
|
Jakub Filak |
395b2d |
pthread_mutex_init(&map->mutex, /*use default attributes*/NULL);
|
|
Jakub Filak |
395b2d |
--
|
|
Jakub Filak |
395b2d |
1.8.3.1
|
|
Jakub Filak |
395b2d |
|