1474928 - [Static Analysis] Resource leaked in gecko/CrashReporterService.java

Status: RESOLVED FIXED

(GeckoView :: General, defect, P3)

Description

[Robert Bartlensky [:rbartlensky]]

Infer outputs the following:

mobile/android/geckoview/src/main/java/org/mozilla/gecko/CrashReporterService.java:101: error: RESOURCE_LEAK
  resource of type `java.io.FileInputStream` acquired by call to `FileInputStream(...)` at line 93 is not released after line 101.
**Note**: potential exception at line 94
  99.               if (transferred > 0)
  100.                   inFile.delete();
  101. >         } catch (Exception e) {
  102.               Log.e(LOGTAG, "exception while copying minidump file: ", e);
  103.               return false;

mobile/android/geckoview/src/main/java/org/mozilla/gecko/CrashReporterService.java:101: error: RESOURCE_LEAK
  resource of type `java.io.FileOutputStream` acquired by call to `FileOutputStream(...)` at line 94 is not released after line 101.
**Note**: potential exception at line 96
  99.               if (transferred > 0)
  100.                   inFile.delete();
  101. >         } catch (Exception e) {
  102.               Log.e(LOGTAG, "exception while copying minidump file: ", e);
  103.               return false;

Comment 1

[Robert Bartlensky [:rbartlensky]]

mobile/android/geckoview/src/main/java/org/mozilla/gecko/CrashReporterService.java:237: error: RESOURCE_LEAK
  resource of type `java.io.FileReader` acquired by call to `new()` at line 236 is not released after line 237.
**Note**: potential exception at line 237
  235.           try {
  236.               BufferedReader reader = new BufferedReader(new FileReader(filePath));
  237. >             return readStringsFromReader(reader, stringMap);
  238.           } catch (Exception e) {
  239.               Log.e(LOGTAG, "exception while reading strings: ", e);

mobile/android/geckoview/src/main/java/org/mozilla/gecko/CrashReporterService.java:365: error: RESOURCE_LEAK
  resource of type `java.net.HttpURLConnection` acquired to `conn` by call to `openConnectionWithProxy(...)` at line 302 is not released after line 365.
**Note**: potential exception at line 309
  363.                   Log.w(LOGTAG, "Received failure HTTP response code from server: " + conn.getResponseCode());
  364.               }
  365. >         } catch (IOException e) {
  366.               Log.e(LOGTAG, "exception during send: ", e);
  367.           } catch (URISyntaxException e) {

Comment 2

[Robert Bartlensky [:rbartlensky]]

Attachment: Bug 1474928: Close files and connections.

Review commit: https://reviewboard.mozilla.org/r/256244/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/256244/

Comment 3

[Nick Alexander :nalexander [he/him] (PTO Feb 24-March 3)]

Comment on attachment 8991345 [details]
Bug 1474928: Close files and connections.

https://reviewboard.mozilla.org/r/256244/#review263526

This looks fine to me.  If it's green on try, bombs away.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d28be9c59ecb
Close files and connections. r=nalexander

Comment 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/d28be9c59ecb

Comment 6

[Chris Peterson [:cpeterson]]

I'm setting status-firefox62=wontfix because potential resource leaks in the crash reporter won't affect many users. :) If you think we might be losing crash reports without this fix, please feel free to change status-firefox62 and request beta uplift.