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