Page MenuHomeFreeBSD
Differential D50461

icmp6: zero out pad space
ClosedPublic
Actions

Authored by kp on May 22 2025, 12:39 PM.
Tags
None
Referenced Files
F165689622: D50461.id155914.diff
Sun, Aug 9, 7:00 AM
Unknown Object (File)
Sat, Aug 8, 3:42 PM
Unknown Object (File)
Sat, Aug 8, 11:24 AM
Unknown Object (File)
Sat, Aug 8, 8:46 AM
Unknown Object (File)
Fri, Aug 7, 2:00 PM
Unknown Object (File)
Thu, Aug 6, 8:52 AM
Unknown Object (File)
Fri, Jul 31, 10:37 AM
Unknown Object (File)
Sat, Jul 25, 7:39 PM
View All 73 Files
Subscribers
ae
glebius
imp
melifaro
zlei

Details

Reviewers
zlei
Group Reviewers
network
pfsense
Commits
rG0a16f62701f1: icmp6: zero out pad space
Summary

In icmp6_redirect_output() we potentially add padding, but failed to clear this
memory. This triggered a KMSAN panic during the sys/netinet/carp:unicast_v6
test.

Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 64348
Build 61232: arc lint + arc unit

Event Timeline

kp created this revision.May 22 2025, 12:39 PM
Herald added subscribers: glebius, melifaro, ae, imp. · View Herald TranscriptMay 22 2025, 12:39 PM
kp requested review of this revision.May 22 2025, 12:39 PM
Harbormaster completed remote builds in B64348: Diff 155858.May 22 2025, 12:39 PM
zlei accepted this revision as: zlei.May 22 2025, 2:24 PM
zlei added a subscriber: zlei.

Looks good to me.

This revision is now accepted and ready to land.May 22 2025, 2:24 PM
Closed by commit rG0a16f62701f1: icmp6: zero out pad space (authored by kp). · Explain WhyMay 23 2025, 1:07 PM
This revision was automatically updated to reflect the committed changes.
kp added a commit: rG0a16f62701f1: icmp6: zero out pad space.

Revision Contents
Changeset List

PathSize
sys/
netinet6/
1 line

Diff 155858

View Options

sys/netinet6/icmp6.c

Loading...