Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "AnomalyReport catch any exception when try to read deviceConfig" into tm-qpr-dev am: f60a915197 am: 292caba3ab

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/20003467

Change-Id: Idc1c0b1fc96a4ca3ea17e0a54b68bb427756818c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
TreeHugger Robot
2022-09-21 21:49:22 +00:00
committed by Automerger Merge Worker
parent 4a112aaf43 292caba3ab
commit 5728b473d9
1 changed files with 12 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
telephony/java/android/telephony/AnomalyReporter.java
View File

@@ -105,17 +105,23 @@ public final class AnomalyReporter {
* @param carrierId the carrier of the id associated with this event.
*/
public static void reportAnomaly(@NonNull UUID eventId, String description, int carrierId) {
// Don't report if the server-side flag isn't loaded, as it implies other anomaly report
// related config hasn't loaded.
boolean isAnomalyReportEnabledFromServer = DeviceConfig.getBoolean(
DeviceConfig.NAMESPACE_TELEPHONY, KEY_IS_TELEPHONY_ANOMALY_REPORT_ENABLED, false);
if (!isAnomalyReportEnabledFromServer) return;
if (sContext == null) {
Rlog.w(TAG, "AnomalyReporter not yet initialized, dropping event=" + eventId);
return;
}
// Don't report if the server-side flag isn't loaded, as it implies other anomaly report
// related config hasn't loaded.
try {
boolean isAnomalyReportEnabledFromServer = DeviceConfig.getBoolean(
DeviceConfig.NAMESPACE_TELEPHONY, KEY_IS_TELEPHONY_ANOMALY_REPORT_ENABLED,
false);
if (!isAnomalyReportEnabledFromServer) return;
} catch (Exception e) {
Rlog.w(TAG, "Unable to read device config, dropping event=" + eventId);
return;
}
TelephonyStatsLog.write(
TELEPHONY_ANOMALY_DETECTED,
carrierId,