Bug 59369

Summary: Faked bootp record has misleading ESA
Product: [Retired] eCos Reporter: Jonathan Larmour <jlarmour>
Component: TCPIPAssignee: eCos bugs internal list <es-ecos-bugs-int>
Status: CLOSED CURRENTRELEASE QA Contact: eCos bugs internal list <es-ecos-bugs-int>
Severity: low Docs Contact:
Priority: medium    
Version: CVS   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2002-02-06 16:11:19 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Jonathan Larmour 2002-02-06 16:11:14 UTC 
Description of Problem:

In network_support.c: build_bootp_record() (used when using static addrs, NOT
bootp/dhcp) contains:

    for (i = 0;  i < bp->bp_hlen;  i++) {
        bp->bp_chaddr[i] = 0xFF;
    }

which has confused people because it causes the displayed ESA address (displayed
by show_bootp) to be all Fs, which worries some people. Instead it should fetch
the real ESA using ioctl(s, SIOCGIFHWADDR, &ifr) like in
boot_support.c:do_bootp()