cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Wilder Rodrigues" <wrodrig...@schubergphilis.com>
Subject Review Request 15508: Make sure that if the file does not exist an Exception is thrown and that once it exists it is also closed after the properties are loaded.
Date Thu, 14 Nov 2013 08:03:04 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15508/
-----------------------------------------------------------

Review request for cloudstack and Hugo Trippaers.


Repository: cloudstack-git


Description
-------

Make sure that if the file does not exist an Exception is thrown and that once it exists it
is also closed after the properties are loaded.

fix for Coverity bug cv_1125364 Resource leak
The system resource will not be reclaimed and reused, reducing the future availability of
the resource.
In org.?apache.?cloudstack.?network.?contrail.?management.?ManagementNetworkGuru.?configure(java.?lang.?String,
java.?util.?Map): Leak of a system resource (CWE-404)


Diffs
-----

  plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ManagementNetworkGuru.java
e457023 

Diff: https://reviews.apache.org/r/15508/diff/


Testing
-------

A test branch was created, the patch was applied against the branch and a full build was executed.
Everything is working fine. The class changed is tested by MockLocalNfsSecondaryStorageResource.


Thanks,

Wilder Rodrigues


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message