Skip to content

Commit

Permalink
lxccontainer: non-functional changes
Browse files Browse the repository at this point in the history
Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>
  • Loading branch information
Christian Brauner committed Nov 20, 2017
1 parent 3390336 commit 28c42df
Showing 1 changed file with 14 additions and 12 deletions.
26 changes: 14 additions & 12 deletions src/lxc/lxccontainer.c
Expand Up @@ -771,6 +771,7 @@ static bool do_lxcapi_start(struct lxc_container *c, int useinit, char * const a
NULL,
};
char **init_cmd = NULL;
int keepfds[3] = {-1, -1, -1};

/* container does exist */
if (!c)
Expand Down Expand Up @@ -891,11 +892,11 @@ static bool do_lxcapi_start(struct lxc_container *c, int useinit, char * const a
exit(EXIT_FAILURE);
}

ret = lxc_check_inherited(conf, true,
(int[]){handler->conf->maincmd_fd,
handler->state_socket_pair[0],
handler->state_socket_pair[1]},
3);
keepfds[0] = handler->conf->maincmd_fd;
keepfds[1] = handler->state_socket_pair[0];
keepfds[2] = handler->state_socket_pair[1];
ret = lxc_check_inherited(conf, true, keepfds,
sizeof(keepfds) / sizeof(keepfds[0]));
if (ret < 0)
exit(EXIT_FAILURE);

Expand Down Expand Up @@ -980,11 +981,11 @@ static bool do_lxcapi_start(struct lxc_container *c, int useinit, char * const a
}
}

ret = lxc_check_inherited(conf, daemonize,
(int[]){handler->conf->maincmd_fd,
handler->state_socket_pair[0],
handler->state_socket_pair[1]},
3);
keepfds[0] = handler->conf->maincmd_fd;
keepfds[1] = handler->state_socket_pair[0];
keepfds[2] = handler->state_socket_pair[1];
ret = lxc_check_inherited(conf, daemonize, keepfds,
sizeof(keepfds) / sizeof(keepfds[0]));
if (ret < 0) {
lxc_free_handler(handler);
ret = 1;
Expand Down Expand Up @@ -1745,7 +1746,7 @@ static bool do_lxcapi_shutdown(struct lxc_container *c, int timeout)
if (c->lxc_conf && c->lxc_conf->haltsignal)
haltsignal = c->lxc_conf->haltsignal;

INFO("Using signal number '%d' as halt signal.", haltsignal);
INFO("Using signal number '%d' as halt signal", haltsignal);

/* Add a new state client before sending the shutdown signal so that we
* don't miss a state.
Expand All @@ -1756,13 +1757,14 @@ static bool do_lxcapi_shutdown(struct lxc_container *c, int timeout)

/* Send shutdown signal to container. */
if (kill(pid, haltsignal) < 0)
WARN("Could not send signal %d to pid %d.", haltsignal, pid);
WARN("Could not send signal %d to pid %d", haltsignal, pid);

/* Retrieve the state. */
if (state_client_fd >= 0) {
int state;
state = lxc_cmd_sock_rcv_state(state_client_fd, timeout);
close(state_client_fd);
TRACE("Received state \"%s\"", lxc_state2str(state));
if (state != STOPPED)
return false;
retv = true;
Expand Down

0 comments on commit 28c42df

Please sign in to comment.