From: Serge E. Hallyn <serue@xxxxxxxxxx> If we want to support an admin clearing all entries, let's not do it through some subtle hidden channel, but rather implement a 'CLEAR' command to /dev/caphash, which requires privilege to use. Signed-off-by: Serge E. Hallyn <serue@xxxxxxxxxx> Cc: Greg KH <greg@xxxxxxxxx> cc: rsc@xxxxxxxxx Cc: Ashwin Ganti <ashwin.ganti@xxxxxxxxx> Cc: ericvh@xxxxxxxxx Cc: devel@xxxxxxxxxxxxxxxxxxxxxx Cc: linux-kernel@xxxxxxxxxxxxxxx Cc: Ron Minnich <rminnich@xxxxxxxxx> --- drivers/staging/p9auth/p9auth.c | 7 ------- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/drivers/staging/p9auth/p9auth.c b/drivers/staging/p9auth/p9auth.c index 6012bd9..06128c3 100644 --- a/drivers/staging/p9auth/p9auth.c +++ b/drivers/staging/p9auth/p9auth.c @@ -146,13 +146,6 @@ static int cap_open(struct inode *inode, struct file *filp) dev = container_of(inode->i_cdev, struct cap_dev, cdev); filp->private_data = dev; - /* trim to 0 the length of the device if open was write-only */ - if ((filp->f_flags & O_ACCMODE) == O_WRONLY) { - if (down_interruptible(&dev->sem)) - return -ERESTARTSYS; - cap_trim(dev); - up(&dev->sem); - } /* initialise the head if it is NULL */ if (dev->head == NULL) { dev->head = kmalloc(sizeof(struct cap_node), GFP_KERNEL); -- 1.6.1 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel