summaryrefslogtreecommitdiffstats
path: root/fs/f2fs
diff options
context:
space:
mode:
authorP J P <ppandit@redhat.com>2013-04-03 11:38:00 +0900
committerJaegeuk Kim <jaegeuk.kim@samsung.com>2013-04-03 17:27:52 +0900
commitcfb185a1488810fbae9256c7d52f66c558c6ea04 (patch)
tree5d6f89c0d25fcbf4cfdf37edc1e59f7e43df8636 /fs/f2fs
parentb2f2c390c5612df97f0403e1ef1e4e41c24b7d4f (diff)
downloadop-kernel-dev-cfb185a1488810fbae9256c7d52f66c558c6ea04.zip
op-kernel-dev-cfb185a1488810fbae9256c7d52f66c558c6ea04.tar.gz
f2fs: add NULL pointer check
Commit - fa9150a84c - replaces a call to generic_writepages() in f2fs_write_data_pages() with write_cache_pages(), with a function pointer argument pointing to routine: __f2fs_writepage. -> https://git.kernel.org/linus/fa9150a84ca333f68127097c4fa1eda4b3913a22 This patch adds a NULL pointer check in f2fs_write_data_pages() to avoid a possible NULL pointer dereference, in case if - mapping->a_ops->writepage - is NULL. Signed-off-by: P J P <ppandit@redhat.com> Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs/f2fs')
-rw-r--r--fs/f2fs/data.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index 47a2d7c..cf9ff5f 100644
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -559,6 +559,10 @@ static int f2fs_write_data_pages(struct address_space *mapping,
int ret;
long excess_nrtw = 0, desired_nrtw;
+ /* deal with chardevs and other special file */
+ if (!mapping->a_ops->writepage)
+ return 0;
+
if (wbc->nr_to_write < MAX_DESIRED_PAGES_WP) {
desired_nrtw = MAX_DESIRED_PAGES_WP;
excess_nrtw = desired_nrtw - wbc->nr_to_write;
OpenPOWER on IntegriCloud