pf: explicitly NULL state key pointers
ClosedPublic
Actions

Authored by kp on Fri, Mar 28, 2:48 PM.

Details

Reviewers

glebius
markj

Group Reviewers

network

Commits

rG592418343348: pf: explicitly NULL state key pointers

Summary

After the pf_state_insert() call we may not use these pointers again.
Explicitly NULL them to ensure we don't.
Also NULL them out if we free the keys directly.

MFC after: 3 weeks
Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kp created this revision.Fri, Mar 28, 2:48 PM

Herald added subscribers: vegeta_tuxpowered.net, melifaro, farrokhi, imp. · View Herald TranscriptFri, Mar 28, 2:48 PM

kp requested review of this revision.Fri, Mar 28, 2:48 PM

Harbormaster completed remote builds in B63177: Diff 152792.Fri, Mar 28, 2:48 PM

kp added a parent revision: D49552: pf: simplify action resolution in pf_test_rule().Fri, Mar 28, 2:48 PM

glebius accepted this revision as: glebius.Fri, Mar 28, 8:09 PM

This revision is now accepted and ready to land.Fri, Mar 28, 8:09 PM

markj accepted this revision as: markj.Sat, Mar 29, 8:20 AM

Closed by commit rG592418343348: pf: explicitly NULL state key pointers (authored by kp). · Explain WhyMon, Mar 31, 2:58 PM

This revision was automatically updated to reflect the committed changes.

kp added a commit: rG592418343348: pf: explicitly NULL state key pointers.

Revision Contents
Changeset List

Path

Size

sys/

netpfil/

pf/

pf.c

4 lines

Diff 152907

View Options

pf: explicitly NULL state key pointersClosedPublicActions