ifnet: if_vmove_reclaim(): Be more robust to check the src / dst vnet
AbandonedPublic
Actions

Authored by zlei on Mar 17 2026, 4:56 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Jun 15, 2:05 PM

	Unknown Object (File)
	Mon, Jun 8, 1:27 PM

	Unknown Object (File)
	May 15 2026, 2:43 AM

	Unknown Object (File)
	May 14 2026, 10:36 PM

	Unknown Object (File)
	May 12 2026, 6:24 AM

	Unknown Object (File)
	May 8 2026, 8:35 AM

	Unknown Object (File)
	Apr 19 2026, 1:45 AM

	Unknown Object (File)
	Apr 19 2026, 12:50 AM

Subscribers

Details

Reviewers

glebius

Group Reviewers

network

Summary

The ifp is only weak referenced, if_vmove_reclaim() may race with other
threads' if_vmove() or if_detach(), hence ifp->if_vnet is not safe nor
stable to access. Given we know the source and destination prison/vnet,
check them instead.

MFC after: 2 weeks

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

zlei created this revision.Mar 17 2026, 4:56 PM

Herald added subscribers: melifaro, ae, imp. · View Herald TranscriptMar 17 2026, 4:56 PM

zlei requested review of this revision.Mar 17 2026, 4:56 PM

zlei mentioned this in D55832: ifnet: if_vmove_(loan|reclaim): Refactor a bit the checking of src / dst vnet.

zlei abandoned this revision.Mar 20 2026, 6:56 PM

zlei added inline comments.

sys/net/if.c
1286	This assert is wrong. Post fix to D55997.

Revision Contents
Changeset List

Path

Size

sys/

net/

if.c

18 lines

Diff 173813

View Options

ifnet: if_vmove_reclaim(): Be more robust to check the src / dst vnetAbandonedPublicActions