lwIP - A Lightweight TCP/IP stack - Bugs: bug #35928, BSD sockets functions must set...
You are not allowed to post comments on this tracker with your current authentication level.
bug #35928: BSD sockets functions must set errno for POSIX-compliance
Submitter: | Mason <mason> | ||
Submitted: | Wed 21 Mar 2012 09:39:03 AM 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 11 Mar 2014 07:54:32 PM UTC, comment #13: |
Simon Goldschmidt <goldsimon> |
Mon 09 Apr 2012 03:34:11 PM UTC, comment #12: Sorry for not getting round to this sooner. It changes the port API and so shouldn't be in 1.4.1.
|
Kieran Mansley <kieranm> |
Fri 06 Apr 2012 08:19:32 AM UTC, comment #11: Can this bug be marked as planned for a specific release? (Either 1.5.0 or 1.4.1) That way, it will appear when searching for open bugs in a given release. Thanks. |
Mason <mason> |
Wed 28 Mar 2012 09:46:06 AM UTC, comment #10: Simon Goldschmidt wrote:
|
Mason <mason> |
Wed 28 Mar 2012 08:49:13 AM UTC, comment #9:
|
Simon Goldschmidt <goldsimon> |
Wed 28 Mar 2012 08:25:19 AM UTC, comment #8: Simon Goldschmidt wrote:
|
Mason <mason> |
Tue 27 Mar 2012 05:48:30 PM UTC, comment #7: I do understand it's not cool lwIP provides 'errno' but doesn't set it. However, since this is a potentially port-breaking change, one might argue whether 1.4.1 or 1.5.0 is the correct version for this change. I'll leave it up to our project leader to decide... (although I'd probably vote for 1.4.1 :) |
Simon Goldschmidt <goldsimon> |
Mon 26 Mar 2012 11:36:56 AM UTC, comment #6: Simon, Kieran,
|
Mason <mason> |
Wed 21 Mar 2012 03:04:34 PM UTC, comment #5: Trying to upload the correct patch. |
Mason <mason> |
Wed 21 Mar 2012 02:52:44 PM UTC, comment #4: Simon Goldschmidt wrote:
|
Mason <mason> |
Wed 21 Mar 2012 01:21:33 PM UTC, comment #3: I can understand you're upset about wasting several hours. Still, I don't think defining LWIP_SOCKET is the correct guard here: we don't define other functions to their standard name by default (e.g. 'bind()', 'listen()' etc.) but prefix them with 'lwip_' and only use standard names when LWIP_COMPAT_SOCKETS==1.
|
Simon Goldschmidt <goldsimon> |
Wed 21 Mar 2012 12:37:06 PM UTC, comment #2: For the record, the only port that defines ERRNO is coldfire:
|
Mason <mason> |
Wed 21 Mar 2012 11:37:27 AM UTC, comment #1: I'm cautious about making changes that would require all the existing ports to be changed, which this would need. We don't understand why this code was introduced, and I'd like to do that before removing it. For now, please define ERRNO in your port and you will get the behaviour you need. |
Kieran Mansley <kieranm> |
Wed 21 Mar 2012 09:39:03 AM UTC, original submission:
I am porting libcurl on top of lwip on a (mostly) POSIX-compliant embedded
|
Mason <mason> |
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 8 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2014-03-11 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2012-04-09 | kieranm | Planned Release | 1.5.0 | ||
2012-03-21 | mason | Attached File | - | Added errno_patch.txt, #25430 | |
2012-03-21 | mason | Attached File | - | Added errno_patch.txt, #25429 | |
2012-03-21 | mason | Attached File | - | Added errno_patch.txt, #25428 | |
2012-03-21 | mason | Attached File | - | Added patch_sockets.txt, #25426 |
Powered by Savane 3.13-02a9.
Corresponding source code
Fixed (more or less like proposed in the last patch file, but default is to set errno).