From 1d816219eb04c045ed5e08f0707f470997875aeb Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Wed, 13 Feb 2013 07:49:29 -0500 Subject: [PATCH] virsh: Resolve possible NULL dereference Coverity found that commit '41b5e845' had a possible NULL dereference on arg->data processing --- tools/virsh.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/virsh.c b/tools/virsh.c index 4f75e8ef94..f5a01b3d5b 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@ -1,7 +1,7 @@ /* * virsh.c: a shell to exercise the libvirt API * - * Copyright (C) 2005, 2007-2012 Red Hat, Inc. + * Copyright (C) 2005, 2007-2013 Red Hat, Inc. * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -1475,7 +1475,7 @@ vshCommandOptStringReq(vshControl *ctl, if (!arg->data) error = N_("Programming error: Requested option is a boolean"); - if (!*arg->data && !(arg->def->flags & VSH_OFLAG_EMPTY_OK)) + if (arg->data && !*arg->data && !(arg->def->flags & VSH_OFLAG_EMPTY_OK)) error = N_("Option argument is empty"); if (error) { -- GitLab