Skip to content

Fixes for Segger emNet to handle non-blocking want read/want write#7581

Merged
anhu merged 1 commit intowolfSSL:masterfrom
dgarske:embos_emnet
May 24, 2024
Merged

Fixes for Segger emNet to handle non-blocking want read/want write#7581
anhu merged 1 commit intowolfSSL:masterfrom
dgarske:embos_emnet

Conversation

@dgarske
Copy link
Copy Markdown
Member

@dgarske dgarske commented May 23, 2024

Description

Fix for EMNET with non-blocking to handle want read/want write. Fix to not include standard POSIX headers like unistd.h for EMNET.

Fixes ZD 18012

Testing

Customer tested

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@dgarske dgarske self-assigned this May 23, 2024
@bandi13
Copy link
Copy Markdown
Contributor

bandi13 commented May 23, 2024

retest this please

@dgarske dgarske assigned wolfSSL-Bot and unassigned dgarske May 23, 2024
@dgarske dgarske changed the title Fixes for Segger emNet to handle non-blocking want read/want write and missing unistd.h Fixes for Segger emNet to handle non-blocking want read/want write May 24, 2024
@dgarske dgarske assigned dgarske and JacobBarthelmeh and unassigned dgarske May 24, 2024
Comment thread src/wolfio.c Outdated
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This actually calls: IP_SOCK_getsockopt and API is int IP_SOCK_getsockopt(int hSock, int Level, int Name, void* pVal, int ValLen). Since the getsockopt is not fully compatible I might change this to use IP_SOCK_getsockopt directly. That API is included via IP/IP.h.

@dgarske dgarske assigned anhu and unassigned JacobBarthelmeh May 24, 2024
@anhu anhu self-requested a review May 24, 2024 21:29
@anhu anhu merged commit 021b573 into wolfSSL:master May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants