mirror of
https://github.com/checkpoint-restore/criu
synced 2025-09-02 07:15:31 +00:00
img: Don't return fd, return -1 instead
The same -- int-fd will soon go away, so return the explicit int -1 instead of it. Signed-off-by: Pavel Emelyanov <xemul@parallels.com> Acked-by: Cyrill Gorcunov <gorcunov@openvz.org>
This commit is contained in:
2
cgroup.c
2
cgroup.c
@@ -1308,7 +1308,7 @@ int prepare_cgroup(void)
|
|||||||
if (errno == ENOENT) /* backward compatibility */
|
if (errno == ENOENT) /* backward compatibility */
|
||||||
return 0;
|
return 0;
|
||||||
else
|
else
|
||||||
return fd;
|
return -1;
|
||||||
}
|
}
|
||||||
|
|
||||||
ret = pb_read_one_eof(fd, &ce, PB_CGROUP);
|
ret = pb_read_one_eof(fd, &ce, PB_CGROUP);
|
||||||
|
@@ -1984,7 +1984,7 @@ static int prepare_itimers_from_fd(int pid, struct task_restore_args *args)
|
|||||||
|
|
||||||
fd = open_image(CR_FD_ITIMERS, O_RSTR, pid);
|
fd = open_image(CR_FD_ITIMERS, O_RSTR, pid);
|
||||||
if (fd < 0)
|
if (fd < 0)
|
||||||
return fd;
|
return -1;
|
||||||
|
|
||||||
ret = pb_read_one(fd, &ie, PB_ITIMER);
|
ret = pb_read_one(fd, &ie, PB_ITIMER);
|
||||||
if (ret < 0)
|
if (ret < 0)
|
||||||
@@ -2109,7 +2109,7 @@ static int prepare_posix_timers_from_fd(int pid)
|
|||||||
if (errno == ENOENT) /* backward compatibility */
|
if (errno == ENOENT) /* backward compatibility */
|
||||||
return 0;
|
return 0;
|
||||||
else
|
else
|
||||||
return fd;
|
return -1;
|
||||||
}
|
}
|
||||||
|
|
||||||
while (1) {
|
while (1) {
|
||||||
@@ -2178,7 +2178,7 @@ static int prepare_creds(int pid, struct task_restore_args *args)
|
|||||||
|
|
||||||
fd = open_image(CR_FD_CREDS, O_RSTR, pid);
|
fd = open_image(CR_FD_CREDS, O_RSTR, pid);
|
||||||
if (fd < 0)
|
if (fd < 0)
|
||||||
return fd;
|
return -1;
|
||||||
|
|
||||||
ret = pb_read_one(fd, &ce, PB_CREDS);
|
ret = pb_read_one(fd, &ce, PB_CREDS);
|
||||||
close_safe(&fd);
|
close_safe(&fd);
|
||||||
|
Reference in New Issue
Block a user