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

[Top] [All Lists]
<prev [Date] next>
[Advanced]
 <prev [Thread] next>

[Xen-devel] Re: [PATCH, v2]: xl: Check domain existance when doing domai

from [Ian Jackson] [Permanent Link][Original]
To: Gianni Tedesco <gianni.tedesco@xxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH, v2]: xl: Check domain existance when doing domain identifier lookups
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Tue, 25 Jan 2011 17:29:21 +0000
Cc: Christoph Egger <Christoph.Egger@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxxxxx>
Delivery-date: Tue, 25 Jan 2011 09:30:15 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1295895194.28333.34.camel@xxxxxxxxxxxxxxxxxxxxxx>
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>
References: <A24AE1FFE7AEC5489F83450EE98351BF2BF2EC4C9D@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <1295888759.28333.14.camel@xxxxxxxxxxxxxxxxxxxxxx> <201101241827.46797.Christoph.Egger@xxxxxxx> <1295891031.28333.25.camel@xxxxxxxxxxxxxxxxxxxxxx> <1295893129.28333.31.camel@xxxxxxxxxxxxxxxxxxxxxx> <alpine.DEB.2.00.1101241838500.7277@kaball-desktop> <1295895194.28333.34.camel@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx 
Gianni Tedesco writes ("Re: [PATCH, v2]: xl: Check domain existance when doing 
domain identifier lookups"):
> Only if was_name is true... If they pass a number then we need to check
> explicitly. Perhaps it would be better to put that check in an else {}
> statement though.

I think it's fine to get a whole bunch of errors rather than just one,
if you say "xl something <numeric domain>" and the domain doesn't
exist.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
[More with this subject...]
<Prev in Thread] Current Thread [Next in Thread>
Previous by Date:  [Xen-devel] Re: [PATCH, v2]: xl: Check domain existance when doing domain identifier lookups, Ian Jackson
Next by Date:  [Xen-devel] Re: [PATCH, v2]: xl: Check domain existance when doing domain identifier lookups, Gianni Tedesco
Previous by Thread:  [Xen-devel] Re: [PATCH, v2]: xl: Check domain existance when doing domain identifier lookups, Stefano Stabellini
Next by Thread:  [Xen-devel] Re: [PATCH, v2]: xl: Check domain existance when doing domain identifier lookups, Ian Jackson
Indexes:  [Date] [Thread] [Top] [All Lists]
 
   

Copyright , Citrix Systems Inc. All rights reserved. Legal and Privacy