aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2016-12-10 09:56:01 -0500
committerTheodore Ts'o <tytso@mit.edu>2016-12-10 09:56:01 -0500
commit578620f451f836389424833f1454eeeb2ffc9e9f (patch)
treeef437c0575055652c7069b2874c655c569b1e572
parent7e6e1ef48fc02f3ac5d0edecbb0c6087cd758d58 (diff)
ext4: return -ENOMEM instead of success
We should set the error code if kzalloc() fails. Fixes: 67cf5b09a46f ("ext4: add the basic function for inline data support") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Theodore Ts'o <tytso@mit.edu> Cc: stable@vger.kernel.org
-rw-r--r--fs/ext4/inline.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/ext4/inline.c b/fs/ext4/inline.c
index 9b67f75bdcf7..437df6a1a841 100644
--- a/fs/ext4/inline.c
+++ b/fs/ext4/inline.c
@@ -341,8 +341,10 @@ static int ext4_update_inline_data(handle_t *handle, struct inode *inode,
341 341
342 len -= EXT4_MIN_INLINE_DATA_SIZE; 342 len -= EXT4_MIN_INLINE_DATA_SIZE;
343 value = kzalloc(len, GFP_NOFS); 343 value = kzalloc(len, GFP_NOFS);
344 if (!value) 344 if (!value) {
345 error = -ENOMEM;
345 goto out; 346 goto out;
347 }
346 348
347 error = ext4_xattr_ibody_get(inode, i.name_index, i.name, 349 error = ext4_xattr_ibody_get(inode, i.name_index, i.name,
348 value, len); 350 value, len);