Zbigniew Jędrzejewski-Szmek 708deb
From e8c8ddccfc63574069c30b7e75f0ccfd5b03eab9 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 708deb
From: Tom Gundersen <teg@jklm.no>
Zbigniew Jędrzejewski-Szmek 708deb
Date: Mon, 29 Sep 2014 13:20:54 +0200
Zbigniew Jędrzejewski-Szmek 708deb
Subject: [PATCH] nspawn: log when tearing down of loop device fails
Zbigniew Jędrzejewski-Szmek 708deb
Zbigniew Jędrzejewski-Szmek 708deb
---
Zbigniew Jędrzejewski-Szmek 708deb
 src/nspawn/nspawn.c | 13 ++++++++++---
Zbigniew Jędrzejewski-Szmek 708deb
 1 file changed, 10 insertions(+), 3 deletions(-)
Zbigniew Jędrzejewski-Szmek 708deb
Zbigniew Jędrzejewski-Szmek 708deb
diff --git a/src/nspawn/nspawn.c b/src/nspawn/nspawn.c
Zbigniew Jędrzejewski-Szmek 708deb
index 4c1cfabca4..34436b82a2 100644
Zbigniew Jędrzejewski-Szmek 708deb
--- a/src/nspawn/nspawn.c
Zbigniew Jędrzejewski-Szmek 708deb
+++ b/src/nspawn/nspawn.c
Zbigniew Jędrzejewski-Szmek 708deb
@@ -2607,20 +2607,27 @@ static int mount_devices(
Zbigniew Jędrzejewski-Szmek 708deb
 
Zbigniew Jędrzejewski-Szmek 708deb
 static void loop_remove(int nr, int *image_fd) {
Zbigniew Jędrzejewski-Szmek 708deb
         _cleanup_close_ int control = -1;
Zbigniew Jędrzejewski-Szmek 708deb
+        int r;
Zbigniew Jędrzejewski-Szmek 708deb
 
Zbigniew Jędrzejewski-Szmek 708deb
         if (nr < 0)
Zbigniew Jędrzejewski-Szmek 708deb
                 return;
Zbigniew Jędrzejewski-Szmek 708deb
 
Zbigniew Jędrzejewski-Szmek 708deb
         if (image_fd && *image_fd >= 0) {
Zbigniew Jędrzejewski-Szmek 708deb
-                ioctl(*image_fd, LOOP_CLR_FD);
Zbigniew Jędrzejewski-Szmek 708deb
+                r = ioctl(*image_fd, LOOP_CLR_FD);
Zbigniew Jędrzejewski-Szmek 708deb
+                if (r < 0)
Zbigniew Jędrzejewski-Szmek 708deb
+                        log_warning("Failed to close loop image: %m");
Zbigniew Jędrzejewski-Szmek 708deb
                 *image_fd = safe_close(*image_fd);
Zbigniew Jędrzejewski-Szmek 708deb
         }
Zbigniew Jędrzejewski-Szmek 708deb
 
Zbigniew Jędrzejewski-Szmek 708deb
         control = open("/dev/loop-control", O_RDWR|O_CLOEXEC|O_NOCTTY|O_NONBLOCK);
Zbigniew Jędrzejewski-Szmek 708deb
-        if (control < 0)
Zbigniew Jędrzejewski-Szmek 708deb
+        if (control < 0) {
Zbigniew Jędrzejewski-Szmek 708deb
+                log_warning("Failed to open /dev/loop-control: %m");
Zbigniew Jędrzejewski-Szmek 708deb
                 return;
Zbigniew Jędrzejewski-Szmek 708deb
+        }
Zbigniew Jędrzejewski-Szmek 708deb
 
Zbigniew Jędrzejewski-Szmek 708deb
-        ioctl(control, LOOP_CTL_REMOVE, nr);
Zbigniew Jędrzejewski-Szmek 708deb
+        r = ioctl(control, LOOP_CTL_REMOVE, nr);
Zbigniew Jędrzejewski-Szmek 708deb
+        if (r < 0)
Zbigniew Jędrzejewski-Szmek 708deb
+                log_warning("Failed to remove loop %d: %m", nr);
Zbigniew Jędrzejewski-Szmek 708deb
 }
Zbigniew Jędrzejewski-Szmek 708deb
 
Zbigniew Jędrzejewski-Szmek 708deb
 static int spawn_getent(const char *database, const char *key, pid_t *rpid) {