Skip to content

Commit aab3ade

Browse files
author
Cruz Monrreal
authored
Merge pull request #8687 from kjbracey-arm/tls_errorlog
TLSSocketWrapper: decouple error requirements
2 parents 0404701 + 913103c commit aab3ade

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)