You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@cloudstack.apache.org by DaanHoogland <gi...@git.apache.org> on 2015/07/07 12:37:02 UTC

[GitHub] cloudstack pull request: coverity resource leaks

GitHub user DaanHoogland opened a pull request:

    https://github.com/apache/cloudstack/pull/564

    coverity resource leaks

    This PR holds fixes for different resource leaks of the type that can be fixed by using try-with resource. The complete category is to be absent from coverity reports.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/DaanHoogland/cloudstack coverity-resource-leaks

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/564.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #564
    
----
commit 2f1d8b4c5a415534e5f6100ab9871c9021a10a83
Author: Daan Hoogland <da...@onecht.net>
Date:   2015-07-07T10:34:10Z

    coverity 1308118: more resources being opened only one closed
     on exception nothing was closed

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: coverity resource leaks

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/cloudstack/pull/564


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: coverity resource leaks

Posted by karuturi <gi...@git.apache.org>.

Github user karuturi commented on the pull request:

    https://github.com/apache/cloudstack/pull/564#issuecomment-119546107
  
    changes looks good. :+1: 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: coverity resource leaks

Posted by wilderrodrigues <gi...@git.apache.org>.

Github user wilderrodrigues commented on the pull request:

    https://github.com/apache/cloudstack/pull/564#issuecomment-119917601
  
    LGTM :+1: 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---