buglwIP - A Lightweight TCP/IP stack - Bugs: bug #50088, socket/netconn: data before RST...

 
 

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

bug #50088: socket/netconn: data before RST should be readable

Submitted by:  Simon Goldschmidt <goldsimon>
Submitted on:  Thu 19 Jan 2017 10:02:14 AM UTC  
 
Category: sockets/netconnSeverity: 3 - Normal
Item Group: Feature RequestStatus: Fixed
Privacy: PublicAssigned to: Simon Goldschmidt <goldsimon>
Open/Closed: ClosedPlanned Release: 2.1.0
lwIP version: git head

Fri 04 Aug 2017 04:06:48 PM UTC, comment #5:

Hi Simon,

Thanks a million. I've just tested this out and it works fine.

Regards,
Tim

Tim Cussins <tcussins>
Thu 03 Aug 2017 08:29:32 PM UTC, comment #4:

This should be be finally fixed now.

Simon Goldschmidt <goldsimon>
Project AdministratorIn charge of this item.
Tue 30 May 2017 09:42:25 AM UTC, comment #3:

> Is this what you'd expect?


No. In my tests, this worked, but as there are no continuous tests on this, maybe it's broken.

Also, I haven't tested 2 RST...

I'll reopen this to check/fix it.

Simon Goldschmidt <goldsimon>
Project AdministratorIn charge of this item.
Tue 30 May 2017 09:33:53 AM UTC, comment #2:

Hi Simon,

I got a git head version of LWiP working, and have just tried this out.

With a bunch of LWiP debugging enabled, I can see the server's data arriving, then 2 further packets with RST set.

When I call lwip_recv(), I get ECONNRESET, rather than any buffered bytes.

Is this what you'd expect? Should I be enabling a particular complile-time option to get different behaviour, or using some other technique?

In short: What socket-layer calls should I make in order to read the data that arrived before RST.

Thanks,
Tim

Tim Cussins <tcussins>
Wed 12 Apr 2017 10:44:27 AM UTC, comment #1:

This should now work.
http://git.savannah.nongnu.org/cgit/lwip.git/commit/?id=e0a2472706ee7af99cd00d35bc7e5829aea5caea

Simon Goldschmidt <goldsimon>
Project AdministratorIn charge of this item.
Thu 19 Jan 2017 10:02:14 AM UTC, original submission:
Simon Goldschmidt <goldsimon>
Project AdministratorIn charge of this item.

 

Attached Files
file #40815:  minicom.cap added by tcussins (5KiB - application/vnd.tcpdump.pcap - lwip debug trace of double RST and subsequent recv failure)

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

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

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 03 Aug 2017 08:29:32 PM UTCgoldsimonStatusIn Progress=>Fixed
      Open/ClosedOpen=>Closed
    Tue 30 May 2017 10:12:34 AM UTCtcussinsAttached File-=>Added minicom.cap, #40815
    Tue 30 May 2017 09:42:25 AM UTCgoldsimonStatusFixed=>In Progress
      Open/ClosedClosed=>Open
    Wed 12 Apr 2017 10:44:27 AM UTCgoldsimonStatusNone=>Fixed
      Assigned toNone=>goldsimon
      Open/ClosedOpen=>Closed
    Wed 01 Mar 2017 11:41:15 AM UTCgoldsimonPlanned Release2.0.2=>2.1.0

    Back to the top


    Powered by Savane 3.1-cleanup1