]> git.proxmox.com Git - mirror_lxc.git/commitdiff
change log macro of error case from lxc_ambient_caps_up/down
author2xsec <dh48.jeong@samsung.com>
Sun, 22 Jul 2018 12:03:46 +0000 (21:03 +0900)
committerChristian Brauner <christian.brauner@ubuntu.com>
Tue, 24 Jul 2018 13:00:06 +0000 (15:00 +0200)
Signed-off-by: 2xsec <dh48.jeong@samsung.com>
src/lxc/caps.c
src/lxc/start.c

index ae5d7ef16f036f8f7ccc8105d655f524dafd5c6c..4c39ce5c98794c4643cce4657c71a2abcaf85f2b 100644 (file)
@@ -114,7 +114,6 @@ int lxc_caps_up(void)
        }
 
        for (cap = 0; cap <= CAP_LAST_CAP; cap++) {
-
                cap_flag_value_t flag;
 
                ret = cap_get_flag(caps, cap, CAP_PERMITTED, &flag);
@@ -201,8 +200,10 @@ int lxc_ambient_caps_up(void)
        }
 
        cap_names = cap_to_text(caps, NULL);
-       if (!cap_names)
+       if (!cap_names) {
+               SYSWARN("Failed to convert capabilities %d", cap);
                goto out;
+       }
 
        TRACE("Raised %s in inheritable and ambient capability set", cap_names);
 
@@ -311,6 +312,7 @@ static int _real_caps_last_cap(void)
                if ((n = read(fd, buf, 31)) >= 0) {
                        buf[n] = '\0';
                        errno = 0;
+
                        result = strtol(buf, &ptr, 10);
                        if (!ptr || (*ptr != '\0' && *ptr != '\n') || errno != 0)
                                result = -1;
@@ -323,7 +325,10 @@ static int _real_caps_last_cap(void)
        * each capability indiviually from the kernel */
        if (result < 0) {
                int cap = 0;
-               while (prctl(PR_CAPBSET_READ, cap) >= 0) cap++;
+
+               while (prctl(PR_CAPBSET_READ, cap) >= 0)
+                       cap++;
+
                result = cap - 1;
        }
 
@@ -333,7 +338,9 @@ static int _real_caps_last_cap(void)
 int lxc_caps_last_cap(void)
 {
        static int last_cap = -1;
-       if (last_cap < 0) last_cap = _real_caps_last_cap();
+
+       if (last_cap < 0)
+               last_cap = _real_caps_last_cap();
 
        return last_cap;
 }
index aa9f5c0eca8f93b11bcd6bffd849bbdbfe2fc7fb..739866d8437cf368d630d441f6b7c8744f7c4709 100644 (file)
@@ -1074,7 +1074,7 @@ static int do_start(void *data)
 
        ret = lxc_ambient_caps_up();
        if (ret < 0) {
-               SYSERROR("Failed to raise ambient capabilities");
+               ERROR("Failed to raise ambient capabilities");
                goto out_warn_father;
        }
 
@@ -1379,7 +1379,7 @@ static int do_start(void *data)
 
        ret = lxc_ambient_caps_down();
        if (ret < 0) {
-               SYSERROR("Failed to clear ambient capabilities");
+               ERROR("Failed to clear ambient capabilities");
                goto out_warn_father;
        }