Filenames with double quotes seem to have broken sometime before 4.0.26 versions.

With standard 4.0.26 a file called say "Test” (including the double quotes, ASCII 0x22 as seen on the server side) gets mangled to an 8.3 name when presented to the client. I have tested this with OS X 10.9 OSX 10.10.4 and Windows 7 clients.

The files were originally created with Samba from one of these clients, but at this point I cannot conclusively say which Samba version worked properly last. The problem became apparent after files that were created from the client are no longer accessible using their name with double quotes and only the mangled name is available.

Changing source3/smbd/mangle_hash2.c like to not treat double quotes as a special character (patch below) make the mangling stop but causes translation on 0x22 as shown below. I don’t see where this is happening.

Doing

  touch \”XYZ\”
  ls *XYZ* | xxd

when in a share directory on the client side gives:

0000000: 2258 595a 220a                           "XYZ”.

On the server side, in the same directory, ls *XYZ* | xxd gives:

00000000: ef80 a058 595a ef80 a00a                 ...XYZ….

Files are then available on the client side with conventional 0x22 double quotes, but in the Unix filesystem the quote has been changed to the UTF-8 sequence 0xef 0x80 0xa0, which by reading is the Tibetan digit zero, 0xf20, which does not seem to be a reasonable mapping for a double quote.

This is on FreeBSD 10.2-PRERELEASE with a ZFS filesystem, with Unicode filenames enabled and enforced. This is samba 4.0.26 installed using the standard FreeBSD port.

--- source3/smbd/mangle_hash2.c.orig	2015-07-29 13:10:58.889516029 +1000
+++ source3/smbd/mangle_hash2.c	2015-07-29 13:14:11.914503055 +1000
@@ -109,7 +109,7 @@
 { "AUX", "LOCK$", "CON", "COM1", "COM2", "COM3", "COM4",
   "LPT1", "LPT2", "LPT3", "NUL", "PRN", NULL };
 
-#define DYNAMIC_MANGLE_TABLES 0
+#define DYNAMIC_MANGLE_TABLES 1
 
 #if DYNAMIC_MANGLE_TABLES
 
@@ -145,7 +145,7 @@
 			char_flags[i] |= FLAG_ASCII;
 		}
 
-		if (strchr("*\\/?<>|\":", i)) {
+		if (strchr("*\\/?<>|:", i)) {
 			char_flags[i] |= FLAG_ILLEGAL;
 		}