diff options
| author | Kir Kolyshkin <kir@openvz.org> | 2016-12-17 14:21:56 +0300 |
|---|---|---|
| committer | Andrei Vagin <avagin@virtuozzo.com> | 2017-03-15 00:09:54 +0300 |
| commit | 1be75af874aba6515167462b69741309dd1dd2a6 (patch) | |
| tree | e894f1a74248de3bb8dffd070c1ecd932d2e3389 /compel/src | |
| parent | 7d181cda7376a0ea97fecd9e5376aa09cd1e19ef (diff) | |
compel_syscall(): make ret argument signed
I saw this line in the code
unsigned long sret = -ENOSYS;
and ended up with this patch. Note syscall(2) man page says return value
is long -- who am I to disagree?
travis-ci: success for More polishing for compel cli
Signed-off-by: Kir Kolyshkin <kir@openvz.org>
Acked-by: Cyrill Gorcunov <gorcunov@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com>
Signed-off-by: Andrei Vagin <avagin@virtuozzo.com>
Diffstat (limited to 'compel/src')
| -rw-r--r-- | compel/src/lib/infect.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/compel/src/lib/infect.c b/compel/src/lib/infect.c index 3dba438ef..599c37274 100644 --- a/compel/src/lib/infect.c +++ b/compel/src/lib/infect.c @@ -735,7 +735,7 @@ static int parasite_memfd_exchange(struct parasite_ctl *ctl, unsigned long size) void *where = (void *)ctl->ictx.syscall_ip + BUILTIN_SYSCALL_SIZE; uint8_t orig_code[MEMFD_FNAME_SZ] = MEMFD_FNAME; pid_t pid = ctl->rpid; - unsigned long sret = -ENOSYS; + long sret = -ENOSYS; int ret, fd, lfd; bool __maybe_unused compat_task = !compel_mode_native(ctl); @@ -1303,7 +1303,7 @@ int compel_stop_daemon(struct parasite_ctl *ctl) int compel_cure_remote(struct parasite_ctl *ctl) { - unsigned long ret; + long ret; if (compel_stop_daemon(ctl)) return -1; |