lxccontainer: non-functional changes

parent 33903369
...@@ -771,6 +771,7 @@ static bool do_lxcapi_start(struct lxc_container *c, int useinit, char * const a ...@@ -771,6 +771,7 @@ static bool do_lxcapi_start(struct lxc_container *c, int useinit, char * const a
NULL, NULL,
}; };
char **init_cmd = NULL; char **init_cmd = NULL;
int keepfds[3] = {-1, -1, -1};
/* container does exist */ /* container does exist */
if (!c) if (!c)
...@@ -891,11 +892,11 @@ static bool do_lxcapi_start(struct lxc_container *c, int useinit, char * const a ...@@ -891,11 +892,11 @@ static bool do_lxcapi_start(struct lxc_container *c, int useinit, char * const a
exit(EXIT_FAILURE); exit(EXIT_FAILURE);
} }
ret = lxc_check_inherited(conf, true, keepfds[0] = handler->conf->maincmd_fd;
(int[]){handler->conf->maincmd_fd, keepfds[1] = handler->state_socket_pair[0];
handler->state_socket_pair[0], keepfds[2] = handler->state_socket_pair[1];
handler->state_socket_pair[1]}, ret = lxc_check_inherited(conf, true, keepfds,
3); sizeof(keepfds) / sizeof(keepfds[0]));
if (ret < 0) if (ret < 0)
exit(EXIT_FAILURE); exit(EXIT_FAILURE);
...@@ -980,11 +981,11 @@ reboot: ...@@ -980,11 +981,11 @@ reboot:
} }
} }
ret = lxc_check_inherited(conf, daemonize, keepfds[0] = handler->conf->maincmd_fd;
(int[]){handler->conf->maincmd_fd, keepfds[1] = handler->state_socket_pair[0];
handler->state_socket_pair[0], keepfds[2] = handler->state_socket_pair[1];
handler->state_socket_pair[1]}, ret = lxc_check_inherited(conf, daemonize, keepfds,
3); sizeof(keepfds) / sizeof(keepfds[0]));
if (ret < 0) { if (ret < 0) {
lxc_free_handler(handler); lxc_free_handler(handler);
ret = 1; ret = 1;
...@@ -1745,7 +1746,7 @@ static bool do_lxcapi_shutdown(struct lxc_container *c, int timeout) ...@@ -1745,7 +1746,7 @@ static bool do_lxcapi_shutdown(struct lxc_container *c, int timeout)
if (c->lxc_conf && c->lxc_conf->haltsignal) if (c->lxc_conf && c->lxc_conf->haltsignal)
haltsignal = 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 /* Add a new state client before sending the shutdown signal so that we
* don't miss a state. * don't miss a state.
...@@ -1756,13 +1757,14 @@ static bool do_lxcapi_shutdown(struct lxc_container *c, int timeout) ...@@ -1756,13 +1757,14 @@ static bool do_lxcapi_shutdown(struct lxc_container *c, int timeout)
/* Send shutdown signal to container. */ /* Send shutdown signal to container. */
if (kill(pid, haltsignal) < 0) 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. */ /* Retrieve the state. */
if (state_client_fd >= 0) { if (state_client_fd >= 0) {
int state; int state;
state = lxc_cmd_sock_rcv_state(state_client_fd, timeout); state = lxc_cmd_sock_rcv_state(state_client_fd, timeout);
close(state_client_fd); close(state_client_fd);
TRACE("Received state \"%s\"", lxc_state2str(state));
if (state != STOPPED) if (state != STOPPED)
return false; return false;
retv = true; retv = true;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment