summaryrefslogtreecommitdiffstats
path: root/drivers/xen/privcmd.c
diff options
context:
space:
mode:
authorPaul Durrant <paul.durrant@citrix.com>2017-02-13 17:03:22 +0000
committerBoris Ostrovsky <boris.ostrovsky@oracle.com>2017-02-13 13:49:00 -0500
commitdc9eab6fd94dd26340749321bba2c58634761516 (patch)
treef8c05ab25635c57dc168172b4307f7ae0bab7fe9 /drivers/xen/privcmd.c
parentfd8aa9095a95c02dcc35540a263267c29b8fda9d (diff)
downloadop-kernel-dev-dc9eab6fd94dd26340749321bba2c58634761516.zip
op-kernel-dev-dc9eab6fd94dd26340749321bba2c58634761516.tar.gz
xen/privcmd: return -ENOTTY for unimplemented IOCTLs
The code sets the default return code to -ENOSYS but then overrides this to -EINVAL in the switch() statement's default case, which is clearly silly. This patch removes the override and sets the default return code to -ENOTTY, which is the conventional return for an unimplemented ioctl. Signed-off-by: Paul Durrant <paul.durrant@citrix.com> Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Diffstat (limited to 'drivers/xen/privcmd.c')
-rw-r--r--drivers/xen/privcmd.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/xen/privcmd.c b/drivers/xen/privcmd.c
index 6e3306f..5e5c7ae 100644
--- a/drivers/xen/privcmd.c
+++ b/drivers/xen/privcmd.c
@@ -551,7 +551,7 @@ out_unlock:
static long privcmd_ioctl(struct file *file,
unsigned int cmd, unsigned long data)
{
- int ret = -ENOSYS;
+ int ret = -ENOTTY;
void __user *udata = (void __user *) data;
switch (cmd) {
@@ -572,7 +572,6 @@ static long privcmd_ioctl(struct file *file,
break;
default:
- ret = -EINVAL;
break;
}
OpenPOWER on IntegriCloud