[v2] Fix some Coverity found issues.

[libvirt] [PATCH v2 0/2] Fix some Coverity found issues.

John Ferlan posted 2 patches 5 years, 6 months ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20180924125635.5884-1-jferlan@redhat.com

src/lxc/lxc_native.c | 38 +++++++++++++++++++-------------------
1 file changed, 19 insertions(+), 19 deletions(-)

Expand all Fold all

[libvirt] [PATCH v2 0/2] Fix some Coverity found issues.

Posted by John Ferlan 5 years, 6 months ago

v1: https://www.redhat.com/archives/libvir-list/2018-September/msg01115.html

Already pushed patch 1 and 3 from v1, just reworked patch 2
(the lxc change) based on review and repost.

John Ferlan (2):
  lxc: Remove unnecessary error label
  lxc: Resolve memory leak

 src/lxc/lxc_native.c | 38 +++++++++++++++++++-------------------
 1 file changed, 19 insertions(+), 19 deletions(-)

-- 
2.17.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v2 0/2] Fix some Coverity found issues.

Posted by Erik Skultety 5 years, 6 months ago

On Mon, Sep 24, 2018 at 08:56:33AM -0400, John Ferlan wrote:
> v1: https://www.redhat.com/archives/libvir-list/2018-September/msg01115.html
>
> Already pushed patch 1 and 3 from v1, just reworked patch 2
> (the lxc change) based on review and repost.
>
> John Ferlan (2):
>   lxc: Remove unnecessary error label
>   lxc: Resolve memory leak
>
>  src/lxc/lxc_native.c | 38 +++++++++++++++++++-------------------
>  1 file changed, 19 insertions(+), 19 deletions(-)

Reviewed-by: Erik Skultety <eskultet@redhat.com>

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list