[libvirt] [PATCH v2 6/6] xenapi: Resolve Coverity FORWARD_NULL
Peter Krempa
pkrempa at redhat.com
Wed Oct 7 06:40:11 UTC 2015
Subject is technically right, but entirely misleading. This patch is
more a refactor of the code that fixes a coverity error rather than the
other way around.
On Fri, Sep 25, 2015 at 12:31:45 -0400, John Ferlan wrote:
> Coverity flagged that if the input 'mask'parameter was NULL, then the
> strtok_r call would have a NULL first and third parameter, which would
> cause an issue.
>
> Although the caller doesn't call getCpuBitMapfromString with a non-NULL
> mask value, Coverity doesn't check that. A fix could have been to check
So a false positive again?
> the 'mask' in the function and keep Coverity happy; however, it was noted
> in a previous review that virBitmap* functions could be used instead.
>
> So this patch modifies xenapiDomainGetVcpus to use virBitmap* functions
> and removes getCpuBitMapfromString in favor of those.
>
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
> src/xenapi/xenapi_driver.c | 12 +++++++-----
> src/xenapi/xenapi_utils.c | 21 ---------------------
> src/xenapi/xenapi_utils.h | 2 --
> 3 files changed, 7 insertions(+), 28 deletions(-)
This makes me like this patch though.
>
ACK if you fix the commit message.
Peter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20151007/99fccb42/attachment-0001.sig>
More information about the libvir-list
mailing list