Attachment 283151 Details for Bug 417961 – patch 6 -- fix oops on second mount to same server when null auth is used

[patch] patch 6 -- fix oops on second mount to same server when null auth is used

0006-BZ-354371-CIFS-fix-oops-on-second-mount-to-same-s.patch (text/plain), 1.58 KB, created by Jeff Layton on 2007-12-10 20:20:29 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeff Layton

Created: 2007-12-10 20:20:29 UTC

Size: 1.58 KB

patch

obsolete

>From f434fa0d2f70a2298cab4799a0f750eaf81128ad Mon Sep 17 00:00:00 2001
>From: Jeff Layton <jlayton@redhat.com>
>Date: Fri, 9 Nov 2007 23:25:04 +0000
>Subject: [RHEL5.2 PATCH 06/16] BZ#354371: [CIFS] fix oops on second mount to same server when null auth is used
>
>When a share is mounted using no username, cifs_mount sets
>volume_info.username as a NULL pointer, and the sesInfo userName as an
>empty string. The volume_info.username is passed to a couple of other
>functions to see if there is an existing unc or tcp connection that can
>be used. These functions assume that the username will be a valid
>string that can be passed to strncmp. If the pointer is NULL, then the
>kernel will oops if there's an existing session to which the string
>can be compared.
>
>This patch changes cifs_mount to set volume_info.username to an empty
>string in this situation, which prevents the oops and should make it
>so that the comparison to other null auth sessions match.
>
>Signed-off-by: Jeff Layton <jlayton@redhat.com>
>Signed-off-by: Steve French <sfrench@us.ibm.com>
>---
> fs/cifs/connect.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
>index 4f787c3..60dc369 100644
>--- a/fs/cifs/connect.c
>+++ b/fs/cifs/connect.c
>@@ -1879,7 +1879,7 @@ cifs_mount(struct super_block *sb, struct cifs_sb_info *cifs_sb,
> 
> 	if (volume_info.nullauth) {
> 		cFYI(1, ("null user"));
>-		volume_info.username = NULL;
>+		volume_info.username = "";
> 	} else if (volume_info.username) {
> 		/* BB fixme parse for domain name here */
> 		cFYI(1, ("Username: %s", volume_info.username));
>-- 
>1.5.3.3
>

Actions: View | Diff

Attachments on bug 417961: 283091 | 283101 | 283111 | 283121 | 283131 | 283141 | 283151 | 283161 | 283171 | 283181 | 283191 | 283201 | 283211 | 283221 | 283231 | 283241 | 283251