[lxc-devel] [PATCH 2/2] Use SYSERROR instead of perror
Serge Hallyn
serge.hallyn at ubuntu.com
Sat Apr 4 01:54:10 UTC 2015
Quoting Tycho Andersen (tycho.andersen at canonical.com):
> It is impolite to print stuff to stderror owned by other people, and they might
> miss it anyway since it's not in the normal log location.
>
> Signed-off-by: Tycho Andersen <tycho.andersen at canonical.com>
Acked-by: Serge E. Hallyn <serge.hallyn at ubuntu.com>
> ---
> src/lxc/lxccontainer.c | 10 ++++------
> 1 file changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/src/lxc/lxccontainer.c b/src/lxc/lxccontainer.c
> index 88cdf55..528101b 100644
> --- a/src/lxc/lxccontainer.c
> +++ b/src/lxc/lxccontainer.c
> @@ -3880,7 +3880,7 @@ static bool lxcapi_checkpoint(struct lxc_container *c, char *directory, bool sto
> } else {
> pid_t w = waitpid(pid, &status, 0);
> if (w == -1) {
> - perror("waitpid");
> + SYSERROR("waitpid");
> return false;
> }
>
> @@ -4003,7 +4003,7 @@ static void do_restore(struct lxc_container *c, int pipe, char *directory, bool
>
> pid_t w = waitpid(pid, &status, 0);
> if (w == -1) {
> - perror("waitpid");
> + SYSERROR("waitpid");
> goto out_fini_handler;
> }
>
> @@ -4012,8 +4012,7 @@ static void do_restore(struct lxc_container *c, int pipe, char *directory, bool
> pipe = -1;
>
> if (sizeof(status) != ret) {
> - perror("write");
> - ERROR("failed to write all of status");
> + SYSERROR("failed to write all of status");
> goto out_fini_handler;
> }
>
> @@ -4024,8 +4023,7 @@ static void do_restore(struct lxc_container *c, int pipe, char *directory, bool
> int ret;
> FILE *f = fopen(pidfile, "r");
> if (!f) {
> - perror("reading pidfile");
> - ERROR("couldn't read restore's init pidfile %s\n", pidfile);
> + SYSERROR("couldn't read restore's init pidfile %s\n", pidfile);
> goto out_fini_handler;
> }
>
> --
> 2.1.4
>
> _______________________________________________
> lxc-devel mailing list
> lxc-devel at lists.linuxcontainers.org
> http://lists.linuxcontainers.org/listinfo/lxc-devel
More information about the lxc-devel
mailing list