r11861: Fix inspired by Thomas Neumann <t.neumann@iku-ag.de> to ensure that
authorJeremy Allison <jra@samba.org>
Tue, 22 Nov 2005 16:21:52 +0000 (16:21 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 16:05:31 +0000 (11:05 -0500)
default case applies only to new files and correctly examines 8.3
and long names.
Jeremy.
(This used to be commit ba931a015132f7e8fffa52c617cd8345a1da35d2)

source3/smbd/filename.c

index b389b9c1ae72acb05da43e43254f37e5da3b6774..2ee8ba1e4ffa40c4e3a0b04079fb98a71a798a19 100644 (file)
@@ -150,9 +150,6 @@ BOOL unix_convert(pstring name,connection_struct *conn,char *saved_last_componen
                        pstrcpy(saved_last_component, name);
        }
 
-       if (!conn->case_preserve || (mangle_is_8_3(name, False, SNUM(conn)) && !conn->short_case_preserve))
-               strnorm(name, lp_defaultcase(SNUM(conn)));
-
        start = name;
        pstrcpy(orig_path, name);
 
@@ -301,16 +298,17 @@ BOOL unix_convert(pstring name,connection_struct *conn,char *saved_last_componen
                                        return(False);
                                }
 
-                               /* 
+                               /*
                                 * Just the last part of the name doesn't exist.
-                                * We may need to strupper() or strlower() it in case
-                                * this conversion is being used for file creation 
-                                * purposes. If the filename is of mixed case then 
-                                * don't normalise it.
+                                * We need to strupper() or strlower() it as
+                                * this conversion may be used for file creation 
+                                * purposes. Fix inspired by Thomas Neumann <t.neumann@iku-ag.de>.
                                 */
-
-                               if (!conn->case_preserve && (!strhasupper(start) || !strhaslower(start)))               
+                               if (!conn->case_preserve ||
+                                               (mangle_is_8_3(start, False, SNUM(conn)) &&
+                                                !conn->short_case_preserve)) {
                                        strnorm(start, lp_defaultcase(SNUM(conn)));
+                               }
 
                                /*
                                 * check on the mangled stack to see if we can recover the