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
F167361347: D19214.id53991.diff
Fri, Aug 21, 7:50 AM
Unknown Object (File)
Wed, Aug 19, 4:37 AM
Unknown Object (File)
Tue, Aug 18, 9:18 AM
Unknown Object (File)
Wed, Aug 12, 12:54 PM
Unknown Object (File)
Mon, Aug 10, 2:09 PM
Unknown Object (File)
Mon, Aug 10, 2:05 PM
Unknown Object (File)
Mon, Aug 10, 1:58 PM
Unknown Object (File)
Mon, Aug 10, 1:56 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...