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

Submitted by:  None
Submitted on:  Wed 15 Dec 2004 10:55:25 AM UTC  
 
Category:  None Priority:  7 - High
Status:  Ready For Test Privacy:  Public
Assigned to:  Jonathan Larmour <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>
Project MemberIn charge of this item.
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>
Project 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>
Project MemberIn charge of this item.
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>
Project 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

 

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)
  •  

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

    Only logged-in users can vote.

     

     

     

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