pf: Zero table stats structures
AbandonedPublic
Actions

Authored by markj on Jul 26 2021, 9:26 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Sep 11, 2:04 PM

	Unknown Object (File)
	Thu, Sep 10, 6:14 PM

	Unknown Object (File)
	Thu, Sep 10, 8:20 AM

	Unknown Object (File)
	Tue, Sep 8, 10:57 AM

	Unknown Object (File)
	Sun, Sep 6, 1:40 PM

	Unknown Object (File)
	Sat, Sep 5, 3:31 PM

	Unknown Object (File)
	Sat, Sep 5, 1:26 PM

	Unknown Object (File)
	Sat, Sep 5, 10:35 AM

Subscribers

Details

Reviewers

kp
mjg

Summary

They contain some uninitialized pad bytes that were being copied out to
userland.

Reported by: KMSAN
Sponsored by: The FreeBSD Foundation

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 40686
Build 37575: arc lint + arc unit

Event Timeline

markj created this revision.Jul 26 2021, 9:26 PM

Herald added subscribers: melifaro, farrokhi, imp. · View Herald TranscriptJul 26 2021, 9:26 PM

markj requested review of this revision.Jul 26 2021, 9:26 PM

markj added a parent revision: D31313: pf: Fix ioctls which copy out arrays.

Harbormaster completed remote builds in B40686: Diff 92768.Jul 26 2021, 9:26 PM

kp accepted this revision.Jul 27 2021, 7:54 AM

This revision is now accepted and ready to land.Jul 27 2021, 7:54 AM

This is no longer needed with the revised version of D31313.

Revision Contents
Changeset List

Path

Size

sys/

netpfil/

pf/

pf_table.c

1 line

Diff 92768

View Options

pf: Zero table stats structuresAbandonedPublicActions