xense-devel

[Top] [All Lists]

[Xen-devel] [PATCH] dom0_ops -EPERM not -EACCES?

from [Rusty Russell]

[Permanent Link][Original]

To:	xense-devel@xxxxxxxxxxxxxxxxxxx
Subject:	[Xen-devel] [PATCH] dom0_ops -EPERM not -EACCES?
From:	Rusty Russell <rusty@xxxxxxxxxxxxxxx>
Date:	Thu, 24 Nov 2005 12:35:42 +1100
Cc:	Xen Mailing List <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date:	Thu, 24 Nov 2005 01:35:34 +0000
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxxx
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

Hi all,

        dom0_ops return -EACCES when the interface version in wrong.  I think
that an acm failure is closer to being non-priv'ed, which returns
-EPERM.  Wouldn't want dom0 tools to report they were the wrong version
just because ACM limited some operation, surely?

Signed-off-by: Rusty Russell <rusty@xxxxxxxxxxxxxxx> (authored)

diff -r e2d1c88d50f7 xen/common/dom0_ops.c
--- a/xen/common/dom0_ops.c     Thu Nov 24 00:55:37 2005
+++ b/xen/common/dom0_ops.c     Thu Nov 24 12:31:37 2005
@@ -102,7 +102,7 @@
         return -EACCES;
 
     if ( acm_pre_dom0_op(op, &ssid) )
-        return -EACCES;
+        return -EPERM;
 
     spin_lock(&dom0_lock);
 

-- 
A bad analogy is like a leaky screwdriver -- Richard Braakman


_______________________________________________
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>
[Xen-devel] [PATCH] dom0_ops -EPERM not -EACCES?, Rusty Russell <=

Previous by Date:	Re: [Xense-devel] Labeling resources, Reiner Sailer
Previous by Thread:	Re: [Xense-devel] Labeling resources, David Palmer
Indexes:	[Date] [Thread] [Top] [All Lists]