diff options
| author | Furkan Sahin <furkan-dev@proton.me> | 2016-12-15 17:05:12 -0500 |
|---|---|---|
| committer | Furkan Sahin <furkan-dev@proton.me> | 2016-12-15 17:05:12 -0500 |
| commit | 97619e37d6863fed25c789fd61d3f628f8afd691 (patch) | |
| tree | 2e80e1258f5e1af0465a7dfd9959397f7009e6df /common/ipc-client.c | |
| parent | 7b2a94efb4478167c33927024497fd3a3a824266 (diff) | |
Handle malloc failure in ipc_recv_response
Diffstat (limited to 'common/ipc-client.c')
| -rw-r--r-- | common/ipc-client.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/common/ipc-client.c b/common/ipc-client.c index 106f9d86..d011bd26 100644 --- a/common/ipc-client.c +++ b/common/ipc-client.c @@ -52,10 +52,18 @@ struct ipc_response *ipc_recv_response(int socketfd) { } struct ipc_response *response = malloc(sizeof(struct ipc_response)); + if (!response) { + goto error_1; + } + total = 0; response->size = data32[0]; response->type = data32[1]; char *payload = malloc(response->size + 1); + if (!payload) { + goto error_2; + } + while (total < response->size) { ssize_t received = recv(socketfd, payload + total, response->size - total, 0); if (received < 0) { @@ -67,6 +75,11 @@ struct ipc_response *ipc_recv_response(int socketfd) { response->payload = payload; return response; +error_2: + free(response); +error_1: + sway_log(L_ERROR, "Unable to allocate memory for IPC response"); + return NULL; } void free_ipc_response(struct ipc_response *response) { |
