diff options
author | Eric Van Hensbergen <ericvh@opteron.homeip.net> | 2008-02-06 20:25:05 -0500 |
---|---|---|
committer | Eric Van Hensbergen <ericvh@opteron.homeip.net> | 2008-02-06 20:25:05 -0500 |
commit | 14b8869ff4f00f105ea76678997b4d49d27baae3 (patch) | |
tree | 201c06c106053c609f847531a482d73e41a8cfc9 /fs | |
parent | d199d652c513a010145a5a0d4e64c96361b97bee (diff) |
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>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/9p/vfs_file.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c index ba4b1caa9c43..a616fff8906d 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 = { | |||
184 | .open = v9fs_file_open, | 184 | .open = v9fs_file_open, |
185 | .release = v9fs_dir_release, | 185 | .release = v9fs_dir_release, |
186 | .lock = v9fs_file_lock, | 186 | .lock = v9fs_file_lock, |
187 | .mmap = generic_file_mmap, | 187 | .mmap = generic_file_readonly_mmap, |
188 | }; | 188 | }; |
189 | 189 | ||
190 | const struct file_operations v9fs_file_operations = { | 190 | const struct file_operations v9fs_file_operations = { |
@@ -194,5 +194,5 @@ const struct file_operations v9fs_file_operations = { | |||
194 | .open = v9fs_file_open, | 194 | .open = v9fs_file_open, |
195 | .release = v9fs_dir_release, | 195 | .release = v9fs_dir_release, |
196 | .lock = v9fs_file_lock, | 196 | .lock = v9fs_file_lock, |
197 | .mmap = generic_file_mmap, | 197 | .mmap = generic_file_readonly_mmap, |
198 | }; | 198 | }; |