aboutsummaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorDavid Sterba <dsterba@suse.cz>2011-06-13 11:18:23 -0400
committerChris Mason <chris.mason@oracle.com>2011-06-17 14:54:18 -0400
commit35a30d7ce54e087d8025a725d4e5a2fdee723a9f (patch)
treeba7ab5023cde23bb0d7e0ea2e23400db8de80668 /fs
parent19fd294957e426bfdd8e19085096467ec18df5c4 (diff)
btrfs: fix uninitialized return value
When allocation fails in btrfs_read_fs_root_no_name, ret is not set although it is returned, holding a garbage value. Signed-off-by: David Sterba <dsterba@suse.cz> Reviewed-by: Li Zefan <lizf@cn.fujitsu.com> Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs')
-rw-r--r--fs/btrfs/disk-io.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c
index c25ef5a0ccd..1ac8db5dc0a 100644
--- a/fs/btrfs/disk-io.c
+++ b/fs/btrfs/disk-io.c
@@ -1299,12 +1299,12 @@ again:
1299 return root; 1299 return root;
1300 1300
1301 root->free_ino_ctl = kzalloc(sizeof(*root->free_ino_ctl), GFP_NOFS); 1301 root->free_ino_ctl = kzalloc(sizeof(*root->free_ino_ctl), GFP_NOFS);
1302 if (!root->free_ino_ctl)
1303 goto fail;
1304 root->free_ino_pinned = kzalloc(sizeof(*root->free_ino_pinned), 1302 root->free_ino_pinned = kzalloc(sizeof(*root->free_ino_pinned),
1305 GFP_NOFS); 1303 GFP_NOFS);
1306 if (!root->free_ino_pinned) 1304 if (!root->free_ino_pinned || !root->free_ino_ctl) {
1305 ret = -ENOMEM;
1307 goto fail; 1306 goto fail;
1307 }
1308 1308
1309 btrfs_init_free_ino_ctl(root); 1309 btrfs_init_free_ino_ctl(root);
1310 mutex_init(&root->fs_commit_mutex); 1310 mutex_init(&root->fs_commit_mutex);