diff options
| author | Theodore Ts'o <tytso@mit.edu> | 2013-01-12 22:19:36 +0100 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2013-01-12 22:19:36 +0100 |
| commit | 860d21e2c585f7ee8a4ecc06f474fdc33c9474f4 (patch) | |
| tree | f4b8f664599f043b7aa7b86a9a135aa275f0a5e2 /fs/ext4/xattr.c | |
| parent | Linux 3.8-rc3 (diff) | |
| download | linux-860d21e2c585f7ee8a4ecc06f474fdc33c9474f4.tar.xz linux-860d21e2c585f7ee8a4ecc06f474fdc33c9474f4.zip | |
ext4: return ENOMEM if sb_getblk() fails
The only reason for sb_getblk() failing is if it can't allocate the
buffer_head. So ENOMEM is more appropriate than EIO. In addition,
make sure that the file system is marked as being inconsistent if
sb_getblk() fails.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Cc: stable@vger.kernel.org
Diffstat (limited to '')
| -rw-r--r-- | fs/ext4/xattr.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/ext4/xattr.c b/fs/ext4/xattr.c index 3a91ebc2b66f..07d684a4e523 100644 --- a/fs/ext4/xattr.c +++ b/fs/ext4/xattr.c @@ -887,16 +887,17 @@ inserted: new_bh = sb_getblk(sb, block); if (!new_bh) { + error = -ENOMEM; getblk_failed: ext4_free_blocks(handle, inode, NULL, block, 1, EXT4_FREE_BLOCKS_METADATA); - error = -EIO; goto cleanup; } lock_buffer(new_bh); error = ext4_journal_get_create_access(handle, new_bh); if (error) { unlock_buffer(new_bh); + error = -EIO; goto getblk_failed; } memcpy(new_bh->b_data, s->base, new_bh->b_size); |