Page MenuHomeFreeBSD
Differential D55613

rtnetlink: call rt_get_rnd when required in getroute
AbandonedPublic
Actions

Authored by pouria on Mar 2 2026, 3:00 PM.
Tags
None
Referenced Files
F166823250: D55613.id173007.diff
Sun, Aug 16, 9:11 PM
Unknown Object (File)
Thu, Aug 13, 2:12 PM
Unknown Object (File)
Thu, Aug 13, 6:45 AM
Unknown Object (File)
Thu, Aug 13, 1:25 AM
Unknown Object (File)
Wed, Aug 12, 7:22 PM
Unknown Object (File)
Mon, Aug 10, 7:26 AM
Unknown Object (File)
Sun, Aug 9, 3:21 PM
Unknown Object (File)
Sat, Aug 8, 7:10 PM
View All 43 Files
Subscribers
imp

Details

Reviewers
melifaro
glebius
zlei
Group Reviewers
network
Summary

the rib_lookup_prefix_plen() will fill the rnd data structure
in handle_rtm_getroute.
Therefore, only call rt_get_rnd() when it's necessary.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 71131
Build 68014: arc lint + arc unit

Event Timeline

pouria created this revision.Mar 2 2026, 3:00 PM
Herald added a subscriber: imp. · View Herald TranscriptMar 2 2026, 3:00 PM
pouria requested review of this revision.Mar 2 2026, 3:00 PM
Harbormaster completed remote builds in B71131: Diff 173007.Mar 2 2026, 3:00 PM
glebius requested changes to this revision.Mar 2 2026, 4:43 PM

This can't be correct cause now rt_get_rnd() may receive a NULL argument and inside it definitely dereferences it.

This revision now requires changes to proceed.Mar 2 2026, 4:43 PM
pouria abandoned this revision.Mar 2 2026, 4:45 PM

Yes, I made a stupid mistake. Thank you!

Revision Contents
Changeset List

PathSize
sys/
netlink/
route/
6 lines

Diff 173007

View Options

sys/netlink/route/rt.c

Loading...