Skip to content

Commit

Permalink
CIFS: Fix memory leak in cifs_do_mount
Browse files Browse the repository at this point in the history
and simplify error handling code.

Signed-off-by: Pavel Shilovsky <piastry@etersoft.ru>
Signed-off-by: Steve French <sfrench@us.ibm.com>
  • Loading branch information
Pavel Shilovsky authored and Steve French committed May 27, 2011
1 parent 6848b73 commit 641a58d
Showing 1 changed file with 10 additions and 9 deletions.
19 changes: 10 additions & 9 deletions fs/cifs/cifsfs.c
Original file line number Diff line number Diff line change
Expand Up @@ -556,9 +556,8 @@ cifs_do_mount(struct file_system_type *fs_type,

sb = sget(fs_type, NULL, set_anon_super, NULL);
if (IS_ERR(sb)) {
kfree(cifs_sb);
root = ERR_CAST(sb);
goto out;
goto out_cifs_sb;
}

/*
Expand All @@ -569,7 +568,7 @@ cifs_do_mount(struct file_system_type *fs_type,
cifs_sb->mountdata = kstrndup(data, PAGE_SIZE, GFP_KERNEL);
if (cifs_sb->mountdata == NULL) {
root = ERR_PTR(-ENOMEM);
goto err_out;
goto out_super;
}

sb->s_flags = flags;
Expand All @@ -581,21 +580,23 @@ cifs_do_mount(struct file_system_type *fs_type,
flags & MS_SILENT ? 1 : 0);
if (rc) {
root = ERR_PTR(rc);
goto err_out;
goto out_super;
}

sb->s_flags |= MS_ACTIVE;

root = dget(sb->s_root);
out:
cifs_cleanup_volume_info(&volume_info);
return root;
goto out;

err_out:
out_super:
kfree(cifs_sb->mountdata);
deactivate_locked_super(sb);

out_cifs_sb:
unload_nls(cifs_sb->local_nls);
kfree(cifs_sb);
deactivate_locked_super(sb);

out:
cifs_cleanup_volume_info(&volume_info);
return root;
}
Expand Down

0 comments on commit 641a58d

Please sign in to comment.