]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
machine-image,mount-setup: minor coding style fixes 8607/head
authorLennart Poettering <lennart@poettering.net>
Wed, 28 Mar 2018 15:07:42 +0000 (17:07 +0200)
committerLennart Poettering <lennart@poettering.net>
Wed, 28 Mar 2018 20:04:58 +0000 (22:04 +0200)
src/core/mount-setup.c
src/shared/machine-image.c

index a0b5bff5a7af6d5c7b2df0ac67f8ad3706273b50..6916c63b2d855bb7e10669398ed867826a5a90f9 100644 (file)
@@ -397,19 +397,18 @@ static int relabel_cgroup_filesystems(void) {
                    only when the filesystem has been already populated by a previous instance of systemd
                    running from initrd. Otherwise don't remount anything and leave the filesystem read-write
                    for the cgroup filesystems to be mounted inside. */
-                r = statfs("/sys/fs/cgroup", &st);
-                if (r < 0) {
+                if (statfs("/sys/fs/cgroup", &st) < 0)
                         return log_error_errno(errno, "Failed to determine mount flags for /sys/fs/cgroup: %m");
-                }
 
                 if (st.f_flags & ST_RDONLY)
                         (void) mount(NULL, "/sys/fs/cgroup", NULL, MS_REMOUNT, NULL);
 
                 (void) label_fix("/sys/fs/cgroup", 0);
-                nftw("/sys/fs/cgroup", nftw_cb, 64, FTW_MOUNT|FTW_PHYS|FTW_ACTIONRETVAL);
+                (void) nftw("/sys/fs/cgroup", nftw_cb, 64, FTW_MOUNT|FTW_PHYS|FTW_ACTIONRETVAL);
 
                 if (st.f_flags & ST_RDONLY)
                         (void) mount(NULL, "/sys/fs/cgroup", NULL, MS_REMOUNT|MS_RDONLY, NULL);
+
         } else if (r < 0)
                 return log_error_errno(r, "Failed to determine whether we are in all unified mode: %m");
 
@@ -435,9 +434,9 @@ int mount_setup(bool loaded_policy) {
 
                 before_relabel = now(CLOCK_MONOTONIC);
 
-                nftw("/dev", nftw_cb, 64, FTW_MOUNT|FTW_PHYS|FTW_ACTIONRETVAL);
-                nftw("/dev/shm", nftw_cb, 64, FTW_MOUNT|FTW_PHYS|FTW_ACTIONRETVAL);
-                nftw("/run", nftw_cb, 64, FTW_MOUNT|FTW_PHYS|FTW_ACTIONRETVAL);
+                (void) nftw("/dev", nftw_cb, 64, FTW_MOUNT|FTW_PHYS|FTW_ACTIONRETVAL);
+                (void) nftw("/dev/shm", nftw_cb, 64, FTW_MOUNT|FTW_PHYS|FTW_ACTIONRETVAL);
+                (void) nftw("/run", nftw_cb, 64, FTW_MOUNT|FTW_PHYS|FTW_ACTIONRETVAL);
 
                 r = relabel_cgroup_filesystems();
                 if (r < 0)
index faa2eb0343077befbbb278f673d6ae17009e5463..89fd8c5dba41724f2523b98b28696e353c71b2da 100644 (file)
@@ -879,8 +879,7 @@ int image_path_lock(const char *path, int operation, LockFile *global, LockFile
                 r = make_lock_file_for(path, operation, &t);
                 if (r < 0) {
                         if ((operation & LOCK_SH) && r == -EROFS)
-                                log_debug_errno(r, "Failed to create shared "
-                                    "lock for %s: %m", path);
+                                log_debug_errno(r, "Failed to create shared lock for '%s', ignoring: %m", path);
                         else
                                 return r;
                 }