[CRIU] [PATCH 2/8] util: set cr_errno to ESRCH if no PID dir in proc
Pavel Emelyanov
xemul at parallels.com
Tue Dec 2 00:42:18 PST 2014
On 12/02/2014 01:52 AM, Ruslan Kuprieiev wrote:
> Signed-off-by: Ruslan Kuprieiev <kupruser at gmail.com>
> ---
> include/cr-errno.h | 5 +++++
> util.c | 3 +++
> 2 files changed, 8 insertions(+)
>
> diff --git a/include/cr-errno.h b/include/cr-errno.h
> index 6ca6c68..bec72a8 100644
> --- a/include/cr-errno.h
> +++ b/include/cr-errno.h
> @@ -1,4 +1,9 @@
> #ifndef __CR_ERRNO_H__
> #define __CR_ERRNO_H__
> extern int cr_errno;
> +/*
> + * List of symbolic error names:
> + * ESRCH - no process can be found corresponding to that specified by pid
That's a nice question to thing about -- is it normal to use
glibc error codes for higher level errors? Thoughts?
> + */
> +
> #endif /* __CR_ERRNO_H__ */
> diff --git a/util.c b/util.c
> index dd76863..7b38351 100644
> --- a/util.c
> +++ b/util.c
> @@ -42,6 +42,8 @@
> #include "servicefd.h"
> #include "cr-service.h"
>
> +#include "cr-errno.h"
> +
> #define VMA_OPT_LEN 128
>
> static void vma_opt_str(const struct vma_area *v, char *opt)
> @@ -268,6 +270,7 @@ inline int open_pid_proc(pid_t pid)
> fd = openat(dfd, path, O_RDONLY);
> if (fd < 0) {
> pr_perror("Can't open %s", path);
> + cr_errno = ESRCH;
> return -1;
> }
>
>
More information about the CRIU
mailing list