[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] xl: Perform minimal validation of virtual disk file while parsing config file


  • To: Gianni Tedesco <gianni.tedesco@xxxxxxxxxx>
  • From: Kamala Narasimhan <kamala.narasimhan@xxxxxxxxx>
  • Date: Fri, 14 Jan 2011 12:17:01 -0500
  • Cc: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Fri, 14 Jan 2011 09:21:27 -0800
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=DJ277q/5Gmmzj4MLmgD3Y2OdkwKo27ZZuOmbWJi3Ft+X4+DWBzWpP6YcknfKtjgBc6 XER9i4tdgQYyIWS60dByDvXRON8I7A85FYsCtfomg2GdhEx0U5yUUlTCVH4UQcn2X630 gLLXLLWgvx9xQ1CFQJYDS2cB3l7c2pVl4szYw=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

> I think Ians point is that your change affects the 'xl' binary and not
> the libxl.so library.
>
Ah!  I mistook it then.

> Perhaps libxl_device_disk_add() and libxl_cdrom_insert() would be the
> reasonable places to add this, replacing fprintf() calls with libxl
> logging functions.
>
I will make necessary changes and resend the patch.

Kamala

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.