Re: [Xen-devel] [PATCH 08 of 27 v2] libxl: do not read f.e. xenstore dir in disk list function

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2011-10-17 at 15:20 +0100, Ian Jackson wrote:
> Ian Campbell writes ("[Xen-devel] [PATCH 08 of 27 v2] libxl: do not read f.e. 
> xenstore dir in disk list function"):
> > libxl: do not read f.e. xenstore dir in disk list function
> > 
> > Instead store a duplicate of the "device-type" node in the backend dir
> > and use that instead.
> > 
> > This maintains the invariant that the list function is always "safe".
> > 
> > XXX I'm not sure this is an actual issue. The entries in the frontend device
> > XXX area are writeable by the guest but maybe the solution is to
> > restrict that ?
> 
> No, I don't think so.  The frontend directory is precisely where the
> frontend writes things that it wants the backend to be able to read
> but the backend shouldn't rely on it for its own operation.

So you agree with the patch and not with my vacillating WRT whether it
is necessary?

Ian.


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