teknoraver / rpms / systemd

Forked from rpms/systemd 2 months ago
Clone

Blame 0434-shared-util-use-nicer-idiom-to-silence-Coverity.patch

Zbigniew Jędrzejewski-Szmek 62fe94
From acb3b3ddc082880eaaa1d7773296ad9abd756f23 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 62fe94
From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl>
Zbigniew Jędrzejewski-Szmek 62fe94
Date: Tue, 30 Sep 2014 07:44:04 -0400
Zbigniew Jędrzejewski-Szmek 62fe94
Subject: [PATCH] shared: util - use nicer idiom to silence Coverity
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
Change the other spot too.
Zbigniew Jędrzejewski-Szmek 62fe94
---
Zbigniew Jędrzejewski-Szmek 62fe94
 src/shared/util.c | 9 ++-------
Zbigniew Jędrzejewski-Szmek 62fe94
 1 file changed, 2 insertions(+), 7 deletions(-)
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
diff --git a/src/shared/util.c b/src/shared/util.c
Zbigniew Jędrzejewski-Szmek 62fe94
index ebacee55b1..bbd9bd11d6 100644
Zbigniew Jędrzejewski-Szmek 62fe94
--- a/src/shared/util.c
Zbigniew Jędrzejewski-Szmek 62fe94
+++ b/src/shared/util.c
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -3306,13 +3306,8 @@ unsigned lines(void) {
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         l = 0;
Zbigniew Jędrzejewski-Szmek 62fe94
         e = getenv("LINES");
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (e) {
Zbigniew Jędrzejewski-Szmek 62fe94
-                int r;
Zbigniew Jędrzejewski-Szmek 62fe94
-
Zbigniew Jędrzejewski-Szmek 62fe94
-                r = safe_atou(e, &l);
Zbigniew Jędrzejewski-Szmek 62fe94
-                if (r < 0) {}
Zbigniew Jędrzejewski-Szmek 62fe94
-                        /* do nothing, we fall back to l = 0 */
Zbigniew Jędrzejewski-Szmek 62fe94
-        }
Zbigniew Jędrzejewski-Szmek 62fe94
+        if (e)
Zbigniew Jędrzejewski-Szmek 62fe94
+                (void) safe_atou(e, &l);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         if (l <= 0)
Zbigniew Jędrzejewski-Szmek 62fe94
                 l = fd_lines(STDOUT_FILENO);