summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2012-01-24 11:39:22 +0300
committerSteve French <smfrench@gmail.com>2012-01-24 10:37:19 -0600
commit803ab977618eae2b292cda0a97eed75f42250ddf (patch)
tree846b9d4086148ea91c440dde2b8ef5131ce6ecc2 /fs
parentc1aab02dac690af7ff634d8e1cb3be6a04387eef (diff)
downloadtalos-op-linux-803ab977618eae2b292cda0a97eed75f42250ddf.tar.gz
talos-op-linux-803ab977618eae2b292cda0a97eed75f42250ddf.zip
cifs: NULL dereference on allocation failure
We should just return directly here, the goto causes a NULL dereference. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Jeff Layton <jlayton@redhat.com> Signed-off-by: Steve French <smfrench@gmail.com>
Diffstat (limited to 'fs')
-rw-r--r--fs/cifs/connect.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
index 986709a8d903..026d6464335b 100644
--- a/fs/cifs/connect.c
+++ b/fs/cifs/connect.c
@@ -3857,10 +3857,8 @@ cifs_construct_tcon(struct cifs_sb_info *cifs_sb, uid_t fsuid)
struct smb_vol *vol_info;
vol_info = kzalloc(sizeof(*vol_info), GFP_KERNEL);
- if (vol_info == NULL) {
- tcon = ERR_PTR(-ENOMEM);
- goto out;
- }
+ if (vol_info == NULL)
+ return ERR_PTR(-ENOMEM);
vol_info->local_nls = cifs_sb->local_nls;
vol_info->linux_uid = fsuid;
OpenPOWER on IntegriCloud