Commit 9ac9b847 authored by Dave Hansen's avatar Dave Hansen Committed by Al Viro
Browse files

[PATCH] r/o bind mounts: write counts for truncate() 



Acked-by: default avatarAl Viro <viro@ZenIV.linux.org.uk>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Signed-off-by: default avatarDave Hansen <haveblue@us.ibm.com>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 2af482a7

fs/open.c

+8 −6
Original line number Diff line number Diff line
@@ -244,21 +244,21 @@ static long do_sys_truncate(const char __user * path, loff_t length) 
	if (!S_ISREG(inode->i_mode))

		goto dput_and_out;



	error = vfs_permission(&nd, MAY_WRITE);

	error = mnt_want_write(nd.path.mnt);

	if (error)

		goto dput_and_out;



	error = -EROFS;

	if (IS_RDONLY(inode))

		goto dput_and_out;

	error = vfs_permission(&nd, MAY_WRITE);

	if (error)

		goto mnt_drop_write_and_out;



	error = -EPERM;

	if (IS_IMMUTABLE(inode) || IS_APPEND(inode))

		goto dput_and_out;

		goto mnt_drop_write_and_out;



	error = get_write_access(inode);

	if (error)

		goto dput_and_out;

		goto mnt_drop_write_and_out;



	/*

	 * Make sure that there are no leases.  get_write_access() protects
@@ -276,6 +276,8 @@ static long do_sys_truncate(const char __user * path, loff_t length) 


put_write_and_out:

	put_write_access(inode);

mnt_drop_write_and_out:

	mnt_drop_write(nd.path.mnt);

dput_and_out:

	path_put(&nd.path);

out: