Review Request (1-line fix): 7032589 "FileHandler leaking file descriptor of the file lock"
forax at univ-mlv.fr
Fri Apr 15 18:05:53 UTC 2011
On 04/15/2011 07:55 PM, Daniel D. Daugherty wrote:
> Thumbs up.
if fc.close() throws an IOException , it goes wrong.
I think you need to put fc.close() in its own a try/catch(IOException).
> On 4/15/2011 11:01 AM, Mandy Chung wrote:
>> Fix for 7032589: FileHandler leaking file descriptor of the file lock
>> Webrev at:
>> When a log file is currently being used, tryLock() will fail and it
>> has to close the file channel of the lock file
More information about the core-libs-dev