----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22859/#review46382 -----------------------------------------------------------
Ship it! c690320f0b10b2465dfcb06581f62299a2fa641c on 4.4-forward - daan Hoogland On June 23, 2014, 6:13 a.m., Santhosh Edukulla wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22859/ > ----------------------------------------------------------- > > (Updated June 23, 2014, 6:13 a.m.) > > > Review request for cloudstack and daan Hoogland. > > > Bugs: coverity > https://issues.apache.org/jira/browse/coverity > > > Repository: cloudstack-git > > > Description > ------- > > fixed resource leaks reported by coverity > > > Diffs > ----- > > engine/schema/src/com/cloud/upgrade/dao/DatabaseAccessObject.java 836a537 > framework/db/src/com/cloud/utils/crypt/EncryptionSecretKeyChanger.java > 4cee081 > framework/db/src/com/cloud/utils/db/ScriptRunner.java 932b37b > > framework/jobs/src/org/apache/cloudstack/framework/jobs/dao/AsyncJobJoinMapDaoImpl.java > 7bc29db > > Diff: https://reviews.apache.org/r/22859/diff/ > > > Testing > ------- > > Post the changes, built the code and ran a deploy Datacenter. > > > Thanks, > > Santhosh Edukulla > >
