WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] xenbackendd: clean variable in loop

To: Roger Pau Monné <roger.pau@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xenbackendd: clean variable in loop
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Thu, 14 Jul 2011 17:36:46 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 14 Jul 2011 09:37:31 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <CAPLaKK5wnGZdicsHE6YfvqneRrNX7rr6GzuOBf5P+eYLvCznAw@xxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Newsgroups: chiark.mail.xen.devel
References: <CAPLaKK5wnGZdicsHE6YfvqneRrNX7rr6GzuOBf5P+eYLvCznAw@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Roger Pau Monné writes ("[Xen-devel] [PATCH] xenbackendd: clean variable in 
loop"):
> Fixed a cleanup of a variable used in a loop that was giving error
> messages, and added a debug message.

Thanks, but I'm not entirely clear about the bug you think this is
fixing.

Looking at the existing code it seems like the problem is that if the
loop is repeated and the 2nd device isn't a vbd or a vif, the previous
type is erroneously reused.

I think this woudl be better fixed by moving the initialisation of
type from the top of the function to just before it might first be
set.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel