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

[Xen-devel] Re: [PATCH] fix ioemu build


  • To: xen-devel@xxxxxxxxxxxxxxxxxxx
  • From: Christian Tramnitz <chris.ace@xxxxxxx>
  • Date: Fri, 07 Aug 2009 00:44:48 +0200
  • Delivery-date: Thu, 06 Aug 2009 15:45:33 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

Keir Fraser wrote:
Anyway checking for the existence of a file before actually deleting it
shouldn't really hurt...

But... 'rm -f' doesn't fail when the file doesn't exist?

It doesn't, but under circumstances this is escaping the sandbox and generating an access violation. This was initially reported for 3.3.1 as gentoo bug and fixed with this patch. Now I'm trying to get the live ebuilds working but patching is not an option there...

Details are available here:
http://bugs.gentoo.org/262124

Not really sure if this only manifests itself in the gentoo build environment, but since other places in xen-setup check for the existence of the file before doing sth with it, I don't think it would hurt to apply.


Btw, the path in my patch was wrong, I guess it should be qemu-xen-unstable instead of ioemu-dir.


Best regards,
  Christian


_______________________________________________
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®.