do not FALLTHROUGH for SIOC{ADD,DEL}MULTI
ClosedPublic
Actions

Authored by sjg on Oct 15 2015, 12:02 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Jan 19 2024, 8:33 PM

	Unknown Object (File)
	Dec 20 2023, 3:21 AM

	Unknown Object (File)
	Dec 3 2023, 7:02 AM

	Unknown Object (File)
	Nov 29 2023, 2:08 PM

	Unknown Object (File)
	Jun 20 2023, 8:34 PM

	Unknown Object (File)
	Jun 19 2023, 5:57 AM

	Unknown Object (File)
	Apr 26 2023, 3:28 PM

	Unknown Object (File)
	Apr 8 2023, 8:59 PM

View All 28 Files

Subscribers

None

Details

Reviewers

marcel

Summary

FALLTHROUGH results in return of EINVAL since ifmedia_ioctl()
does not grok SIOC{ADD,DEL}MULTI

This hurts if caller is paying attention to errors.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 769
Build 769: arc lint + arc unit

Event Timeline

sjg updated this revision to Diff 9398.Oct 15 2015, 12:02 AM

sjg retitled this revision from to do not FALLTHROUGH for SIOC{ADD,DEL}MULTI.

sjg updated this object.

sjg edited the test plan for this revision. (Show Details)

sjg added a reviewer: marcel.Oct 30 2015, 4:47 AM

LGTM

This revision is now accepted and ready to land.Oct 30 2015, 4:20 PM

Code was committed.

Revision Contents
Changeset List

Path

Size

sys/

dev/

xen/

netfront/

netfront.c

2 lines

Diff 9398

View Options

sys/dev/xen/netfront/netfront.c

do not FALLTHROUGH for SIOC{ADD,DEL}MULTIClosedPublicActions