michaelh/zephyr
1
0
Fork 0
Code Releases Activity

net: lib: sockets: sockets_tls: prefix mbedtls error with 0x

Browse source 
The errors are printed in hex, but no prefix was used. This could be
confused with usual errno return values. The 0x prefix makes clear
that it's a hex value.

Also a missing minus sign is added to one log message.

Signed-off-by: Martin Jäger <martin@libre.solar>
This commit is contained in:
Martin Jäger 2023-09-14 10:51:32 +02:00 • committed by Fabio Baltieri
commit eae44a55d8
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
subsys/net/lib/sockets/sockets_tls.c
View file

@ -602,7 +602,7 @@ static int tls_session_save(const struct sockaddr *peer_addr,
ret = mbedtls_ssl_session_save(session, entry->session, session_len, ret = mbedtls_ssl_session_save(session, entry->session, session_len,
&session_len); &session_len);
if (ret < 0) { if (ret < 0) {
NET_ERR("Failed to serialize session, err: 0x%x.", -ret); NET_ERR("Failed to serialize session, err: -0x%x.", -ret);
mbedtls_free(entry->session); mbedtls_free(entry->session);
entry->session = NULL; entry->session = NULL;
return -ENOMEM; return -ENOMEM;
@ -1231,7 +1231,7 @@ static int tls_mbedtls_handshake(struct tls_context *context,
/* MbedTLS API documentation requires session to /* MbedTLS API documentation requires session to
* be reset in other error cases * be reset in other error cases
*/ */
NET_ERR("TLS handshake error: -%x", -ret); NET_ERR("TLS handshake error: -0x%x", -ret);
ret = tls_mbedtls_reset(context); ret = tls_mbedtls_reset(context);
if (ret == 0) { if (ret == 0) {
ret = -ECONNABORTED; ret = -ECONNABORTED;
@ -1240,7 +1240,7 @@ static int tls_mbedtls_handshake(struct tls_context *context,
} }
/* Avoid constant loop if tls_mbedtls_reset fails */ /* Avoid constant loop if tls_mbedtls_reset fails */
NET_ERR("TLS reset error: -%x", -ret); NET_ERR("TLS reset error: -0x%x", -ret);
ret = -ECONNABORTED; ret = -ECONNABORTED;
break; break;
} }