riot-ci/RIOT
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #16963 from miri64/gcoap_dtls/fix/session-destroy-only-on-conn-errors

Browse Source 
gcoap_dtls: destroy session in _tl_send only on connection errors
This commit is contained in:
benpicco 2021-10-12 13:35:16 +02:00 committed by GitHub
commit 94c852cfa7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
sys/net/application_layer/gcoap/gcoap.c
View File

@ -910,9 +910,18 @@ static ssize_t _tl_send(gcoap_socket_t *sock, const void *data, size_t len,
/* send application data */ /* send application data */
res = sock_dtls_send(sock->socket.dtls, &sock->ctx_dtls_session, data, len, res = sock_dtls_send(sock->socket.dtls, &sock->ctx_dtls_session, data, len,
SOCK_NO_TIMEOUT); SOCK_NO_TIMEOUT);
if (res <= 0 ) { switch (res) {
case -EHOSTUNREACH:
case -ENOTCONN:
case 0:
DEBUG("gcoap: DTLS sock not connected or remote unreachable. "
"Destroying session.\n");
dsm_remove(sock->socket.dtls, &sock->ctx_dtls_session); dsm_remove(sock->socket.dtls, &sock->ctx_dtls_session);
sock_dtls_session_destroy(sock->socket.dtls, &sock->ctx_dtls_session); sock_dtls_session_destroy(sock->socket.dtls, &sock->ctx_dtls_session);
break;
default:
/* Temporary error. Keeping the DTLS session */
break;
} }
#endif #endif
} else if (sock->type == GCOAP_SOCKET_TYPE_UDP) { } else if (sock->type == GCOAP_SOCKET_TYPE_UDP) {