Page MenuHomeFreeBSD
Differential D54918

ifconfig: Report errors from failed FIB commands
ClosedPublic
Actions

Authored by markj on Jan 27 2026, 6:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 12, 4:55 AM
Unknown Object (File)
Tue, Feb 10, 12:22 AM
Unknown Object (File)
Sat, Feb 7, 2:13 PM
Unknown Object (File)
Sat, Feb 7, 1:34 AM
Unknown Object (File)
Sat, Jan 31, 8:15 AM
Unknown Object (File)
Sat, Jan 31, 6:30 AM
Unknown Object (File)
Fri, Jan 30, 11:11 PM
Unknown Object (File)
Thu, Jan 29, 8:07 AM
View All 12 Files
Subscribers
imp
melifaro
pouria
zlei

Details

Reviewers
pouria
zlei
melifaro
Group Reviewers
network
Commits
rGd4763c1fe5bc: ifconfig: Exit with a non-zero status when SIOCSIFFIB fails
rG147873a1a7d8: ifconfig: Exit with a non-zero status when SIOCSIFFIB fails
rG2ea85a622bcb: ifconfig: Exit with a non-zero status when SIOCSIFFIB fails
Summary

Previously, setting an interface FIB to some invalid value would result
in a warning being printed, but the ifconfig command would exit with
status 0, but this seems wrong.

Diff Detail

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

Event Timeline

markj created this revision.Jan 27 2026, 6:32 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 27 2026, 6:32 PM
markj requested review of this revision.Jan 27 2026, 6:32 PM
Harbormaster completed remote builds in B70171: Diff 170547.Jan 27 2026, 6:32 PM
pouria accepted this revision as: pouria.Jan 27 2026, 9:20 PM
pouria added a subscriber: pouria.

LGTM

This revision is now accepted and ready to land.Jan 27 2026, 9:20 PM
zlei accepted this revision as: zlei.Jan 28 2026, 3:20 AM
zlei added a subscriber: zlei.

Looks good to me.

zlei added a subscriber: melifaro.Jan 28 2026, 3:28 AM
zlei added inline comments.
sbin/ifconfig/iffib.c
97

CC the author @melifaro to see if the warn() is intentional .

melifaro accepted this revision.Jan 28 2026, 11:27 AM
Closed by commit rG2ea85a622bcb: ifconfig: Exit with a non-zero status when SIOCSIFFIB fails (authored by markj). · Explain WhyJan 28 2026, 4:14 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG2ea85a622bcb: ifconfig: Exit with a non-zero status when SIOCSIFFIB fails.
markj added a commit: rG147873a1a7d8: ifconfig: Exit with a non-zero status when SIOCSIFFIB fails.Mon, Feb 16, 3:44 PM
markj added a commit: rGd4763c1fe5bc: ifconfig: Exit with a non-zero status when SIOCSIFFIB fails.Mon, Feb 16, 7:47 PM

Revision Contents
Changeset List

PathSize
sbin/
ifconfig/
16 lines
tests/
3 lines
32 lines

Diff 170647

View Options

sbin/ifconfig/iffib.c

Loading...
View Options

sbin/ifconfig/tests/Makefile

Loading...
View Options

sbin/ifconfig/tests/ifconfig.sh

Loading...