[libvirt] [PATCH 2/4] virsh: avoid uninitialized variable

Eric Blake eblake at redhat.com
Thu Jun 30 14:14:54 UTC 2011


Detected by Coverity; neither vshCmddefHelp nor vshCmdOptParse
was initializing opts_required.

* tools/virsh.c (vshCmddefOptParse): Always initialize bitmaps.
---
 tools/virsh.c |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/tools/virsh.c b/tools/virsh.c
index 643f05c..fdb4283 100644
--- a/tools/virsh.c
+++ b/tools/virsh.c
@@ -11915,12 +11915,15 @@ vshCmddefGetInfo(const vshCmdDef * cmd, const char *name)
 }

 static int
-vshCmddefOptParse(const vshCmdDef *cmd, uint32_t* opts_need_arg,
+vshCmddefOptParse(const vshCmdDef *cmd, uint32_t *opts_need_arg,
                   uint32_t *opts_required)
 {
     int i;
     bool optional = false;

+    *opts_need_arg = 0;
+    *opts_required = 0;
+
     if (!cmd->opts)
         return 0;

-- 
1.7.4.4




More information about the libvir-list mailing list