Files
linux/fs
Jeff Layton cd7bc18e09 cifs: NULL out tcon, pSesInfo, and srvTcp pointers when chasing DFS referrals
commit a2934c7b36 upstream.

The scenario is this:

The kernel gets EREMOTE and starts chasing a DFS referral at mount time.
The tcon reference is put, which puts the session reference too, but
neither pointer is zeroed out.

The mount gets retried (goto try_mount_again) with new mount info.
Session setup fails fails and rc ends up being non-zero. The code then
falls through to the end and tries to put the previously freed tcon
pointer again.  Oops at: cifs_put_smb_ses+0x14/0xd0

Fix this by moving the initialization of the rc variable and the tcon,
pSesInfo and srvTcp pointers below the try_mount_again label. Also, add
a FreeXid() before the goto to prevent xid "leaks".

Signed-off-by: Jeff Layton <jlayton@redhat.com>
Reported-by: Gustavo Carvalho Homem <gustavo@angulosolido.pt>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2010-01-06 15:03:53 -08:00
..
2009-06-11 21:36:14 -04:00
2009-07-12 12:22:34 -07:00
2009-10-04 15:05:10 -07:00
2009-09-30 12:19:44 -05:00
2009-06-17 00:36:36 -04:00
2009-07-12 12:22:34 -07:00
2009-12-18 14:04:08 -08:00
2009-07-12 12:22:34 -07:00
2009-09-23 18:13:10 -07:00
2009-12-18 14:04:09 -08:00
2009-10-20 18:02:33 -04:00
2009-09-23 07:39:30 -07:00
2009-09-24 08:41:47 -04:00
2009-06-17 00:36:37 -04:00
2009-06-18 13:03:46 -07:00
2009-09-24 08:41:47 -04:00
2009-09-24 07:21:01 -07:00
2009-09-24 07:21:03 -07:00
2009-06-18 13:03:41 -07:00
2009-09-24 07:47:27 -04:00
2009-04-07 08:31:16 -07:00
2009-10-22 08:11:44 +09:00
2009-10-04 15:05:10 -07:00
2009-04-20 23:02:52 -04:00
2009-09-23 07:39:29 -07:00