Skip to content

Commit 913103c

Browse files
committed
TLSSocketWrapper: decouple error requirements
* Don't pull in mbedtls_strerror() if trace is not enabled. * Output error trace even if mbedtls_strerror() is not available.
1 parent 0404701 commit 913103c

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

features/netsocket/TLSSocketWrapper.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -291,16 +291,16 @@ nsapi_size_or_error_t TLSSocketWrapper::recvfrom(SocketAddress *address, void *d
291291
return recv(data, size);
292292
}
293293

294-
void TLSSocketWrapper::print_mbedtls_error(const char *name, int err)
294+
void TLSSocketWrapper::print_mbedtls_error(MBED_UNUSED const char *name, MBED_UNUSED int err)
295295
{
296-
#ifdef MBEDTLS_ERROR_C
296+
// Avoid pulling in mbedtls_strerror when trace is not enabled
297+
#if defined FEA_TRACE_SUPPORT && defined MBEDTLS_ERROR_C
297298
char *buf = new char[128];
298299
mbedtls_strerror(err, buf, 128);
299300
tr_err("%s() failed: -0x%04x (%d): %s", name, -err, err, buf);
300301
delete[] buf;
301302
#else
302-
(void)name;
303-
(void)err;
303+
tr_err("%s() failed: -0x%04x (%d)", name, -err, err);
304304
#endif
305305
}
306306

0 commit comments

Comments
 (0)