maride/guacamole-spice-protocol
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Properly close pipe file descriptors. Wait for input thread to close in SSH thread.

Browse Source
This commit is contained in:
Michael Jumper 2013-05-20 10:52:47 -07:00
parent 639389ced8
commit c220a4875c
3 changed files with 11 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
protocols/ssh/src/client.c
View File

@ -108,13 +108,6 @@ int guac_client_init(guac_client* client, int argc, char** argv) {
return 1;
}
/* Redirect STDOUT to pipe */
if (dup2(client_data->stdout_pipe_fd[1], STDOUT_FILENO) < 0) {
guac_error = GUAC_STATUS_SEE_ERRNO;
guac_error_message = "Unable redirect STDOUT";
return 1;
}
/* Open STDIN pipe */
if (pipe(client_data->stdin_pipe_fd)) {
guac_error = GUAC_STATUS_SEE_ERRNO;
@ -122,13 +115,6 @@ int guac_client_init(guac_client* client, int argc, char** argv) {
return 1;
}
/* Redirect STDIN to pipe */
if (dup2(client_data->stdin_pipe_fd[0], STDIN_FILENO) < 0) {
guac_error = GUAC_STATUS_SEE_ERRNO;
guac_error_message = "Unable redirect STDIN";
return 1;
}
/* Set basic handlers */
client->handle_messages = ssh_guac_client_handle_messages;
client->clipboard_handler = ssh_guac_client_clipboard_handler;

11
protocols/ssh/src/guac_handlers.c
View File

@ -354,9 +354,14 @@ int ssh_guac_client_free_handler(guac_client* client) {
ssh_guac_client_data* guac_client_data = (ssh_guac_client_data*) client->data;
/* Close SSH client */
close(STDOUT_FILENO);
close(STDIN_FILENO);
/* Close terminal output pipe */
close(guac_client_data->stdout_pipe_fd[1]);
close(guac_client_data->stdout_pipe_fd[0]);
/* Close user input pipe */
close(guac_client_data->stdin_pipe_fd[1]);
close(guac_client_data->stdin_pipe_fd[0]);
pthread_join(guac_client_data->client_thread, NULL);
/* Free terminal */

3
protocols/ssh/src/ssh_client.c
View File

@ -262,6 +262,9 @@ void* ssh_client_thread(void* data) {
return NULL;
}
/* Wait for input thread to die */
pthread_join(input_thread, NULL);
guac_client_log_info(client, "SSH connection ended.");
return NULL;