ifnet(9): split ifc_alloc_unit() (should make code a bit cleaner).
ClosedPublic
Actions

Authored by avos on Sep 30 2017, 9:27 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Dec 6, 11:36 PM

	Unknown Object (File)
	Sat, Nov 29, 1:54 PM

	Unknown Object (File)
	Fri, Nov 28, 3:36 PM

	Unknown Object (File)
	Thu, Nov 27, 9:22 AM

	Unknown Object (File)
	Wed, Nov 26, 5:30 PM

	Unknown Object (File)
	Mon, Nov 24, 10:15 PM

	Unknown Object (File)
	Nov 23 2025, 9:22 PM

	Unknown Object (File)
	Nov 21 2025, 12:40 AM

View All Files

Subscribers

imp

Details

Reviewers

glebius

Group Reviewers

network

Commits

rS324813: MFC r324672:
rS324812: MFC r324672:
rS324672: ifnet(9): split ifc_alloc_unit() (should simplify code flow)

Summary

Allocate smallest unit number from pool from ifc_alloc_unit_next() and exact unit number (if available) from ifc_alloc_unit_specific().

While here, fix potential endless loop (*unit < 0 -> alloc_unr() succeeds but ifunit() fails -> (*unit)++ and then alloc_unr_specific() fails ->(*unit) is overridden with return result (-1) and then used instead of previous unit number).

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

avos created this revision.Sep 30 2017, 9:27 PM

Herald added subscribers: ae, imp. · View Herald TranscriptSep 30 2017, 9:27 PM

Does this fix this bug? https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217401

Yes (but without reassigning *unit - it will not change after alloc_unr_specific() anyway).

Can you please grab the PR? :)

Closed by commit rS324672: ifnet(9): split ifc_alloc_unit() (should simplify code flow) (authored by avos). · Explain WhyOct 16 2017, 9:21 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

net/

if_clone.c

60 lines

Diff 34037

View Options

ifnet(9): split ifc_alloc_unit() (should make code a bit cleaner).ClosedPublicActions