Re: [Xen-devel] [PATCH 1/2] xenstat: fix unsigned less-than-0 comparison

[Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>]

On Sun, May 04, 2014 at 08:31:46PM +1200, Matthew Daley wrote:
> Commit 1438d36f ("xenstat: Fix buffer over-run with new_domains being
> negative.") attempted to fix the handling of a negative error result
> from xc_domain_getinfolist in xenstat_get_node. However, it forgot to
> change the result variable from an unsigned type to a signed one.
> 
> Do so, allowing the error result to be handled properly.
> 
> Signed-off-by: Matthew Daley <mattd@xxxxxxxxxxx>

Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>

I was looking at this by changing the underlay libxc calls to
using 'unsigned' instead of 'int' but that has kind of been
preempted.

Thanks for fixing it!
> ---
>  tools/xenstat/libxenstat/src/xenstat.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/xenstat/libxenstat/src/xenstat.c 
> b/tools/xenstat/libxenstat/src/xenstat.c
> index e5facb8..8072a90 100644
> --- a/tools/xenstat/libxenstat/src/xenstat.c
> +++ b/tools/xenstat/libxenstat/src/xenstat.c
> @@ -164,7 +164,7 @@ xenstat_node *xenstat_get_node(xenstat_handle * handle, 
> unsigned int flags)
>       xenstat_node *node;
>       xc_physinfo_t physinfo = { 0 };
>       xc_domaininfo_t domaininfo[DOMAIN_CHUNK_SIZE];
> -     unsigned int new_domains;
> +     int new_domains;
>       unsigned int i;
>  
>       /* Create the node */
> -- 
> 1.9.2
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel