route: remove an outdated comment from rtrequest1_fib
Needs ReviewPublic
Actions

Authored by jacob.e.keller_intel.com on Nov 5 2019, 5:40 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Jan 10, 10:39 PM

	Unknown Object (File)
	Fri, Jan 9, 4:47 PM

	Unknown Object (File)
	Tue, Dec 30, 11:30 AM

	Unknown Object (File)
	Fri, Dec 26, 7:58 PM

	Unknown Object (File)
	Dec 16 2025, 3:49 PM

	Unknown Object (File)
	Nov 26 2025, 8:38 PM

	Unknown Object (File)
	Nov 24 2025, 9:56 AM

	Unknown Object (File)
	Nov 1 2025, 4:57 AM

View All 67 Files

Subscribers

Details

Reviewers

jhb
mmacy
erj

Summary

revision r334222 modified rtrequest1_fib to always take a reference on
the ifa structure. However, the comment above this indicates the code is
relying on a reference from rt_getifa_fib().

Since this is no longer the case, remove this outdated comment.

Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 27358
Build 25608: arc lint + arc unit

Event Timeline

jacob.e.keller_intel.com created this revision.Nov 5 2019, 5:40 PM

Herald added a subscriber: ae. · View Herald TranscriptNov 5 2019, 5:40 PM

Harbormaster completed remote builds in B27358: Diff 63965.Nov 5 2019, 5:40 PM

I found this while debugging an extra reference on the ifa refcount. I think I found a "solution" that isn't quite right, but this comment is misleading.

Revision Contents
Changeset List

Path

Size

sys/

net/

route.c

1 line

Diff 63965

View Options

route: remove an outdated comment from rtrequest1_fibNeeds ReviewPublicActions