diff options
author | Jeff Layton <jlayton@redhat.com> | 2011-07-05 17:37:37 -0400 |
---|---|---|
committer | Steve French <sfrench@us.ibm.com> | 2011-07-06 16:03:04 -0400 |
commit | 677d8537d875832019fa989186f084ba47ecd93d (patch) | |
tree | 28e5931d4c480c8a90e722cb0a0e6f1bef46cea9 /fs | |
parent | de3796e77a587b28f0c9d8e04c02894d3939dc4e (diff) |
cifs: remove bogus call to cifs_cleanup_volume_info
This call to cifs_cleanup_volume_info is clearly wrong. As soon as it's
called the following call to cifs_get_tcp_session will oops as the
volume_info pointer will then be NULL.
The caller of cifs_mount should clean up this data since it passed it
in. There's no need for us to call this here.
Regression introduced by commit 724d9f1cfba.
Reported-by: Adam Williamson <awilliam@redhat.com>
Cc: Pavel Shilovsky <piastryyy@gmail.com>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/cifs/connect.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c index c8cb83ef6f6f..545e85465747 100644 --- a/fs/cifs/connect.c +++ b/fs/cifs/connect.c | |||
@@ -3012,7 +3012,6 @@ try_mount_again: | |||
3012 | else if (pSesInfo) | 3012 | else if (pSesInfo) |
3013 | cifs_put_smb_ses(pSesInfo); | 3013 | cifs_put_smb_ses(pSesInfo); |
3014 | 3014 | ||
3015 | cifs_cleanup_volume_info(&volume_info); | ||
3016 | FreeXid(xid); | 3015 | FreeXid(xid); |
3017 | } | 3016 | } |
3018 | #endif | 3017 | #endif |