summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Van Hensbergen <ericvh@opteron.homeip.net>2008-02-06 19:25:05 -0600
committerEric Van Hensbergen <ericvh@opteron.homeip.net>2008-02-06 19:25:05 -0600
commit14b8869ff4f00f105ea76678997b4d49d27baae3 (patch)
tree201c06c106053c609f847531a482d73e41a8cfc9
parentd199d652c513a010145a5a0d4e64c96361b97bee (diff)
downloadop-kernel-dev-14b8869ff4f00f105ea76678997b4d49d27baae3.zip
op-kernel-dev-14b8869ff4f00f105ea76678997b4d49d27baae3.tar.gz
9p: fix mmap to be read-only
v9fs was allowing writable mmap which could lead to kernel BUG() cases. This sets the mmap function to generic_file_readonly_mmap which (correctly) returns an error to applications which open mmap for writing. Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
-rw-r--r--fs/9p/vfs_file.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c
index ba4b1ca..a616fff 100644
--- a/fs/9p/vfs_file.c
+++ b/fs/9p/vfs_file.c
@@ -184,7 +184,7 @@ static const struct file_operations v9fs_cached_file_operations = {
.open = v9fs_file_open,
.release = v9fs_dir_release,
.lock = v9fs_file_lock,
- .mmap = generic_file_mmap,
+ .mmap = generic_file_readonly_mmap,
};
const struct file_operations v9fs_file_operations = {
@@ -194,5 +194,5 @@ const struct file_operations v9fs_file_operations = {
.open = v9fs_file_open,
.release = v9fs_dir_release,
.lock = v9fs_file_lock,
- .mmap = generic_file_mmap,
+ .mmap = generic_file_readonly_mmap,
};
OpenPOWER on IntegriCloud