aljaz/ttyd
1
1
Fork 0
mirror of https://github.com/tsl0922/ttyd.git synced 2025-12-23 04:14:18 +01:00
Code Issues Packages Projects Releases Wiki Activity

protocol: fix a regression that may crashes the server

Browse Source
This commit is contained in:
Shuanglei Tao
2018-01-10 00:09:30 +08:00
parent f0e4b7b3b3
commit 710aa1cd98
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/protocol.c
View File

@@ -99,7 +99,8 @@ tty_client_remove(struct tty_client *client) {
void void
tty_client_destroy(struct tty_client *client) { tty_client_destroy(struct tty_client *client) {
if (!client->running || client->pid <= 0) if (!client->running || client->pid <= 0)
return; goto cleanup;
client->running = false; client->running = false;
// kill process and free resource // kill process and free resource
@@ -113,6 +114,7 @@ tty_client_destroy(struct tty_client *client) {
lwsl_notice("process exited with code %d, pid: %d\n", status, client->pid); lwsl_notice("process exited with code %d, pid: %d\n", status, client->pid);
close(client->pty); close(client->pty);
cleanup:
// free the buffer // free the buffer
if (client->buffer != NULL) if (client->buffer != NULL)
free(client->buffer); free(client->buffer);