Commit b2a8b88f authored Feb 02, 2018 by Jeff Davidson

Reduce logspam in carrier privilege status check.

UiccCarrierPrivilegeRules can be called by non-platform code via
system APIs (TelephonyManager#checkCarrierPrivilegesForPackage), and
thus it is not an error worthy of logging if the given package is not
installed. It is also easy to infer that a package is not
installed from a bugreport (or determine when it was installed, if
it is installed later). Just return NO_ACCESS.

Change-Id: If068fadb34ea6256287d9e92f00cd2dd5b5ddc9b
Fixes: 72764153
Test: TreeHugger (CL is trivial)

parent 7abeb535

src/java/com/android/internal/telephony/uicc/UiccCarrierPrivilegeRules.java

+2 −2

Original line number	Diff line number	Diff line
		@@ -302,7 +302,7 @@ public class UiccCarrierPrivilegeRules extends Handler {
		\| PackageManager.MATCH_DISABLED_UNTIL_USED_COMPONENTS);
		return getCarrierPrivilegeStatus(pInfo);
		} catch (PackageManager.NameNotFoundException ex) {
		Rlog.e(LOG_TAG, "NameNotFoundException", ex);
		log("Package " + packageName + " not found for carrier privilege status check");
		}
		return TelephonyManager.CARRIER_PRIVILEGE_STATUS_NO_ACCESS;
		}