[CRIU] [PATCH 08/14] criu/namespaces.c: fix pr_perror usage
Kirill Tkhai
ktkhai at virtuozzo.com
Wed Mar 22 02:57:25 PDT 2017
On 22.03.2017 01:01, Kir Kolyshkin wrote:
> 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>
> Signed-off-by: Kir Kolyshkin <kir at openvz.org>
Reviewed-by: Kirill Tkhai <ktkhai at virtuozzo.com>
> ---
> 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;
> }
> }
>
More information about the CRIU
mailing list