Fix for a NullPointerException on first run. (openwallet-foundation-l…

…abs#453) Signed-off-by: Peter Sorotokin <[email protected]>
nicklaswj · Jan 23, 2024 · 06fe0d3 · 06fe0d3
1 parent 33408e2
commit 06fe0d3
Showing 1 changed file with 18 additions and 11 deletions.
diff --git a/...tity-android/src/main/java/com/android/identity/android/storage/AndroidStorageEngine.java b/...tity-android/src/main/java/com/android/identity/android/storage/AndroidStorageEngine.java
@@ -194,7 +194,11 @@ public void delete(@NonNull String key) {
 
     @Override
     public void deleteAll() {
-        for (File file : mStorageDirectory.listFiles()) {
+        File[] fileList = mStorageDirectory.listFiles();
+        if (fileList == null) {
+            return;
+        }
+        for (File file : fileList) {
             String name = file.getName();
             if (!name.startsWith(PREFIX)) {
                 continue;
@@ -207,16 +211,19 @@ public void deleteAll() {
     @NonNull
     public Collection<String> enumerate() {
         ArrayList<String> ret = new ArrayList<>();
-        for (File file : mStorageDirectory.listFiles()) {
-            String name = file.getName();
-            if (!name.startsWith(PREFIX)) {
-                continue;
-            }
-            try {
-                String decodedName = URLDecoder.decode(name.substring(PREFIX.length()), "UTF-8");
-                ret.add(decodedName);
-            } catch (UnsupportedEncodingException e) {
-                throw new IllegalStateException(e);
+        File[] fileList = mStorageDirectory.listFiles();
+        if (fileList != null) {
+            for (File file : fileList) {
+                String name = file.getName();
+                if (!name.startsWith(PREFIX)) {
+                    continue;
+                }
+                try {
+                    String decodedName = URLDecoder.decode(name.substring(PREFIX.length()), "UTF-8");
+                    ret.add(decodedName);
+                } catch (UnsupportedEncodingException e) {
+                    throw new IllegalStateException(e);
+                }
             }
         }
         return ret;