diff options
| author | Namhyung Kim <namhyung@gmail.com> | 2010-10-13 17:17:18 +0900 |
|---|---|---|
| committer | Jan Kara <jack@suse.cz> | 2010-10-28 01:30:04 +0200 |
| commit | 2a0e33889b3f5e3c270dd1b746f4c0a75efa4790 (patch) | |
| tree | 322394e75daccaf692a4adb388efa596a231a59b | |
| parent | 81a4e320e6ee29bde3fe880ab87b2122bc1de88b (diff) | |
| download | op-kernel-dev-2a0e33889b3f5e3c270dd1b746f4c0a75efa4790.zip op-kernel-dev-2a0e33889b3f5e3c270dd1b746f4c0a75efa4790.tar.gz | |
jbd: Check return value of __getblk()
Fail journal creation if __getblk() returns NULL. unlikely() is
added because it is called in a loop and we've been OK without
the check until now.
Signed-off-by: Namhyung Kim <namhyung@gmail.com>
Signed-off-by: Jan Kara <jack@suse.cz>
| -rw-r--r-- | fs/jbd/journal.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/jbd/journal.c b/fs/jbd/journal.c index e561176..ac18404 100644 --- a/fs/jbd/journal.c +++ b/fs/jbd/journal.c @@ -952,6 +952,8 @@ int journal_create(journal_t *journal) if (err) return err; bh = __getblk(journal->j_dev, blocknr, journal->j_blocksize); + if (unlikely(!bh)) + return -ENOMEM; lock_buffer(bh); memset (bh->b_data, 0, journal->j_blocksize); BUFFER_TRACE(bh, "marking dirty"); |
