buglwIP - A Lightweight TCP/IP stack - Bugs: bug #20357, netconn_close hangs if...

 
 

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

bug #20357: netconn_close hangs if conn->err is ERR_MEM and state is not LISTEN or CLOSE_WAIT

Submitter:  Jared Grubb <jgrubb>
Submitted:  Thu 05 Jul 2007 06:40:25 PM UTC
   
 
Category:  TCP Severity:  3 - Normal
Item Group:  Faulty Behaviour Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed Planned Release:  None
lwIP version:  None

Thu 05 Jul 2007 10:27:51 PM UTC, comment #4: 

(Jared Grubb wrote)

...

Thanks! Great job on this project by the way. I'm enjoying it a lot.

Jared

Frédéric Bernon <fbernon>
Group Member
Thu 05 Jul 2007 07:59:23 PM UTC, comment #3: 

If you can be sure it's ok, so it's good thing (I let you close this item when you will be sure)...

Frédéric Bernon <fbernon>
Group Member
Thu 05 Jul 2007 07:23:56 PM UTC, comment #2: 

I was using a snapshot from late last week... Sorry about that! You guys are fast :) The head looks good.

Jared Grubb <jgrubb>
Group Member
Thu 05 Jul 2007 06:46:38 PM UTC, comment #1: 

Hi Jared,

I think you don't used the last CVS HEAD. Simon Goldschimdth fix that... this week !!!

Check out the last release, and test it. If it's good, we could close it at invalid...

Frédéric Bernon <fbernon>
Group Member
Thu 05 Jul 2007 06:40:25 PM UTC, original submission:  

Example: all netbuf's already allocated, state=ESTABLISHED
------------------
inbuf = netconn_recv(conn); // fails with ERR_MEM
netconn_close(conn); // never returns
-------------------

Reason:
do_close only sets conn->err if the socket is in LISTEN or CLOSE_WAIT states. Therefore, the ERR_MEM persists and netconn_close goes into infinite loop.

Jared Grubb <jgrubb>
Group Member

 

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

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

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

    Date Changed by Updated Field Previous Value => Replaced by
    2007-07-05 fbernon StatusNone Fixed
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code