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

xen-devel

[Top] [All Lists]

Re: [Xen-devel] PATCH: Set close-on-exec flag for QEMU disks

To: "Daniel P. Berrange" <berrange@xxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] PATCH: Set close-on-exec flag for QEMU disks
From: Keir Fraser <keir@xxxxxxxxxxxxx>
Date: 2007年3月05日 13:25:23 +0000
Delivery-date: 2007年3月05日 05:24:41 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20070302214037.GC4598@xxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcdfKbp2+S9InMscEdu3zwAX8io7RQ==
Thread-topic: [Xen-devel] PATCH: Set close-on-exec flag for QEMU disks
User-agent: Microsoft-Entourage/11.2.5.060620
On 2/3/07 21:40, "Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote:
> QEMU does not currently set the close-on-exec flag after opening its virtual
> disk images. This causes problems when it later runs the /etc/xen/qemu-ifup
> script because the file descriptors get propagated to networking commands
> like brctl / ifconfig. The SELinux policy quite rightly does not allow the
> networking scripts to access the virtual disk images, so these inherited
> file descriptors for AVC denials to be logged.
>
> The attached patch modifies all the QEMU disk driver backends to make sure
> the close-on-exec flag is turned on
It would be nicer to implement an open_cloexec() function in e.g., vl.c to
do the open() and fcntl() in one go and in one place.
There are lots of uses of open() throughout the qemu sources and the patch
only fixes up a subset of them -- is this correct?
 -- Keir
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>
Previous by Date: Re: [Xen-devel] page fault handling in Xen , Tim Deegan
Next by Date: [Xen-devel] rebase 3.0.4-testing? , S.Çağlar Onur
Previous by Thread: [Xen-devel] PATCH: Set close-on-exec flag for QEMU disks , Daniel P. Berrange
Next by Thread: Re: [Xen-devel] PATCH: Set close-on-exec flag for QEMU disks , Anthony Liguori
Indexes: [Date] [Thread] [Top] [All Lists]

Copyright ©, Citrix Systems Inc. All rights reserved. Legal and Privacy
Citrix This site is hosted by Citrix

AltStyle によって変換されたページ (->オリジナル) /