<br><tt><font size=2>libvir-list-bounces@redhat.com wrote on 03/29/2010
11:35:31 AM:<br>
<br>
</font></tt>
<br><tt><font size=2>> <br>
> I've just run the code through clang, and it spotted a few new problems.<br>
> Here's one:<br>
> <br>
> >From 4c848787c8268e37db9a1d0754b8fcd8e5774f73 Mon Sep 17 00:00:00
2001<br>
> From: Jim Meyering <meyering@redhat.com><br>
> Date: Mon, 29 Mar 2010 17:34:32 +0200<br>
> Subject: [PATCH] nwfilter_conf.c: don't test an uninitialized local
variable<br>
> <br>
> * src/conf/nwfilter_conf.c (virNWIPAddressFormat): Do not use "i"<br>
> uninitialized.<br>
> ---<br>
>  src/conf/nwfilter_conf.c |    2 +-<br>
>  1 files changed, 1 insertions(+), 1 deletions(-)<br>
> <br>
> diff --git a/src/conf/nwfilter_conf.c b/src/conf/nwfilter_conf.c<br>
> index e82aae6..f673e69 100644<br>
> --- a/src/conf/nwfilter_conf.c<br>
> +++ b/src/conf/nwfilter_conf.c<br>
> @@ -2423,7 +2423,7 @@ virNWIPAddressFormat(virBufferPtr buf, <br>
> nwIPAddressPtr ipaddr)<br>
>                    
       ipaddr->addr.ipv4Addr[2],<br>
>                    
       ipaddr->addr.ipv4Addr[3]);<br>
>      } else {<br>
> -        int i;<br>
> +        unsigned int i = 0;<br>
>          int dcshown = 0, in_dc = 0;<br>
>          unsigned short n;<br>
>          while (i < 8) {<br>
> --<br>
> 1.7.0.3.448.g82eeb</font></tt>
<br>
<br>
<br><tt><font size=2>Thanks so much for looking at this. I do have a patch
following Daniel V.'s suggestion to get rid of the home-cooked IPv6 parser/print
function and it will do away with that code section.</font></tt>
<br>
<br><tt><font size=2> Regards,</font></tt>
<br><tt><font size=2>    Stefan</font></tt>
<br>
<br><tt><font size=2><br>
> <br>
> --<br>
> libvir-list mailing list<br>
> libvir-list@redhat.com<br>
> </font></tt><a href="https://www.redhat.com/mailman/listinfo/libvir-list"><tt><font size=2>https://www.redhat.com/mailman/listinfo/libvir-list</font></tt></a><tt><font size=2><br>
</font></tt>