Merge "Close the FileStream to avoid resource leak" am: c54117c71e am: 3a3001fea5

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1710968

Change-Id: I0986c86a8671fc018bfe76d31a0db6bc39bcf6a7
This commit is contained in:
Treehugger Robot
2021-05-21 01:06:00 +00:00
committed by Automerger Merge Worker

View File

@@ -2859,8 +2859,7 @@ public class InputManagerService extends IInputManager.Stub
};
for (File baseDir: baseDirs) {
File confFile = new File(baseDir, EXCLUDED_DEVICES_PATH);
try {
InputStream stream = new FileInputStream(confFile);
try (InputStream stream = new FileInputStream(confFile)) {
names.addAll(ConfigurationProcessor.processExcludedDeviceNames(stream));
} catch (FileNotFoundException e) {
// It's ok if the file does not exist.
@@ -2893,8 +2892,7 @@ public class InputManagerService extends IInputManager.Stub
final File baseDir = Environment.getVendorDirectory();
final File confFile = new File(baseDir, PORT_ASSOCIATIONS_PATH);
try {
final InputStream stream = new FileInputStream(confFile);
try (final InputStream stream = new FileInputStream(confFile)) {
return ConfigurationProcessor.processInputPortAssociations(stream);
} catch (FileNotFoundException e) {
// Most of the time, file will not exist, which is expected.
@@ -3015,10 +3013,10 @@ public class InputManagerService extends IInputManager.Stub
@Override
public void visitKeyboardLayout(Resources resources,
int keyboardLayoutResId, KeyboardLayout layout) {
try {
try (final InputStreamReader stream = new InputStreamReader(
resources.openRawResource(keyboardLayoutResId))) {
result[0] = layout.getDescriptor();
result[1] = Streams.readFully(new InputStreamReader(
resources.openRawResource(keyboardLayoutResId)));
result[1] = Streams.readFully(stream);
} catch (IOException ex) {
} catch (NotFoundException ex) {
}