Merge pull request #122 from ARMmbed/sync_with_mbedOS

Arto Kinnunen · web-flow · commit 3c79829ee3bb · 2019-05-20T15:13:49.000+03:00
Sync with Mbed OS
diff --git a/test/coap-service/unittest/coap_connection_handler/test_coap_connection_handler.c b/test/coap-service/unittest/coap_connection_handler/test_coap_connection_handler.c
@@ -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);
     nsdynmemlib_stub.returnCounter = 2;
     if (0 != coap_connection_handler_open_connection(handler, 22, false, false, true, false)) {
+        free(sckt_data);
         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);
     nsdynmemlib_stub.returnCounter = 2;
     if (0 != coap_connection_handler_open_connection(handler, 22, false, true, true, false)) {
+        free(sckt_data);
         return false;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -403,6 +403,7 @@ bool test_socket_api_callbacks()`
`403`	`403`	`coap_conn_handler_t *handler = connection_handler_create(&receive_from_sock_cb, &send_to_sock_cb, NULL, NULL);`
`404`	`404`	`nsdynmemlib_stub.returnCounter = 2;`
`405`	`405`	`if (0 != coap_connection_handler_open_connection(handler, 22, false, false, true, false)) {`
	`406`	`+ free(sckt_data);`
`406`	`407`	`return false;`
`407`	`408`	`}`
`408`	`409`
`@@ -483,6 +484,7 @@ bool test_security_callbacks()`
`483`	`484`	`coap_conn_handler_t *handler = connection_handler_create(&receive_from_sock_cb, &send_to_sock_cb, NULL, NULL);`
`484`	`485`	`nsdynmemlib_stub.returnCounter = 2;`
`485`	`486`	`if (0 != coap_connection_handler_open_connection(handler, 22, false, true, true, false)) {`
	`487`	`+ free(sckt_data);`
`486`	`488`	`return false;`
`487`	`489`	`}`
`488`	`490`