Page MenuHomeFreeBSD
Differential D8624

Add tunable to disable destructive dtrace Security
ClosedPublic
Actions

Authored by gnn on Nov 23 2016, 4:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 12, 10:42 AM
Unknown Object (File)
Sun, May 11, 10:47 AM
Unknown Object (File)
Thu, May 8, 11:45 PM
Unknown Object (File)
Tue, Apr 15, 5:52 PM
Unknown Object (File)
Apr 11 2025, 7:08 AM
Unknown Object (File)
Apr 11 2025, 1:21 AM
Unknown Object (File)
Mar 23 2025, 11:49 AM
Unknown Object (File)
Mar 3 2025, 2:13 AM
View All 75 Files
Subscribers
imp
rstone

Details

Reviewers
markj
rstone
Commits
rS309069: Add tunable to disable destructive dtrace

Diff Detail

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

Event Timeline

gnn updated this revision to Diff 22463.Nov 23 2016, 4:47 PM
gnn retitled this revision from to Add tunable to disable destructive dtrace Security.
gnn updated this object.
gnn edited the test plan for this revision. (Show Details)
gnn added a reviewer: markj.
Herald added a subscriber: imp. · View Herald TranscriptNov 23 2016, 4:47 PM
rstone added a subscriber: rstone.Nov 23 2016, 4:50 PM
rstone added inline comments.
sys/cddl/contrib/opensolaris/uts/common/dtrace/dtrace.c
162 ↗(On Diff #22463)

is there any reason to make this a global?

rstone accepted this revision.Nov 23 2016, 4:50 PM
rstone added a reviewer: rstone.
rstone added inline comments.
sys/cddl/contrib/opensolaris/uts/common/dtrace/dtrace.c
162 ↗(On Diff #22463)

never mind, missed the sysctl

This revision is now accepted and ready to land.Nov 23 2016, 4:50 PM
markj accepted this revision.Nov 23 2016, 5:18 PM
markj edited edge metadata.
markj added inline comments.
sys/cddl/dev/dtrace/dtrace_load.c
60 ↗(On Diff #22463)

I thought that a RDTUN sysctl would automatically fetch the tunable value, so I'm surprised that this bit is needed.

gnn marked an inline comment as done.Nov 23 2016, 10:45 PM
Closed by commit rS309069: Add tunable to disable destructive dtrace (authored by gnn). · Explain WhyNov 23 2016, 10:50 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
cddl/
contrib/
opensolaris/
uts/
common/
dtrace/
4 lines
dev/
dtrace/
11 lines
3 lines

Diff 22471

View Options

head/sys/cddl/contrib/opensolaris/uts/common/dtrace/dtrace.c

Loading...
View Options

head/sys/cddl/dev/dtrace/dtrace_load.c

Loading...
View Options

head/sys/cddl/dev/dtrace/dtrace_sysctl.c

Loading...