[libvirt] [PATCH 1/3] virsh.c: Fix compiler warning

Doug Goldstein cardoe at gentoo.org
Mon Aug 13 15:09:02 UTC 2012


On Mon, Aug 13, 2012 at 9:28 AM, Cole Robinson <crobinso at redhat.com> wrote:
> For some reason I only get this after applying subsequent upcoming
> patches that touch virsh, but don't seem to actually cause the warning.
>
> virsh.c: In function ‘vshCommandParse’:
> virsh.c:2014:46: error: ‘opt_index’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
> cc1: all warnings being treated as errors
> ---
>  tools/virsh.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/virsh.c b/tools/virsh.c
> index b95a008..64e2e18 100644
> --- a/tools/virsh.c
> +++ b/tools/virsh.c
> @@ -1975,7 +1975,7 @@ vshCommandParse(vshControl *ctl, vshCommandParser *parser)
>              } else if (tkdata[0] == '-' && tkdata[1] == '-' &&
>                         c_isalnum(tkdata[2])) {
>                  char *optstr = strchr(tkdata + 2, '=');
> -                int opt_index;
> +                int opt_index = 0;
>
>                  if (optstr) {
>                      *optstr = '\0'; /* convert the '=' to '\0' */
> --
> 1.7.11.2
>

ACK. Looks good.

-- 
Doug Goldstein




More information about the libvir-list mailing list