Change privilege mapping error to warning in some cases

Some privileges can be mapped to none privacy-related, such
behavior should not be logged as an error.

Change-Id: Ia73aafdb6c846d12b5c77a1265faeca8f0e1e06c

diff --git a/src/common/policy/PrivilegeInfo.cpp b/src/common/policy/PrivilegeInfo.cpp
index 6ebf193e0171b924fe5f5e3e6d1a174ae2726ea2..ab6af771ba932e68c5926e1c5e994bb4d9618fc3 100644 (file)
--- a/src/common/policy/PrivilegeInfo.cpp
+++ b/src/common/policy/PrivilegeInfo.cpp
@@ -157,11 +157,16 @@ std::vector<Privilege> getSamePrivacyPrivilegeMapping(const Privilege &privilege
 
     int ret = privilege_info_get_same_privacy_grouped_privileges(privilege.c_str(), &privMapped);
 
-    if (ret != PRVMGR_ERR_NONE || !privMapped) {
+    if (ret != PRVMGR_ERR_NONE) {
         ALOGE("Unable to get privacy mapping of privilege " << privilege << "; err: <" << ret <<  ">");
         return {};
     }
 
+    if (!privMapped) {
+        ALOGW("Privilege " << privilege << " maps to empty set of related same-privacy privileges");
+        return {};
+    }
+
     GListWrap privMappedWrap(privMapped, privilege_info_list_free);
     std::vector<std::string> privMappedVector;
     for (GList *l = privMappedWrap.get(); l != NULL; l = l->next) {