cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "daan Hoogland" <daan.hoogl...@gmail.com>
Subject Re: Review Request 15667: Fix for Coverity issues CID_1116744, CID_1116718 and CID_1116682, all related to resource leak.
Date Wed, 27 Nov 2013 13:21:50 GMT

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

Ship it!


226b193488638dfeff7a003b8c1ca6c760e00cd1

- daan Hoogland


On Nov. 27, 2013, 1:18 p.m., Wilder Rodrigues wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15667/
> -----------------------------------------------------------
> 
> (Updated Nov. 27, 2013, 1:18 p.m.)
> 
> 
> Review request for cloudstack and Hugo Trippaers.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Fixing resource leak on Coverity issues CID_1116744, CID_1116718 and CID_1116682.
> 
> The resource leak problems were mostly related to InpuStreams instantiated and not closed
after used.
> 
> 
> Diffs
> -----
> 
>   framework/ipc/src/org/apache/cloudstack/framework/serializer/OnwireClassRegistry.java
177ae09 
>   server/src/com/cloud/api/doc/ApiXmlDocWriter.java a91530b 
> 
> Diff: https://reviews.apache.org/r/15667/diff/
> 
> 
> Testing
> -------
> 
> A full build was executed on top of the branch created for these changes. After committed
and patched, the a brand new branch was created from Master and patched with this patch. Everything
worked fine.
> 
> No new feature was added.
> 
> 
> Thanks,
> 
> Wilder Rodrigues
> 
>


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