ARMmbed
/
mbed-os
mirror of https://github.com/ARMmbed/mbed-os.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

test_soap_connection_handler leaks memory when open_connection fails 

sckt_data is not free'd when `coap_connection_handler_open_connection` fails.
pull/9981/head
Jan Jongboom 2019-03-07 16:27:59 +01:00 committed by GitHub
parent a87c7c82f3
commit 3e4010748c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
features/nanostack/coap-service/test/coap-service/unittest/coap_connection_handler/test_coap_connection_handler.c
View File

@ -403,6 +403,7 @@ bool test_socket_api_callbacks()
coap_conn_handler_t *handler = connection_handler_create(&receive_from_sock_cb, &send_to_sock_cb, NULL, NULL); coap_conn_handler_t *handler = connection_handler_create(&receive_from_sock_cb, &send_to_sock_cb, NULL, NULL);
nsdynmemlib_stub.returnCounter = 2; nsdynmemlib_stub.returnCounter = 2;
if (0 != coap_connection_handler_open_connection(handler, 22, false, false, true, false)) { if (0 != coap_connection_handler_open_connection(handler, 22, false, false, true, false)) {
free(sckt_data);
return false; return false;
} }
@ -483,6 +484,7 @@ bool test_security_callbacks()
coap_conn_handler_t *handler = connection_handler_create(&receive_from_sock_cb, &send_to_sock_cb, NULL, NULL); coap_conn_handler_t *handler = connection_handler_create(&receive_from_sock_cb, &send_to_sock_cb, NULL, NULL);
nsdynmemlib_stub.returnCounter = 2; nsdynmemlib_stub.returnCounter = 2;
if (0 != coap_connection_handler_open_connection(handler, 22, false, true, true, false)) { if (0 != coap_connection_handler_open_connection(handler, 22, false, true, true, false)) {
free(sckt_data);
return false; return false;
} }