ryantimwilson / rpms / systemd

Forked from rpms/systemd 2 months ago
Clone
Zbigniew Jędrzejewski-Szmek 5d6eed
From 59ebe5b42ccd7cacbf5975f8ddc1aa42c78e45db Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 03e93e
From: Lennart Poettering <lennart@poettering.net>
Zbigniew Jędrzejewski-Szmek 03e93e
Date: Tue, 6 Dec 2016 19:36:30 +0100
Zbigniew Jędrzejewski-Szmek 03e93e
Subject: [PATCH] sysv-generator: properly translate sysv facilities
Zbigniew Jędrzejewski-Szmek 03e93e
Zbigniew Jędrzejewski-Szmek 03e93e
We used the wrong return value in one case, so that our translations were
Zbigniew Jędrzejewski-Szmek 03e93e
thrown away.
Zbigniew Jędrzejewski-Szmek 03e93e
Zbigniew Jędrzejewski-Szmek 03e93e
While we are at it, make sure to always initialize *ret on successful function
Zbigniew Jędrzejewski-Szmek 03e93e
exits.
Zbigniew Jędrzejewski-Szmek 03e93e
Zbigniew Jędrzejewski-Szmek 03e93e
Fixes: #4762
Zbigniew Jędrzejewski-Szmek 03e93e
(cherry picked from commit e932f5407ef5ad05d25d7dfefa4cda0fe81cc346)
Zbigniew Jędrzejewski-Szmek 03e93e
---
Zbigniew Jędrzejewski-Szmek 03e93e
 src/sysv-generator/sysv-generator.c | 10 +++++++---
Zbigniew Jędrzejewski-Szmek 03e93e
 1 file changed, 7 insertions(+), 3 deletions(-)
Zbigniew Jędrzejewski-Szmek 03e93e
Zbigniew Jędrzejewski-Szmek 03e93e
diff --git a/src/sysv-generator/sysv-generator.c b/src/sysv-generator/sysv-generator.c
Zbigniew Jędrzejewski-Szmek 03e93e
index c2c80175a2..212cf7a988 100644
Zbigniew Jędrzejewski-Szmek 03e93e
--- a/src/sysv-generator/sysv-generator.c
Zbigniew Jędrzejewski-Szmek 03e93e
+++ b/src/sysv-generator/sysv-generator.c
Zbigniew Jędrzejewski-Szmek 03e93e
@@ -292,8 +292,10 @@ static int sysv_translate_facility(SysvStub *s, unsigned line, const char *name,
Zbigniew Jędrzejewski-Szmek 03e93e
                 if (!streq(table[i], n))
Zbigniew Jędrzejewski-Szmek 03e93e
                         continue;
Zbigniew Jędrzejewski-Szmek 03e93e
 
Zbigniew Jędrzejewski-Szmek 03e93e
-                if (!table[i+1])
Zbigniew Jędrzejewski-Szmek 03e93e
+                if (!table[i+1]) {
Zbigniew Jędrzejewski-Szmek 03e93e
+                        *ret = NULL;
Zbigniew Jędrzejewski-Szmek 03e93e
                         return 0;
Zbigniew Jędrzejewski-Szmek 03e93e
+                }
Zbigniew Jędrzejewski-Szmek 03e93e
 
Zbigniew Jędrzejewski-Szmek 03e93e
                 m = strdup(table[i+1]);
Zbigniew Jędrzejewski-Szmek 03e93e
                 if (!m)
Zbigniew Jędrzejewski-Szmek 03e93e
@@ -312,7 +314,7 @@ static int sysv_translate_facility(SysvStub *s, unsigned line, const char *name,
Zbigniew Jędrzejewski-Szmek 03e93e
                 if (r < 0)
Zbigniew Jędrzejewski-Szmek 03e93e
                         return log_error_errno(r, "[%s:%u] Could not build name for facility %s: %m", s->path, line, name);
Zbigniew Jędrzejewski-Szmek 03e93e
 
Zbigniew Jędrzejewski-Szmek 03e93e
-                return r;
Zbigniew Jędrzejewski-Szmek 03e93e
+                return 1;
Zbigniew Jędrzejewski-Szmek 03e93e
         }
Zbigniew Jędrzejewski-Szmek 03e93e
 
Zbigniew Jędrzejewski-Szmek 03e93e
         /* Strip ".sh" suffix from file name for comparison */
Zbigniew Jędrzejewski-Szmek 03e93e
@@ -324,8 +326,10 @@ static int sysv_translate_facility(SysvStub *s, unsigned line, const char *name,
Zbigniew Jędrzejewski-Szmek 03e93e
         }
Zbigniew Jędrzejewski-Szmek 03e93e
 
Zbigniew Jędrzejewski-Szmek 03e93e
         /* Names equaling the file name of the services are redundant */
Zbigniew Jędrzejewski-Szmek 03e93e
-        if (streq_ptr(n, filename))
Zbigniew Jędrzejewski-Szmek 03e93e
+        if (streq_ptr(n, filename)) {
Zbigniew Jędrzejewski-Szmek 03e93e
+                *ret = NULL;
Zbigniew Jędrzejewski-Szmek 03e93e
                 return 0;
Zbigniew Jędrzejewski-Szmek 03e93e
+        }
Zbigniew Jędrzejewski-Szmek 03e93e
 
Zbigniew Jędrzejewski-Szmek 03e93e
         /* Everything else we assume to be normal service names */
Zbigniew Jędrzejewski-Szmek 03e93e
         m = sysv_translate_name(n);