| commit | 1f8d263151e52a170241f3e05acfc86fdc957ed2 | [log] [tgz] |
|---|---|---|
| author | Junyu Lai <junyulai@google.com> | Fri Jan 19 02:54:42 2024 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Fri Jan 19 02:54:42 2024 +0000 |
| tree | ce6a94c83dd1982eb4f65366a29c63fe47f63555 | |
| parent | 5c25dbe57bb606fba183892252ab4b29be31098d [diff] | |
| parent | 91694abf58c27d5d9296141e23d2555f27006681 [diff] |
Merge "Check stats dir instead of making one" into main am: 91694abf58 Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2916466 Change-Id: I4851532de013d0edc7622cd007a98822314fd9b2 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/service-t/src/com/android/metrics/NetworkStatsMetricsLogger.java b/service-t/src/com/android/metrics/NetworkStatsMetricsLogger.java index 3ed21a2..08a8603 100644 --- a/service-t/src/com/android/metrics/NetworkStatsMetricsLogger.java +++ b/service-t/src/com/android/metrics/NetworkStatsMetricsLogger.java
@@ -109,7 +109,7 @@ */ private static Pair<Integer, Integer> getStatsFilesAttributes( @Nullable File statsDir, @NonNull String prefix) { - if (statsDir == null) return new Pair<>(0, 0); + if (statsDir == null || !statsDir.isDirectory()) return new Pair<>(0, 0); // Only counts the matching files. // The files are named in the following format: @@ -118,9 +118,6 @@ // See FileRotator#FileInfo for more detail. final Pattern pattern = Pattern.compile("^" + prefix + "\\.[0-9]+-[0-9]*$"); - // Ensure that base path exists. - statsDir.mkdirs(); - int totalFiles = 0; int totalBytes = 0; for (String name : emptyIfNull(statsDir.list())) {