[CRIU] [PATCH 08/14] criu/namespaces.c: fix pr_perror usage

Kir Kolyshkin kir at openvz.org
Thu Mar 23 15:02:05 PDT 2017


1. No newline (pr_perror() already adds one for you)

2. No colon before file name, e.g.

WAS:	Can't open file: /some/file: permission denied
NOW:	Can't open file /some/file: permission denied

Cc: Kirill Tkhai <ktkhai at virtuozzo.com>
Cc: Dengguangxing <dengguangxing at huawei.com>
Reviewed-by: Kirill Tkhai <ktkhai at virtuozzo.com>
Signed-off-by: Kir Kolyshkin <kir at openvz.org>
---
 criu/namespaces.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/criu/namespaces.c b/criu/namespaces.c
index a0e7397..48a79a5 100644
--- a/criu/namespaces.c
+++ b/criu/namespaces.c
@@ -106,7 +106,7 @@ static int check_ns_file(char *ns_file)
 
 	ret = access(ns_file, 0);
 	if (ret < 0) {
-		pr_perror("Can't access join-ns file: %s", ns_file);
+		pr_perror("Can't access join-ns file %s", ns_file);
 		return -1;
 	}
 	return 0;
@@ -1366,7 +1366,7 @@ static int write_id_map(pid_t pid, UidGidExtent **extents, int n, char *id_map)
 	snprintf(buf, PAGE_SIZE, "%d/%s", pid, id_map);
 	fd = openat(fd, buf, O_WRONLY);
 	if (fd < 0) {
-		pr_perror("Can't open %s\n", buf);
+		pr_perror("Can't open %s", buf);
 		return -1;
 	}
 
@@ -2073,7 +2073,7 @@ static int get_join_ns_fd(struct join_ns *jn)
 
 	fd = open(pnsf, O_RDONLY);
 	if (fd < 0) {
-		pr_perror("Can't open ns file: %s", pnsf);
+		pr_perror("Can't open ns file %s", pnsf);
 		return -1;
 	}
 	jn->ns_fd = fd;
@@ -2254,7 +2254,7 @@ static int create_user_ns_hierarhy_fn(void *in_arg)
 
 		errno = 0;
 		if (wait(&status) < 0 || !WIFEXITED(status) || WEXITSTATUS(status)) {
-			pr_perror("Child process waiting: %d\n", status);
+			pr_perror("Child process waiting: %d", status);
 			goto out;
 		}
 	}
-- 
2.9.3



More information about the CRIU mailing list