Re: [libvirt] [PATCH 17/26] virstring: Resolve Coverity FORWARD_NULL
On 09/05/14 00:26, John Ferlan wrote:
Perhaps a false positive, but since Coverity doesn't understand
the
relationship between the 'count' and the 'strings', rather than leave
the chance the on input 'strings' is NULL and causes a deref - just
check for it and return
Signed-off-by: John Ferlan <jferlan(a)redhat.com>
---
src/util/virstring.c | 3 +++
1 file changed, 3 insertions(+)
Yep, false positive. All callers shall pass 0 as count if strings is NULL.
ACK, doesn't hurt.
Peter
Attachments:
- signature.asc (application/pgp-signature — 819 bytes)