cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Wilder Rodrigues" <>
Subject Re: Review Request 15455: Fixing all Coverity bugs on file Upgrade2214to30 related to resource leak. Now the statements closed and nullified before a new assignment.
Date Mon, 25 Nov 2013 14:32:09 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Nov. 25, 2013, 2:32 p.m.)

Review request for cloudstack and Hugo Trippaers.


Not the right way to do that.

Repository: cloudstack-git


Coverity bug #1116754 Resource leak on an exceptional path.
The system resource will not be reclaimed and reused, reducing the future availability of
the resource.
In com.?cloud.?upgrade.?dao.?Upgrade2214to30.?setupPhysicalNetworks(java.?sql.?Connection):
Leak of a system resource on an exception path (probably error handling) (CWE-404)

Since the file contained the same implementation all over the place - in many different methods
- I already updated everything.


  engine/schema/src/com/cloud/upgrade/dao/ 48b83b4 



All tests passed during build and I also applied the patch to a different branch, based on
Master, and built the project: all passed.


Wilder Rodrigues

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