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

Submitted by:  Cameron Gutman <aicom>
Submitted on:  Mon 20 Jun 2011 02:27:25 PM UTC  
 
Category: TCPSeverity: 3 - Normal
Item Group: Change RequestStatus: Fixed
Privacy: PublicAssigned to: Simon Goldschmidt <goldsimon>
Open/Closed: ClosedPlanned 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>
Project AdministratorIn charge of this item.
Sat 25 Jun 2011 09:44:48 AM UTC, comment #1:

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

Simon Goldschmidt <goldsimon>
Project AdministratorIn charge of this item.
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>

 

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
  • -unavailable- added by goldsimon (Posted a comment)
  • -unavailable- added by aicom (Submitted the item)
  • -unavailable- added by aicom
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 26 Jun 2011 05:10:16 PM UTCgoldsimonStatusNone=>Fixed
      Assigned toNone=>goldsimon
      Open/ClosedOpen=>Closed
    Mon 20 Jun 2011 02:27:25 PM UTCaicomAttached File-=>Added assertpatch.diff, #23539
      Carbon-Copy-=>Added -unavailable-

    Back to the top


    Powered by Savane 3.1-cleanup1