buglwIP - A Lightweight TCP/IP stack - Bugs: bug #25576, DHCP may set ciaddr to invalid...

 
 

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

bug #25576: DHCP may set ciaddr to invalid address

Submitter:  Jakob Stoklund Olesen <stoklund>
Submitted:  Fri 13 Feb 2009 10:01:38 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 01 May 2009 12:16:02 PM UTC, comment #1: 

Already fixed when integrating patch #6721 (dhcp.c v1.93)

Simon Goldschmidt <goldsimon>
Group administrator
Fri 13 Feb 2009 10:01:38 AM UTC, original submission:  

dhcp_create_request sets ciaddr to netif->if_addr.addr. When using AutoIP, ciaddr may thus contain the AutoIP addr and not 0.0.0.0 as the current code assumes.

According to RFC 2131 ciaddr MUST NOT be set unless in the BOUND, RENEWING, or REBINDING state.

My DHCP server does not reply to requests with a wrong ciaddr field, so once an AutoIP address has been chosen DHCP no longer works.

Patch #6721 fixes this bug.

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 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-05-01 goldsimon StatusNone Fixed
        Assigned toNone goldsimon
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code