Maven 1

org.apache.maven.MavenUtils.loadProperties(..) does not close the FileInputStream (relying upon finalization)

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.1-beta-2
  • Fix Version/s: 1.1-beta-3
  • Component/s: core
  • Labels:
    None
  • Number of attachments :
    2

Description

org.apache.maven.MavenUtils.loadProperties(..) does not close the FileInputStream (relying upon finalization)

See attached patch.

Issue Links

Activity

Hide
John Sisson added a comment -

Also have patch for JellyScriptHousing that ensures close() is called in a finally block.

Show
John Sisson added a comment - Also have patch for JellyScriptHousing that ensures close() is called in a finally block.
Hide
John Sisson added a comment -

Also in MavenUtils, the getNonJellyProject(..) method does not close the FileReader that is created.

Show
John Sisson added a comment - Also in MavenUtils, the getNonJellyProject(..) method does not close the FileReader that is created.
Hide
John Sisson added a comment -

Similar problems with the compileScript and runScript methods in JellyUtils.java where a FileInputStream is not closed.

Show
John Sisson added a comment - Similar problems with the compileScript and runScript methods in JellyUtils.java where a FileInputStream is not closed.
Hide
Arnaud Heritier added a comment -

I checked that we close all the InputStreams we open.

Show
Arnaud Heritier added a comment - I checked that we close all the InputStreams we open.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: