Page MenuHomeFreeBSD

usr.sbin/ppp: correct SIOCDIFADDR{,_IN6} calls to use {,in6_}ifreq
ClosedPublic

Authored by def on Jul 18 2024, 2:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Oct 4, 1:58 AM
Unknown Object (File)
Tue, Sep 30, 11:52 PM
Unknown Object (File)
Tue, Sep 30, 9:00 PM
Unknown Object (File)
Tue, Sep 30, 8:14 AM
Unknown Object (File)
Sep 13 2025, 1:48 AM
Unknown Object (File)
Sep 3 2025, 3:19 PM
Unknown Object (File)
Aug 28 2025, 3:20 PM
Unknown Object (File)
Aug 26 2025, 4:12 PM
Subscribers

Details

Summary

The SIOCDIFADDR{,_IN6} ioctls take an ifreq structure object, not an
ifaliasreq structure object, as their argument.

Obtained from: CheriBSD

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

def requested review of this revision.Jul 18 2024, 2:59 PM

@brooks @oshogbo Do you have any suggestions who should review this? It is not clear to me based on the commit history.

brooks added a subscriber: jhb.

I don't see any obvious reviewer candidates. Maybe @jhb?

It looks like the original author of this commit just misunderstood how SIOCDIFADDR* worked.

This revision is now accepted and ready to land.Jul 18 2024, 7:39 PM

sbin/nos-tun/nos-tun.c and usr.sbin/uhsoctl/uhsoctl.c both have the same mistake

Thanks for reviews!

In D46018#1049505, @jhb wrote:

sbin/nos-tun/nos-tun.c and usr.sbin/uhsoctl/uhsoctl.c both have the same mistake

Yes, these are covered in:
https://reviews.freebsd.org/D46019
https://reviews.freebsd.org/D46017