diff options
author | Rui Xiang <rui.xiang@huawei.com> | 2013-11-12 18:06:54 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-11-12 22:09:00 -0500 |
commit | 7391a294b861bf2c3b762dfdcf61b9c5f1bffa1f (patch) | |
tree | 4c34e441b90db49696fe8dffb9599753c2e26db3 /fs/ocfs2/xattr.c | |
parent | f0cb0f0bca233935ac70707d47f5885419a7fd31 (diff) |
ocfs2: return ENOMEM when sb_getblk() fails
The only reason for sb_getblk() failing is if it can't allocate the
buffer_head. So return ENOMEM instead when it fails.
[joseph.qi@huawei.com: ocfs2_symlink_get_block() and ocfs2_read_blocks_sync() and ocfs2_read_blocks() need the same change]
Signed-off-by: Rui Xiang <rui.xiang@huawei.com>
Reviewed-by: Jie Liu <jeff.liu@oracle.com>
Reviewed-by: Mark Fasheh <mfasheh@suse.de>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Joseph Qi <joseph.qi@huawei.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/ocfs2/xattr.c')
-rw-r--r-- | fs/ocfs2/xattr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ocfs2/xattr.c b/fs/ocfs2/xattr.c index aaaab8ba82ab..82257299d5ea 100644 --- a/fs/ocfs2/xattr.c +++ b/fs/ocfs2/xattr.c | |||
@@ -377,7 +377,7 @@ static int ocfs2_init_xattr_bucket(struct ocfs2_xattr_bucket *bucket, | |||
377 | bucket->bu_bhs[i] = sb_getblk(bucket->bu_inode->i_sb, | 377 | bucket->bu_bhs[i] = sb_getblk(bucket->bu_inode->i_sb, |
378 | xb_blkno + i); | 378 | xb_blkno + i); |
379 | if (!bucket->bu_bhs[i]) { | 379 | if (!bucket->bu_bhs[i]) { |
380 | rc = -EIO; | 380 | rc = -ENOMEM; |
381 | mlog_errno(rc); | 381 | mlog_errno(rc); |
382 | break; | 382 | break; |
383 | } | 383 | } |