Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

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

Browse Source 
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
parent e42698f7f7 3a3001fea5
commit 2422cbb8be
1 changed files with 5 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
services/core/java/com/android/server/input/InputManagerService.java
View File

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