[PATCH] fix bogus argument of blkdev_put() in pktcdvd
authorAl Viro <viro@zeniv.linux.org.uk>
Sun, 30 Nov 2008 06:33:57 +0000 (01:33 -0500)
committerAl Viro <viro@zeniv.linux.org.uk>
Thu, 4 Dec 2008 09:22:59 +0000 (04:22 -0500)
final close of ->bdev should match the initial open, i.e.
get FMODE_READ | FMODE_NDELAY; FMODE_READ|FMODE_WRITE has
been a braino.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
drivers/block/pktcdvd.c

index f20bf359b84f20e7ff10f7081f54b37371e12f18..edda7b6b077b290695460e9e2ee92a0476fc197a 100644 (file)
@@ -2790,7 +2790,7 @@ static int pkt_new_dev(struct pktcdvd_device *pd, dev_t dev)
        return 0;
 
 out_mem:
-       blkdev_put(bdev, FMODE_READ|FMODE_WRITE);
+       blkdev_put(bdev, FMODE_READ | FMODE_NDELAY);
        /* This is safe: open() is still holding a reference. */
        module_put(THIS_MODULE);
        return ret;
@@ -2975,7 +2975,7 @@ static int pkt_remove_dev(dev_t pkt_dev)
        pkt_debugfs_dev_remove(pd);
        pkt_sysfs_dev_remove(pd);
 
-       blkdev_put(pd->bdev, FMODE_READ|FMODE_WRITE);
+       blkdev_put(pd->bdev, FMODE_READ | FMODE_NDELAY);
 
        remove_proc_entry(pd->name, pkt_proc);
        DPRINTK(DRIVER_NAME": writer %s unmapped\n", pd->name);