summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> 2024-08-09 17:47:23 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2024-08-09 17:47:23 +0000
commitd439c6190e217780329f747d18e9683fce8a5002 (patch)
treee093dc8e62454d0da5c9bbf1126c040a6983555f
parent12af3c29e5fa09358df16f9a286380f12594356f (diff)
parent70c4c4f886cb4b4d72b2768659d151ffe2d01516 (diff)
Merge "SystemCertificateSource: Mitigate NPE when checking updateable certs directory" into main am: b7e715d8d9 am: 70c4c4f886
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/3165417 Change-Id: Ibe25e36d666597428caa28ec7666dbdacce1e171 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r--core/java/android/security/net/config/SystemCertificateSource.java4
1 files changed, 3 insertions, 1 deletions
diff --git a/core/java/android/security/net/config/SystemCertificateSource.java b/core/java/android/security/net/config/SystemCertificateSource.java
index 3a254c1d92fc..bdda42a389eb 100644
--- a/core/java/android/security/net/config/SystemCertificateSource.java
+++ b/core/java/android/security/net/config/SystemCertificateSource.java
@@ -19,6 +19,8 @@ package android.security.net.config;
import android.os.Environment;
import android.os.UserHandle;
+import com.android.internal.util.ArrayUtils;
+
import java.io.File;
/**
@@ -45,7 +47,7 @@ public final class SystemCertificateSource extends DirectoryCertificateSource {
}
File updatable_dir = new File("/apex/com.android.conscrypt/cacerts");
if (updatable_dir.exists()
- && !(updatable_dir.list().length == 0)) {
+ && !(ArrayUtils.isEmpty(updatable_dir.list()))) {
return updatable_dir;
}
return new File(System.getenv("ANDROID_ROOT") + "/etc/security/cacerts");