Paths

Table of Contentst

Make temp option for ndp work
Needs ReviewPublic
Actions

Authored by lytboris_gmail.com on Mar 23 2025, 8:20 AM.

Tags

Referenced Files

	Unknown Object (File)
	Tue, May 6, 8:35 PM

	Unknown Object (File)
	Mon, Apr 28, 8:24 PM

	Unknown Object (File)
	Thu, Apr 17, 2:47 AM

	Unknown Object (File)
	Wed, Apr 9, 8:07 PM

	Unknown Object (File)
	Apr 7 2025, 5:49 AM

	Unknown Object (File)
	Apr 7 2025, 5:49 AM

	Unknown Object (File)
	Apr 7 2025, 5:49 AM

	Unknown Object (File)
	Apr 7 2025, 5:49 AM

View All 18 Files

Subscribers

imp

Details

Reviewers

melifaro
glebius

Summary

Both ways to create an artificial NDP record are broken: a call via rtsock sets wrong timeout, a netlink code does not check opts.expire_time value at all.

Patch addresses both issues.

Test Plan

add 2001:DB8::/64 to an interface
run ndp -s 2001:DB8::c001 11:22:33:44:55:66 temp
check ndp -an | fgrep 2001:DB8::c001 to have < 20 minutes timeout
rebuild ndp with -DWITHOUT_NETLINK=1 and repeat steps 1-3

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

lytboris_gmail.com created this revision.Mar 23 2025, 8:20 AM

Herald added a subscriber: imp. · View Herald TranscriptMar 23 2025, 8:20 AM

lytboris_gmail.com requested review of this revision.Mar 23 2025, 8:20 AM

lytboris_gmail.com added a parent revision: D43983: arp: fix arp -s/-S.Mar 23 2025, 8:28 AM

ae added a reviewer: glebius.Mar 23 2025, 8:28 AM

Thank you for working on this! Can we add tests here?

Revision Contents
Changeset List

Path

Size

usr.sbin/

ndp/

ndp.c

11 lines

ndp_netlink.c

18 lines

Diff 152548

View Options

usr.sbin/ndp/ndp.c