diff options
author | José Bollo <jose.bollo@iot.bzh> | 2019-07-10 00:06:36 +0200 |
---|---|---|
committer | Jose Bollo <jose.bollo@iot.bzh> | 2019-07-18 15:38:02 +0200 |
commit | 51ab7c2f95d6d459302423a57cc617021ef6126d (patch) | |
tree | 20588afd25b3332c0f3c15eee09c8c61c6ff7f29 /src/afb-api-ws.c | |
parent | 6dfeafe7e4fa582b3db3f950136bc97f8611fc6d (diff) |
Fix bad memory access at client disconnection
The management of structures handling a client
connection to a exported --ws-server was accessing
freed memory. Tha commit fixes that issue.
Bug-AGL: SPEC-2651
Change-Id: I511218afc907308347bc422a8aead32ca00bdae6
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>
Diffstat (limited to 'src/afb-api-ws.c')
-rw-r--r-- | src/afb-api-ws.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/afb-api-ws.c b/src/afb-api-ws.c index 3d2445ac..6b242e15 100644 --- a/src/afb-api-ws.c +++ b/src/afb-api-ws.c @@ -130,7 +130,9 @@ static void api_ws_server_accept(struct api_ws_server *apiws) close(fd); } else { server = afb_stub_ws_create_server(fdev, &apiws->uri[apiws->offapi], apiws->apiset); - if (!server) + if (server) + afb_stub_ws_set_on_hangup(server, afb_stub_ws_unref); + else ERROR("can't serve accepted connection to %s: %m", apiws->uri); } } @@ -144,7 +146,7 @@ static void api_ws_server_listen_callback(void *closure, uint32_t revents, struc if ((revents & EPOLLIN) != 0) api_ws_server_accept(apiws); - if ((revents & EPOLLHUP) != 0) + else if ((revents & EPOLLHUP) != 0) api_ws_server_connect(apiws); } |