Page MenuHomeFreeBSD
Differential D32076

pf: fix pagefault in pf_getstatus()
ClosedPublic
Actions

Authored by kp on Sep 23 2021, 8:42 AM.
Tags
None
Referenced Files
F170613313: D32076.id95551.diff
Sat, Sep 5, 3:43 PM
F170506011: D32076.diff
Sat, Sep 5, 3:43 AM
F170464594: D32076.id95582.diff
Fri, Sep 4, 11:05 PM
Unknown Object (File)
Thu, Sep 3, 5:07 AM
Unknown Object (File)
Thu, Sep 3, 3:01 AM
Unknown Object (File)
Wed, Sep 2, 11:53 PM
Unknown Object (File)
Wed, Sep 2, 11:11 PM
Unknown Object (File)
Wed, Sep 2, 10:38 PM
View All Files
Subscribers
farrokhi
imp
melifaro

Details

Reviewers
mjg
Group Reviewers
network
Commits
rG9419d273e471: pf: fix pagefault in pf_getstatus()
rG3c9253c2f6d1: pf: fix pagefault in pf_getstatus()
rGcb13059663e4: pf: fix pagefault in pf_getstatus()
Summary

We can't copyout() while holding a lock, in case it triggers a page
fault.
Release the lock before copyout, which is safe because we've already
copied all the data into the nvlist.

PR: 258601
MFC after: 1 week
Sponsored by: Modirum MDPay

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 41662
Build 38551: arc lint + arc unit

Revision Contents
Changeset List

PathSize
sys/
netpfil/
pf/
3 lines

Diff 95551

View Options

sys/netpfil/pf/pf_ioctl.c

Loading...