merged On Tue, Jul 5, 2011 at 4:37 PM, Jeff Layton <jlayton@xxxxxxxxxx> wrote: > 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@xxxxxxxxxx> > Cc: Pavel Shilovsky <piastryyy@xxxxxxxxx> > Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx> > --- > 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 c8cb83e..545e854 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 > -- > 1.7.6 > > -- Thanks, Steve -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
