lwIP - A Lightweight TCP/IP stack - Bugs: bug #51990, Calling select() from different...
You are not allowed to post comments on this tracker with your current authentication level.
bug #51990: Calling select() from different threads with MPU enabled triggers memory protection fault
Submitter: | David Lockyer <david_lockyer> | ||
Submitted: | Mon 11 Sep 2017 09:58:09 AM UTC | ||
Category: | sockets/netconn | Severity: | 3 - Normal |
Item Group: | Crash Error | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | 2.0.2 |
Jump to the original submission
Thu 14 Sep 2017 03:45:13 PM UTC, comment #7: |
Simon Goldschmidt <goldsimon> |
Wed 13 Sep 2017 09:53:41 PM UTC, comment #6: There was a double free on select_cb introduced in the refactoring. Fixed in 5c0054d8ee90ab89ec830287f00fbcdd27cb475a |
Joel Cunningham <jcunningham> |
Tue 12 Sep 2017 07:39:20 PM UTC, comment #5: Pushed with some minor changes. Thanks for the patch, David! Hope it still works for you with my changes ;-) |
Simon Goldschmidt <goldsimon> |
Tue 12 Sep 2017 06:35:26 PM UTC, comment #4: API_SELECT_CB_VAR_ALLOC() lets lwip_select() return ERR_MEM, I'll fix that and push it. |
Simon Goldschmidt <goldsimon> |
Tue 12 Sep 2017 02:15:43 PM UTC, comment #3: mpu_lwip_select_master.patch isn't building on MSVC:
|
Joel Cunningham <jcunningham> |
Mon 11 Sep 2017 07:29:45 PM UTC, comment #2:
|
Simon Goldschmidt <goldsimon> |
Mon 11 Sep 2017 04:26:12 PM UTC, comment #1: David,
|
Joel Cunningham <jcunningham> |
Mon 11 Sep 2017 09:58:09 AM UTC, original submission:
Calling select from different threads with MPU enabled triggers an memory access violation exception in lwip_select(), despite LWIP_MPU_COMPATIBLE being set to 1. This is due to structure select_cb being on the stack, as opposed to in a memory pool.
|
David Lockyer <david_lockyer> |
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 7 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2017-09-12 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2017-09-12 | david_lockyer | Attached File | - | Added mpu_lwip_select_master_2nd_attempt.patch, #41788 | |
2017-09-12 | david_lockyer | Attached File | - | Added mpu_lwip_select_STABLE-2_0_0_2nd_attempt.patch, #41782 | |
Attached File | - | Added mpu_lwip_select_master.patch, #41783 | |||
2017-09-11 | david_lockyer | Attached File | - | Added mpu_lwip_select.patch, #41779 |
Powered by Savane 3.13-02a9.
Corresponding source code
Right, thanks for watching ;-)