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 20:03:04 +0000 |
| commit | 677d8537d875832019fa989186f084ba47ecd93d (patch) | |
| tree | 28e5931d4c480c8a90e722cb0a0e6f1bef46cea9 /fs | |
| parent | de3796e77a587b28f0c9d8e04c02894d3939dc4e (diff) | |
| download | linux-677d8537d875832019fa989186f084ba47ecd93d.tar.bz2 | |
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: else if (pSesInfo) cifs_put_smb_ses(pSesInfo); - cifs_cleanup_volume_info(&volume_info); FreeXid(xid); } #endif |