ryantimwilson / rpms / systemd

Forked from rpms/systemd a month ago
Clone
Zbigniew Jędrzejewski-Szmek 62fe94
From cbb452e7ef4419666af7ad343786ee54c23dd977 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 62fe94
From: Thomas Hindoe Paaboel Andersen <phomes@gmail.com>
Zbigniew Jędrzejewski-Szmek 62fe94
Date: Fri, 19 Sep 2014 01:43:04 +0200
Zbigniew Jędrzejewski-Szmek 62fe94
Subject: [PATCH] test: warn if we could not parse the loop count argument
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
Found by coverity. Fixes: CID#1237512
Zbigniew Jędrzejewski-Szmek 62fe94
---
Zbigniew Jędrzejewski-Szmek 62fe94
 src/journal/test-journal-init.c | 8 ++++++--
Zbigniew Jędrzejewski-Szmek 62fe94
 1 file changed, 6 insertions(+), 2 deletions(-)
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
diff --git a/src/journal/test-journal-init.c b/src/journal/test-journal-init.c
Zbigniew Jędrzejewski-Szmek 62fe94
index ada2f6c3e4..11fb150fe8 100644
Zbigniew Jędrzejewski-Szmek 62fe94
--- a/src/journal/test-journal-init.c
Zbigniew Jędrzejewski-Szmek 62fe94
+++ b/src/journal/test-journal-init.c
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -31,8 +31,12 @@ int main(int argc, char *argv[]) {
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         log_set_max_level(LOG_DEBUG);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (argc >= 2)
Zbigniew Jędrzejewski-Szmek 62fe94
-                safe_atoi(argv[1], &I);
Zbigniew Jędrzejewski-Szmek 62fe94
+        if (argc >= 2) {
Zbigniew Jędrzejewski-Szmek 62fe94
+                r = safe_atoi(argv[1], &I);
Zbigniew Jędrzejewski-Szmek 62fe94
+                if (r < 0)
Zbigniew Jędrzejewski-Szmek 62fe94
+                        log_info("Could not parse loop count argument. Using default.");
Zbigniew Jędrzejewski-Szmek 62fe94
+        }
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
         log_info("Running %d loops", I);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         assert_se(mkdtemp(t));