search

quantcast / android-measurement

Quantcast Measure for Android
13 stars 12 forks source link

Fixing FileInputStream resource leak. #7

Closed quinnjn closed 10 years ago

quinnjn commented 10 years ago

fis isn't being referenced. Instead there is another FileInputStream opening and reading the file which is causing a resource leak.

Fixing by reading form fis, which is closed in the finally scope.

SmithKevin commented 10 years ago

Great catch, that was definitely overlooked. We will add this changed to the next release in the next day or so. Thanks for pointing it out to us.

-Kevin

quinnjn commented 10 years ago

Awesome! Thanks for the quick reply :+1: