summaryrefslogtreecommitdiffstats
path: root/drivers/xen/gntdev.c
diff options
context:
space:
mode:
authorDan Carpenter <error27@gmail.com>2011-03-19 08:45:43 +0300
committerKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>2011-03-24 10:32:45 -0400
commita93e20a83077f57638f8574b1c072ce6712e95ec (patch)
tree1cb7c814949381ae44638ca9bd62c81432c94abf /drivers/xen/gntdev.c
parent12f0258d5b44b3b5a9442ec461bbac1f7edab8c6 (diff)
downloadop-kernel-dev-a93e20a83077f57638f8574b1c072ce6712e95ec.zip
op-kernel-dev-a93e20a83077f57638f8574b1c072ce6712e95ec.tar.gz
xen-gntdev: unlock on error path in gntdev_mmap()
We should unlock here and also decrement the number of &map->users. Signed-off-by: Dan Carpenter <error27@gmail.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Diffstat (limited to 'drivers/xen/gntdev.c')
-rw-r--r--drivers/xen/gntdev.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/xen/gntdev.c b/drivers/xen/gntdev.c
index 00b3acc..b0f9e8f 100644
--- a/drivers/xen/gntdev.c
+++ b/drivers/xen/gntdev.c
@@ -662,7 +662,7 @@ static int gntdev_mmap(struct file *flip, struct vm_area_struct *vma)
if (map->flags) {
if ((vma->vm_flags & VM_WRITE) &&
(map->flags & GNTMAP_readonly))
- return -EINVAL;
+ goto out_unlock_put;
} else {
map->flags = GNTMAP_host_map;
if (!(vma->vm_flags & VM_WRITE))
@@ -700,6 +700,8 @@ unlock_out:
spin_unlock(&priv->lock);
return err;
+out_unlock_put:
+ spin_unlock(&priv->lock);
out_put_map:
if (use_ptemod)
map->vma = NULL;
OpenPOWER on IntegriCloud