Page MenuHomeFreeBSD
Differential D19214

Fix refcount leak in SGX Linux compat ioctl handlers.
ClosedPublic
Actions

Authored by markj on Feb 16 2019, 6:31 PM.
Tags
None
Referenced Files
F174475077: D19214.id54016.diff
Sat, Oct 3, 1:05 PM
Unknown Object (File)
Wed, Sep 30, 7:13 PM
Unknown Object (File)
Wed, Sep 30, 7:11 PM
Unknown Object (File)
Wed, Sep 30, 5:31 PM
Unknown Object (File)
Wed, Sep 30, 5:52 AM
Unknown Object (File)
Tue, Sep 22, 9:33 PM
Unknown Object (File)
Mon, Sep 21, 11:40 AM
Unknown Object (File)
Fri, Sep 11, 12:31 PM
View All Files
Subscribers
imp

Details

Reviewers
br
kib
Commits
rS344232: Fix refcount leaks in the SGX Linux compat ioctl handler.
Summary

Also get rid of some debug prints while here.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj created this revision.Feb 16 2019, 6:31 PM
Harbormaster completed remote builds in B22530: Diff 53991.Feb 16 2019, 6:31 PM
markj updated this revision to Diff 53992.Feb 16 2019, 6:37 PM
  • Return the error from copyin(9).
Harbormaster completed remote builds in B22531: Diff 53992.Feb 16 2019, 6:37 PM
markj added reviewers: br, kib.Feb 16 2019, 6:38 PM
kib accepted this revision.Feb 16 2019, 8:26 PM
kib added inline comments.
sys/amd64/sgx/sgx_linux.c
84 ↗(On Diff #53992)

Fix style there as well ? (!= 0)

This revision is now accepted and ready to land.Feb 16 2019, 8:26 PM
Closed by commit rS344232: Fix refcount leaks in the SGX Linux compat ioctl handler. (authored by markj). · Explain WhyFeb 17 2019, 4:44 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptFeb 17 2019, 4:44 PM

Revision Contents
Changeset List

PathSize
head/
sys/
amd64/
sgx/
22 lines

Diff 54016

View Options

head/sys/amd64/sgx/sgx_linux.c

Loading...