buglwIP - A Lightweight TCP/IP stack - Bugs: bug #33604, Callback registering functions...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #33604: Callback registering functions don't check for a legal socket state

Submitter:  Cameron Gutman <aicom>
Submitted:  Mon 20 Jun 2011 02:27:25 PM UTC
   
 
Category:  TCP Severity:  3 - Normal
Item Group:  Change Request Status:  Fixed
Privacy:  Public Assigned to:  goldsimon
Open/Closed:  Closed Planned Release:  None
lwIP version:  CVS Head

Sun 26 Jun 2011 05:10:16 PM UTC, comment #2: 

Done. Thanks for the hint.

Simon Goldschmidt <goldsimon>
Group administrator
Sat 25 Jun 2011 09:44:48 AM UTC, comment #1: 

That's a good point. I'll add the asserts.

Simon Goldschmidt <goldsimon>
Group administrator
Mon 20 Jun 2011 02:27:25 PM UTC, original submission:  

This issue was one that I ran into when integrating lwIP into the network stack for the ReactOS project. I had accidentally called tcp_err for a listening PCB and it caused all sorts of trouble by corrupting random memory. It took me forever to figure out what was causing the corruption that was randomly leading to BSODs. I added these assertions so hopefully nobody will end up in the same boat again.

Cameron Gutman <aicom>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #23539:  assertpatch.diff added by aicom (1KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by goldsimon (Posted a comment)
  • -email is unavailable- added by aicom (Submitted the item)
  • -email is unavailable- added by aicom
  •  

    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 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-06-26 goldsimon StatusNone Fixed
        Assigned toNone goldsimon
        Open/ClosedOpen Closed
    2011-06-20 aicom Attached File- Added assertpatch.diff, #23539
        Carbon-Copy- Added -email is unavailable-

    Back to the top

    Powered by Savane 3.13-bb6a.
    Corresponding source code