cifs: remove bogus call to cifs_cleanup_volume_info
authorJeff Layton <jlayton@redhat.com>
Tue, 5 Jul 2011 21:37:37 +0000 (17:37 -0400)
committerSteve French <sfrench@us.ibm.com>
Wed, 6 Jul 2011 20:03:04 +0000 (20:03 +0000)
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>

fs/cifs/connect.c

index c8cb83e..545e854 100644 (file)
@@ -3012,7 +3012,6 @@ try_mount_again:
                else if (pSesInfo)
                        cifs_put_smb_ses(pSesInfo);
 
-               cifs_cleanup_volume_info(&volume_info);
                FreeXid(xid);
        }
 #endif