Add UDP support to NetX sockets for DTLS sessions#10273
Closed
hmohide wants to merge 2 commits intowolfSSL:masterfrom
Closed
Add UDP support to NetX sockets for DTLS sessions#10273hmohide wants to merge 2 commits intowolfSSL:masterfrom
hmohide wants to merge 2 commits intowolfSSL:masterfrom
Conversation
|
Can one of the admins verify this patch? |
Contributor
|
OK to test. Contributor agreement on file. |
|
Contributor
|
@hmohide ThreadX/NetX build failure: |
Author
UINT16 is platform specific definition. Can change to USHORT which is NetX Standard type |
Author
UINT16 is platform specific definition. Can change to USHORT which is NetX Standard type |
dgarske
requested changes
Apr 27, 2026
Member
dgarske
left a comment
There was a problem hiding this comment.
[check-source-text] [1 of 1] [f9b24de017]
autogen.sh f9b24de017... real 0m7.640s user 0m6.265s sys 0m0.425s
configure... real 0m8.328s user 0m5.160s sys 0m3.954s
trailing whitespace:
./src/wolfio.c:2655: if (nxCtx == NULL || nxCtx->nxUdpSocket == NULL·
./src/wolfio.c:2683: }·
./src/wolfio.c:2691: }·
./src/wolfio.c:2709: ssl->nxCtx.nxUdpSocket = nxsocket;··
./wolfssl/wolfio.h:794: WOLFSSL_API void wolfSSL_SetIO_NetX_Dtls(WOLFSSL* ssl, NX_UDP_SOCKET* nxsocket,·
./wolfssl/wolfio.h:795: NXD_ADDRESS *nxdip,·
./wolfssl/wolfio.h:796: USHORT* nxport,·
[skipping "codespell" subtest in check-source-text -- codespell is missing or too old]
overlong lines added:
linuxkm/lkcapi_sha_glue.c:1514 ((struct DRBG_internal *)WC_RNG_BANK_INST_TO_RNG(drbg)->drbg)->V[V_offset++] += ((byte *)buf)[i];
linuxkm/lkcapi_sha_glue.c:1515 if (V_offset == (int)sizeof ((struct DRBG_internal *)WC_RNG_BANK_INST_TO_RNG(drbg)->drbg)->V)
src/wolfio.c:2678 status = nx_udp_socket_send(nxCtx->nxUdpSocket, packet, nxCtx->nxdIp->nxd_ip_address.v4, (UINT)(*nxCtx->nxPort));
src/wolfio.c:2686 status = nxd_udp_socket_send(nxCtx->nxUdpSocket, packet, nxCtx->nxdIp, (UINT)(*nxCtx->nxPort));
src/wolfio.c:2706 void wolfSSL_SetIO_NetX_Dtls(WOLFSSL* ssl, NX_UDP_SOCKET* nxsocket, NXD_ADDRESS *nxdip, USHORT *nxport, ULONG waitoption)
wolfcrypt/src/random.c:443 static int Hash_DRBG_Reseed(DRBG_internal* drbg, const byte* seed, word32 seedSz)
wolfcrypt/src/random.c:757 static int Hash_DRBG_Instantiate(DRBG_internal* drbg, const byte* seed, word32 seedSz,
wolfcrypt/src/random.c:1591 if (Hash_DRBG_Uninstantiate((DRBG_internal *)rng->drbg_scratch) != DRBG_SUCCESS)
wolfcrypt/src/rng_bank.c:485 (((struct DRBG_internal *)(*rng_inst)->rng.drbg)->reseedCtr >=
wolfssl/wolfio.h:789 WOLFSSL_LOCAL int NetX_ReceiveFrom(WOLFSSL *ssl, char *buf, int sz, void *ctx);
[...and more...]
Member
|
Hi @hmohide you opened this on your local master instead of a different branch, which causes some issues. Would you mind closing this PR and opening a new one against latest upstream master with a better local fork branch name? Thanks |
Author
|
@dgarske i am unable to create the fork. Can you guide me. |
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Add UDP support to NetX sockets for DTLS sessions
Fixes zd#
Testing
Yes, verified the DTLS communication with multiple session
Checklist