commit 84f48cdf9f208962d28d7f34643094f3c2437c8d
author Chenbo Feng <fengc@google.com> Mon Apr 22 15:34:40 2019 -0700
committer Chenbo Feng <fengc@google.com> Tue May 07 12:58:01 2019 -0700
tree 0da9b690dfe9d3c2b0926bdea5102aded799d466
parent aae6f53ed0a1b0769c58774f702f9cbb49a0bf98

replace NO_PERMISSIONS with PERMISSION_NONE

As part of the effort to simplify the design of permission control
between system server and netd, remove the NO_PERMISSIONS constant used
by trafficController and share the constant PERMISSION_NONE with
networkController instead.

Bug: 128944261
Test: netd_integration_test netd_unit_test
Change-Id: If3eb19adc15b6c23c95fc4b9bc699fd1a515e0ac

server/TrafficController.cpp

diff --git a/server/TrafficController.cpp b/server/TrafficController.cpp
index 9d9a3b5..e669f17 100644
--- a/server/TrafficController.cpp
+++ b/server/TrafficController.cpp
@@ -114,8 +114,8 @@
 }
 
 const std::string UidPermissionTypeToString(uint8_t permission) {
-    if (permission == INetd::NO_PERMISSIONS) {
-        return "NO_PERMISSIONS";
+    if (permission == INetd::PERMISSION_NONE) {
+        return "PERMISSION_NONE";
     }
     if (permission == INetd::PERMISSION_UNINSTALLED) {
         // This should never appear in the map, complain loudly if it does.