Merge "close file stream" into main

This commit is contained in:
Zhi Dou
2024-07-30 23:35:42 +00:00
committed by Gerrit Code Review

View File

@@ -18,6 +18,7 @@ package android.aconfig.storage;
import android.compat.annotation.UnsupportedAppUsage;
import java.io.Closeable;
import java.io.FileInputStream;
import java.nio.MappedByteBuffer;
import java.nio.channels.FileChannel;
@@ -68,13 +69,26 @@ public class StorageInternalReader {
// Map a storage file given file path
private static MappedByteBuffer mapStorageFile(String file) {
FileInputStream stream = null;
try {
FileInputStream stream = new FileInputStream(file);
stream = new FileInputStream(file);
FileChannel channel = stream.getChannel();
return channel.map(FileChannel.MapMode.READ_ONLY, 0, channel.size());
} catch (Exception e) {
throw new AconfigStorageException(
String.format("Fail to mmap storage file %s", file), e);
} finally {
quietlyDispose(stream);
}
}
private static void quietlyDispose(Closeable closable) {
try {
if (closable != null) {
closable.close();
}
} catch (Exception e) {
// no need to care, at least as of now
}
}
}