Differential D56099

ifconfig: fix Perrorc to use error parameter instead of errno
Needs ReviewPublic
Actions

Authored by chris.longros_gmail.com on Thu, Mar 26, 8:53 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Mar 31, 2:21 PM

	Unknown Object (File)
	Tue, Mar 31, 3:26 AM

	Unknown Object (File)
	Mon, Mar 30, 3:45 AM

	Unknown Object (File)
	Sun, Mar 29, 4:28 PM

	Unknown Object (File)
	Sun, Mar 29, 2:52 AM

Subscribers

Details

Reviewers

Summary

The switch statement checks errno instead of the error argument
passed by the caller, causing ENXIO and EPERM to be missed for
Netlink operations.

Signed-off-by: Christos Longros <chris.longros@gmail.com>

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 71730
Build 68613: arc lint + arc unit

Event Timeline

chris.longros_gmail.com created this revision.Thu, Mar 26, 8:53 PM

Herald added a subscriber: imp. · View Herald TranscriptThu, Mar 26, 8:53 PM

chris.longros_gmail.com requested review of this revision.Thu, Mar 26, 8:53 PM

Harbormaster completed remote builds in B71730: Diff 174355.Thu, Mar 26, 8:53 PM

chris.longros_gmail.com added a reviewer: melifaro.Thu, Mar 26, 8:54 PM

Revision Contents
Changeset List

Path

Size

sbin/

ifconfig/

2 lines

Diff 174355

sbin/ifconfig/ifconfig.c

Loading...