Fix a couple of Coverity found issues from recent patches

[libvirt] [PATCH 0/2] Fix a couple of Coverity found issues from recent patches

John Ferlan posted 2 patches 7 years, 1 month ago

Download series mbox

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

src/util/virsysinfo.c | 17 ++++++++++++-----
1 file changed, 12 insertions(+), 5 deletions(-)

Expand all Fold all

[libvirt] [PATCH 0/2] Fix a couple of Coverity found issues from recent patches

Posted by John Ferlan 7 years, 1 month ago

The virsysinfo changes caused Coverity to "take another look" at the code
and it found some issues. Each is described in the patch commit.

John Ferlan (2):
  util: Resource some resource leaks in virsysinfo code
  util: Avoid possible NULL dereference in virSysinfoParsePPCProcessor

 src/util/virsysinfo.c | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

-- 
2.9.3

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

Re: [libvirt] [PATCH 0/2] Fix a couple of Coverity found issues from recent patches

Posted by Peter Krempa 7 years, 1 month ago

On Tue, Mar 07, 2017 at 07:15:40 -0500, John Ferlan wrote:
> The virsysinfo changes caused Coverity to "take another look" at the code

I think it was more like a first look, since all of that code was in a
huge #if defined(__ppc__) and similar blocks.

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