lwIP - A Lightweight TCP/IP stack - Bugs: bug #51531, getsockopt/SO_ERROR not returning...
You are not allowed to post comments on this tracker with your current authentication level.
bug #51531: getsockopt/SO_ERROR not returning value stored in errno
Submitter: | Joel Cunningham <jcunningham> | ||
Submitted: | Fri 21 Jul 2017 09:29:03 PM UTC | ||
Category: | sockets/netconn | Severity: | 3 - Normal |
Item Group: | Faulty Behaviour | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | git head |
Jump to the original submission
Tue 25 Jul 2017 07:54:48 PM UTC, comment #8: |
Simon Goldschmidt <goldsimon> |
Tue 25 Jul 2017 07:20:45 PM UTC, comment #7:
|
Joel Cunningham <jcunningham> |
Tue 25 Jul 2017 06:42:20 PM UTC, comment #6: In my opinion, we had to remove the SO_ERROR handling to meet the standards.
|
Simon Goldschmidt <goldsimon> |
Mon 24 Jul 2017 08:49:27 PM UTC, comment #5:
|
Joel Cunningham <jcunningham> |
Mon 24 Jul 2017 07:16:43 PM UTC, comment #4: I've just added this to UPGRADING. |
Simon Goldschmidt <goldsimon> |
Mon 24 Jul 2017 06:25:45 PM UTC, comment #3:
|
Simon Goldschmidt <goldsimon> |
Mon 24 Jul 2017 03:17:02 PM UTC, comment #2: David,
|
Joel Cunningham <jcunningham> |
Fri 21 Jul 2017 11:20:29 PM UTC, comment #1: What you're describing is expected behavior. Once an error has made it into 'errno', ht will not be available through SO_ERROR anymore. SO_ERROR is effectively only to be used for cases where there was no possibility to move the error code into 'errno'. As such, your application logic will not work on other POSIX-compliant systems either. |
David van Moolenbroek <dcvmoole> |
Fri 21 Jul 2017 09:29:03 PM UTC, original submission:
I've found that getsocktop with SO_ERROR is not returning the value stored in errno.
|
Joel Cunningham <jcunningham> |
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 9 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2017-07-24 | jcunningham | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2017-07-24 | jcunningham | Status | Fixed | None | |
Assigned to | goldsimon | None | |||
Open/Closed | Closed | Open | |||
2017-07-24 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed |
Powered by Savane 3.13-02a9.
Corresponding source code
So let's remove LWIP_SOCKET_SET_ERRNO then. My understanding is that a "proper" OS/C-library combination nowadays should have TLS errno support built in (and all I used did). If it's too lightweight, it's probably not worth using sockets anyway.
-> I've removed it LWIP_SOCKET_SET_ERRNO.