buglwIP - A Lightweight TCP/IP stack - Bugs: bug #25345, DHCPDECLINE is sent with...

 
 

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

bug #25345: DHCPDECLINE is sent with "Maximum message size" option

Submitter:  Jakob Stoklund Olesen <stoklund>
Submitted:  Fri 16 Jan 2009 08:18:07 AM UTC
   
 
Category:  IPv4 Severity:  3 - Normal
Item Group:  Faulty Behaviour Status:  Fixed
Privacy:  Public Assigned to:  goldsimon
Open/Closed:  Closed Planned Release:  None
lwIP version:  CVS Head

Fri 13 Feb 2009 09:44:54 AM UTC, comment #3: 

Yes, please track separate issues in their own bug reports.

Kieran Mansley <kieranm>
Group Member
Fri 13 Feb 2009 08:45:07 AM UTC, comment #2: 

Thanks for committing the fix.

There are two more DHCP bugs that I fixed in patch #6721. They are:

  • Turn AutoIP off before binding the new DHCP address.
  • Don't set ciaddr unless in BOUND, RENEWING, or REBINDING state as per RFC 2131.


Should I open separate bug reports for them?

Jakob Stoklund Olesen <stoklund>
Group Member
Thu 12 Feb 2009 04:18:56 PM UTC, comment #1: 

Thanks for spotting that. For those interested, it is mentioned in the table in chapter 4.4.1 in the RFC.

I'm too lazy to check for other violations of the RFC right now, but if you find any, please add them here!

Simon Goldschmidt <goldsimon>
Group administrator
Fri 16 Jan 2009 08:18:07 AM UTC, original submission:  

RFC-2131 says the DHCP client MUST NOT send the "Maximum message size" option with DHCPRECLINE, but dhcp_recline() adds that option.

I don't think this causes any problems.


Jakob Stoklund Olesen <stoklund>
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 kieranm (Posted a comment)
  • -email is unavailable- added by goldsimon (Posted a comment)
  • -email is unavailable- added by stoklund (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 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2009-02-12 goldsimon StatusNone Fixed
        Assigned toNone goldsimon
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code