[PATCH] on 2nd try mount.cifs must also uppercase "orig_dev" - recent kernels now ignore "unc=..." mount option

[PATCH] on 2nd try mount.cifs must also uppercase "orig_dev" - recent kernels now ignore "unc=..." mount option

[Günter Kukkukk]

[-- Attachment #1: Type: text/plain, Size: 153 bytes --]

userland mount helper mount.cifs.

Also uppercase the now used (kernel) "orig_dev" parameter, to get the
mount retry working again.

Cheers, Günter
-- 


[-- Attachment #2: 0001-on-2nd-try-mount.cifs-must-also-uppercase-orig_dev-r.patch --]
[-- Type: text/x-patch, Size: 1247 bytes --]

>From 5b6633b93b4df19b6c2863b8c9cf11d622132b60 Mon Sep 17 00:00:00 2001
From: Guenter Kukkukk <kukks-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>
Date: Tue, 1 Jul 2014 17:43:55 +0200
Subject: [PATCH] on 2nd try mount.cifs must also uppercase "orig_dev" - recent
 kernels now ignore "unc=..." mount option

mount.cifs.c, when getting errno=ENXIO, retries the mount with uppercased
hostname, sharename and prefixpath in the "unc=..." mount option, which
is ignored now in the kernel. Used e.g. during OS/2 mounts, which fail now.

Also uppercase the now used "orig_dev" parameter.

Signed-off-by: Guenter Kukkukk <kukks-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>
---
 mount.cifs.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/mount.cifs.c b/mount.cifs.c
index 497665d..3535096 100644
--- a/mount.cifs.c
+++ b/mount.cifs.c
@@ -2097,7 +2097,8 @@ mount_retry:
 			if (!already_uppercased &&
 			    uppercase_string(parsed_info->host) &&
 			    uppercase_string(parsed_info->share) &&
-			    uppercase_string(parsed_info->prefix)) {
+			    uppercase_string(parsed_info->prefix) &&
+			    uppercase_string(orig_dev)) {
 				fprintf(stderr,
 					"Retrying with upper case share name\n");
 				already_uppercased = 1;
-- 
1.8.4.5

Re: [PATCH] on 2nd try mount.cifs must also uppercase "orig_dev" - recent kernels now ignore "unc=..." mount option

[Jeff Layton]

On Tue, 01 Jul 2014 19:05:29 +0200
Günter Kukkukk <linux-KQewbsS9MvBBDgjK7y7TUQ@public.gmane.org> wrote:

> userland mount helper mount.cifs.
> 
> Also uppercase the now used (kernel) "orig_dev" parameter, to get the
> mount retry working again.
> 
> Cheers, Günter

Thanks Günter. Patch looks correct to me. I'll merge it within the next
few days unless someone objects.

Cheers!
-- 
Jeff Layton <jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>

Re: [PATCH] on 2nd try mount.cifs must also uppercase "orig_dev" - recent kernels now ignore "unc=..." mount option

[Jeff Layton]

On Tue, 01 Jul 2014 19:05:29 +0200
Günter Kukkukk <linux-KQewbsS9MvBBDgjK7y7TUQ@public.gmane.org> wrote:

> userland mount helper mount.cifs.
> 
> Also uppercase the now used (kernel) "orig_dev" parameter, to get the
> mount retry working again.
> 
> Cheers, Günter

Patch merged into master.

Thanks!
-- 
Jeff Layton <jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>