Page MenuHomeFreeBSD
Differential D45769

dhclient: correct struct ifreq allocation
ClosedPublic
Actions

Authored by brooks on Jun 28 2024, 2:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 8, 10:32 PM
Unknown Object (File)
Sat, Apr 4, 3:24 AM
Unknown Object (File)
Fri, Apr 3, 11:43 PM
Unknown Object (File)
Wed, Apr 1, 7:19 AM
Unknown Object (File)
Mon, Mar 30, 3:34 AM
Unknown Object (File)
Sun, Mar 29, 7:56 AM
Unknown Object (File)
Sat, Mar 28, 12:14 PM
Unknown Object (File)
Fri, Mar 27, 3:43 AM
View All 87 Files
Subscribers
imp

Details

Reviewers
melifaro
def
Commits
rGa2e65d45a5c6: dhclient: correct struct ifreq allocation
Summary

ioctl commands such as BIOCSETIF take a struct ifreq and due to
FreeBSD's ioctl implementation copy exactly sizeof(struct ifreq) bytes
in so allocate exactly that much space.

The over-allocaton was harmless, but useless.

Reported by: def
Fixes: e2dc8d789f68a dhclient: do not add 0.0.0.0 interface alias.
Sponsored by: DARPA, AFRL

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 58385
Build 55273: arc lint + arc unit

Event Timeline

brooks created this revision.Jun 28 2024, 2:08 PM
Herald added a subscriber: imp. · View Herald TranscriptJun 28 2024, 2:08 PM
brooks requested review of this revision.Jun 28 2024, 2:08 PM
Harbormaster completed remote builds in B58385: Diff 140340.Jun 28 2024, 2:08 PM
brooks added a comment.Jun 28 2024, 2:09 PM

This was previously been fixed in f2c99d387c4770a7516237d5829b140e41a748d9 where the under allocation was potentially not harmless.

def accepted this revision.Jun 28 2024, 2:19 PM
This revision is now accepted and ready to land.Jun 28 2024, 2:19 PM
Closed by commit rGa2e65d45a5c6: dhclient: correct struct ifreq allocation (authored by brooks). · Explain WhyJul 4 2024, 11:32 AM
This revision was automatically updated to reflect the committed changes.
brooks added a commit: rGa2e65d45a5c6: dhclient: correct struct ifreq allocation.

Revision Contents
Changeset List

PathSize
sbin/
dhclient/
3 lines

Diff 140340

View Options

sbin/dhclient/dispatch.c

Loading...