Zbigniew Jędrzejewski-Szmek 62fe94
From d457ff8319b1e7c522c146f75e272f1226f4720c Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 62fe94
From: Tom Gundersen <teg@jklm.no>
Zbigniew Jędrzejewski-Szmek 62fe94
Date: Thu, 18 Sep 2014 19:07:02 +0200
Zbigniew Jędrzejewski-Szmek 62fe94
Subject: [PATCH] udevd: check return of various functions
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
One reported by Coverity. Fixes CID #996252.
Zbigniew Jędrzejewski-Szmek 62fe94
---
Zbigniew Jędrzejewski-Szmek 62fe94
 src/udev/udevd.c | 19 ++++++++++++++++---
Zbigniew Jędrzejewski-Szmek 62fe94
 1 file changed, 16 insertions(+), 3 deletions(-)
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
diff --git a/src/udev/udevd.c b/src/udev/udevd.c
Zbigniew Jędrzejewski-Szmek 62fe94
index 95c6468029..9210da6c47 100644
Zbigniew Jędrzejewski-Szmek 62fe94
--- a/src/udev/udevd.c
Zbigniew Jędrzejewski-Szmek 62fe94
+++ b/src/udev/udevd.c
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -1139,13 +1139,26 @@ int main(int argc, char *argv[]) {
Zbigniew Jędrzejewski-Szmek 62fe94
                 goto exit;
Zbigniew Jędrzejewski-Szmek 62fe94
         }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        label_init("/dev");
Zbigniew Jędrzejewski-Szmek 62fe94
+        r = label_init("/dev");
Zbigniew Jędrzejewski-Szmek 62fe94
+        if (r < 0) {
Zbigniew Jędrzejewski-Szmek 62fe94
+                log_error("could not initialize labelling: %s", strerror(-r));
Zbigniew Jędrzejewski-Szmek 62fe94
+                goto exit;
Zbigniew Jędrzejewski-Szmek 62fe94
+        }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         /* set umask before creating any file/directory */
Zbigniew Jędrzejewski-Szmek 62fe94
-        chdir("/");
Zbigniew Jędrzejewski-Szmek 62fe94
+        r = chdir("/");
Zbigniew Jędrzejewski-Szmek 62fe94
+        if (r < 0) {
Zbigniew Jędrzejewski-Szmek 62fe94
+                log_error("could not change dir to /: %m");
Zbigniew Jędrzejewski-Szmek 62fe94
+                goto exit;
Zbigniew Jędrzejewski-Szmek 62fe94
+        }
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
         umask(022);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        mkdir("/run/udev", 0755);
Zbigniew Jędrzejewski-Szmek 62fe94
+        r = mkdir("/run/udev", 0755);
Zbigniew Jędrzejewski-Szmek 62fe94
+        if (r < 0) {
Zbigniew Jędrzejewski-Szmek 62fe94
+                log_error("could not create /run/udev: %m");
Zbigniew Jędrzejewski-Szmek 62fe94
+                goto exit;
Zbigniew Jędrzejewski-Szmek 62fe94
+        }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         dev_setup(NULL);
Zbigniew Jędrzejewski-Szmek 62fe94