lwIP - A Lightweight TCP/IP stack - Bugs: bug #52471, Change in netconn_recv_data_tcp...
You are not allowed to post comments on this tracker with your current authentication level.
bug #52471: Change in netconn_recv_data_tcp causing recv regression
Submitter: | Joel Cunningham <jcunningham> | ||
Submitted: | Mon 20 Nov 2017 07:45:09 PM UTC | ||
Category: | sockets/netconn | Severity: | 3 - Normal |
Item Group: | Faulty Behaviour | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | 2.1.0 |
lwIP version: | git head |
Jump to the original submission
Mon 05 Feb 2018 08:03:31 PM UTC, comment #10: |
Joel Cunningham <jcunningham> |
Mon 05 Feb 2018 07:59:58 PM UTC, comment #9: Looks good, please push :-) |
Simon Goldschmidt <goldsimon> |
Mon 05 Feb 2018 07:57:15 PM UTC, comment #8: Simon,
|
Joel Cunningham <jcunningham> |
Sun 04 Feb 2018 07:21:13 PM UTC, comment #7: Should be correctly fixed now. |
Simon Goldschmidt <goldsimon> |
Tue 30 Jan 2018 10:23:47 AM UTC, comment #6: The original commit was "socket/netconn recv: FIN should only be reported once (as '0' for sockets, as 'ERR_CLSD' for netconns)" (db3a4e31586f8614b0a27b482406b0c212a4e08a) |
Simon Goldschmidt <goldsimon> |
Tue 21 Nov 2017 07:25:07 PM UTC, comment #5: I'll reopen this to keep it as a reminder this has to be fixed.
|
Simon Goldschmidt <goldsimon> |
Mon 20 Nov 2017 08:36:40 PM UTC, comment #4: Yes. Thanks. |
Simon Goldschmidt <goldsimon> |
Mon 20 Nov 2017 08:22:19 PM UTC, comment #3: I also had to disable the unit tests that checked for an error after receiving remote closure (0) from recv() |
Joel Cunningham <jcunningham> |
Mon 20 Nov 2017 07:56:36 PM UTC, comment #2: Reverted in ebcae98ae65c26a0c210c802540bf027d07fe2f1 |
Joel Cunningham <jcunningham> |
Mon 20 Nov 2017 07:47:14 PM UTC, comment #1: Ooops! Could you please just revert that commit? I don't have access to git right now. I'll rework that patch later then. |
Simon Goldschmidt <goldsimon> |
Mon 20 Nov 2017 07:45:09 PM UTC, original submission:
Found this after pulling up to include db3a4e31586f8614b0a27b482406b0c212a4e08a, but recv no longer returns 0 during a remote closure, instead it returns ENOTCONN.
|
Joel Cunningham <jcunningham> |
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 10 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2018-02-05 | jcunningham | Attached File | - | Added 0001-sockets-remove-ERR_CLSD-case-after-data-receive.patch, #43180 | |
2018-02-04 | goldsimon | Status | None | Fixed | |
Open/Closed | Open | Closed | |||
2018-01-30 | goldsimon | Status | Fixed | None | |
2017-11-21 | goldsimon | Assigned to | jcunningham | goldsimon | |
Open/Closed | Closed | Open | |||
Planned Release | None | 2.1.0 | |||
2017-11-20 | jcunningham | Status | None | Fixed | |
Assigned to | None | jcunningham | |||
Open/Closed | Open | Closed |
Powered by Savane 3.13-758e.
Corresponding source code
Committed in 030ffdee82e915065ac1ceb949d443fd15626fe9