patchlwIP - A Lightweight TCP/IP stack - Patches: patch #3574, lwip_ioctl FIONREAD patch

 
 

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

patch #3574: lwip_ioctl FIONREAD patch

Submitter:  None
Submitted:  Wed 15 Dec 2004 10:55:25 AM UTC
   
 
Category:  None Priority:  7 - High
Status:  Ready For Test Privacy:  Public
Assigned to:  jifl Originator Email:  -email is unavailable-
Open/Closed:  Closed Planned Release:  None

Thu 03 May 2007 07:13:36 PM UTC, comment #4: 

Well, Unixes are what brought us almost all the APIs anyway :). It's very common, but I don't know why it isn't in the Single Unix Spec.

I've checked the patch in.

Jonathan Larmour <jifl>
Group Member
Thu 03 May 2007 05:19:30 PM UTC, comment #3: 

Is FIONREAD a non-standard option? I can't find documentation (except from unixes) about it.

Anyway, as I also think the patch does the right thing, I think we should finally include it (after so many years...)!

Simon Goldschmidt <goldsimon>
Group administrator
Mon 19 Feb 2007 11:40:22 AM UTC, comment #2: 

The change looks right to me. FIONREAD should return the whole amount of data remaining in the stream. Being a stream, it's irrespective of how much was read in the last recv.

Jonathan Larmour <jifl>
Group Member
Mon 19 Feb 2007 11:22:09 AM UTC, comment #1: 

Can anyone comment about the correct operation of FIONREAD.  This is a pretty simple change, but I don't want to make things worse, and I don't have any particular knowledge about this particular bit of the sockets API

Kieran Mansley <kieranm>
Group Member
Wed 15 Dec 2004 10:55:25 AM UTC, original submission:  

After playing around with lwip on a embedded device for a while i suspects lwip_ioctl with FIONREAD command misbehave a little. My only reference is winsock2, but please do not flame me if its winsock2 misbehavior.

Before this patch ioctl-FIONREAD assumes that last call to recv read everything.

Applied patch makes even the data left behind in last recv call count.

/maq (excuse my non native english)

Anonymous

 

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

Attached Files
file #8180:  ioctl_fionread.patch added by None (931B - application/octet-stream - ioctl FIONREAD patch)

 

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 jifl (Posted a comment)
  • -email is unavailable- added by kieranm (Posted a comment)
  •  

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

    Date Changed by Updated Field Previous Value => Replaced by
    2007-05-03 jifl StatusNone Ready For Test
        Assigned toNone jifl
        Open/ClosedOpen Closed
    2004-12-15 None Attached File- Added ioctl_fionread.patch, #3947

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code