Skip to content

Commit

Permalink
Device SELinux policy for secure persistent netd storage
Browse files Browse the repository at this point in the history
This is used to persist RFC 7217 stable secrets across device reboots.
For now this is device-specific; in future releases it will be moved
to public sepolicy.

Bug: 17613910
Test: stable_secret is generated on first use and persists across reboots
Test: "adb shell getprop persist.netd.stable_secret" doesn't work after "adb unroot"
Change-Id: I0a609c724799a15b1926e62534c16810d34f2275
  • Loading branch information
lcolitti committed Jul 11, 2017
1 parent 4e86a6c commit 77e8a62
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 0 deletions.
10 changes: 10 additions & 0 deletions sepolicy/private/netd.te
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
# TODO: delete this once it's in common sepolicy.
set_prop(netd, netd_stable_secret_prop)

# persist.netd.stable_secret contains RFC 7217 secret key which should never be
# leaked to other processes. Make sure it never leaks.
neverallow { domain -netd -init } netd_stable_secret_prop:file r_file_perms;

# We want to ensure that no other process ever tries tampering with persist.netd.stable_secret,
# the RFC 7217 secret key managed by netd. Doing so could compromise user privacy.
neverallow { domain -netd -init } netd_stable_secret_prop:property_service set;
2 changes: 2 additions & 0 deletions sepolicy/private/property.te
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
# TODO: delete this once it's in common sepolicy.
type netd_stable_secret_prop, property_type;
2 changes: 2 additions & 0 deletions sepolicy/private/property_contexts
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
# TODO: delete this once it's in common sepolicy.
persist.netd.stable_secret u:object_r:netd_stable_secret_prop:s0

0 comments on commit 77e8a62

Please sign in to comment.