summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Rubin Xu <rubinxu@google.com> 2019-09-04 15:58:07 +0100
committer Rubin Xu <rubinxu@google.com> 2019-09-04 17:14:28 +0100
commit40f14ac22e9798441e368f7f319ceaa7bfcc0590 (patch)
treede64103c2951348653728ffa587c3aa0acb0b733
parentf963e0d5804489101df5f58464a18baa9fc1001c (diff)
Suppress FileNotFoundException in LockSettingsService
Remove log spam resulted from LockSettingsStorage looking for user credential hashes on the disk by trying several possible paths. Bug: 62410009 Test: boot device, check there is no exception log from LSS Change-Id: I1530c381316dce21d862f82244cc99cc45a8774a
-rw-r--r--services/core/java/com/android/server/locksettings/LockSettingsStorage.java16
1 files changed, 5 insertions, 11 deletions
diff --git a/services/core/java/com/android/server/locksettings/LockSettingsStorage.java b/services/core/java/com/android/server/locksettings/LockSettingsStorage.java
index fe12a945bd4c..559461485042 100644
--- a/services/core/java/com/android/server/locksettings/LockSettingsStorage.java
+++ b/services/core/java/com/android/server/locksettings/LockSettingsStorage.java
@@ -47,6 +47,7 @@ import java.io.ByteArrayOutputStream;
import java.io.DataInputStream;
import java.io.DataOutputStream;
import java.io.File;
+import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.RandomAccessFile;
import java.nio.channels.FileChannel;
@@ -325,23 +326,16 @@ class LockSettingsStorage {
version = mCache.getVersion();
}
- RandomAccessFile raf = null;
byte[] stored = null;
- try {
- raf = new RandomAccessFile(name, "r");
+ try (RandomAccessFile raf = new RandomAccessFile(name, "r")) {
stored = new byte[(int) raf.length()];
raf.readFully(stored, 0, stored.length);
raf.close();
+ } catch (FileNotFoundException suppressed) {
+ // readFile() is also called by hasFile() to check the existence of files, in this
+ // case FileNotFoundException is expected.
} catch (IOException e) {
Slog.e(TAG, "Cannot read file " + e);
- } finally {
- if (raf != null) {
- try {
- raf.close();
- } catch (IOException e) {
- Slog.e(TAG, "Error closing file " + e);
- }
- }
}
mCache.putFileIfUnchanged(name, stored, version);
return stored;