buglwIP - A Lightweight TCP/IP stack - Bugs: bug #38714, Missing option and client address...

 
 

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

bug #38714: Missing option and client address in DHCPRELEASE message

Submitter:  Florent Matignon <fmatignon>
Submitted:  Fri 12 Apr 2013 09:51:39 AM UTC
   
 
Category:  DHCP Severity:  3 - Normal
Item Group:  Faulty Behaviour Status:  Fixed
Privacy:  Public Assigned to:  goldsimon
Open/Closed:  Closed Planned Release:  None
lwIP version:  git head

Fri 20 Mar 2015 08:04:12 PM UTC, comment #2: 

Fixed, thanks for reporting.

Additionally, the RELEASE message was sent to 0.0.0.0 because the server address was cleared before sending...

Simon Goldschmidt <goldsimon>
Group administrator
Wed 24 Apr 2013 07:25:24 PM UTC, comment #1: 

I wonder why this didn't seem to be a problem, yet. Maybe because "RELEASE" isn't used often? Anyway, here are two lines to support the requested change (from RFC2131):

"If the client used a 'client identifier' when it obtained the lease, it MUST use the same 'client identifier' in the DHCPRELEASE message."

"The client identifies the lease to be released with its 'client identifier', or 'chaddr' and network address in the DHCPRELEASE message."

Simon Goldschmidt <goldsimon>
Group administrator
Fri 12 Apr 2013 09:51:39 AM UTC, original submission:  

According to RFC 2131 "4.4 DHCP client behavior", DHCPRELEASE messages should set ciaddr field to the client's network address and have the Server identifier option field set to the server address (Table 5: Fields and options used by DHCP clients).

lwip actually set the ciaddr field to 0 and the Server identifier option field is missing.

Florent Matignon <fmatignon>

 

(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 goldsimon (Posted a comment)
  • -email is unavailable- added by fmatignon (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
    2015-03-20 goldsimon StatusNone Fixed
        Assigned toNone goldsimon
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code