Bug 2152 - mangling method = hash, problem with LFNs for dotfiles
Summary: mangling method = hash, problem with LFNs for dotfiles
Status: CLOSED FIXED
Alias: None
Product: Samba 3.0
Classification: Unclassified
Component: File Services (show other bugs)
Version: 3.0.9
Hardware: All Solaris
: P3 normal
Target Milestone: none
Assignee: Jeremy Allison
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-12-13 09:25 UTC by Neil Hoggarth
Modified: 2005-08-24 10:16 UTC (History)
0 users

See Also:


Attachments
unidiff for removing the troublesome code from mangle_hash.c (414 bytes, patch)
2005-03-04 08:07 UTC, Neil Hoggarth
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Neil Hoggarth 2004-12-13 09:25:13 UTC
I've just upgraded a Samba 2.2.8a file server to Samba 3.0.9. I planned my
transition carefully and added "mangling method = hash" to the smb.conf, so that
existing mangled names stored on clients would remain valid.

I've noticed that when a long filename capable client lists the contents of a
directory, any files with Unix filename starting with a period character are
returned in their mangled 8.3 form. So, for example, ".profile" is listed as
"PROFI~M#.___", whereas both Samba 3.0.9 with mangling method hash2 and Samba
2.2.8a with mangling method hash will return ".profile" as a long filename (and
clients seem happy with such names).

This comes about because in 3.0.9 smbd/mangle_hash.c:name_map() checks the name
is with "is_valid_name()" and forces "need83 = True" if this check fails.
is_valid_name() explicitly checks for names that begin with a period character
and returns NT_STATUS_UNSUCCESSFUL for such names.

The corresponding check in 3.0.9 smbd/mangle_hash2.c:name_map() and 2.2.8a
smbd/mangle_hash.c:name_map_mangle() are less stringent - they both call
functions which check for illegal characters and *trailing* periods, but not
leading periods.

The problem was observed on a SPARC Solaris 9 system. I think that it is general
though.
Comment 1 Neil Hoggarth 2005-03-04 08:07:37 UTC
Created attachment 1008 [details]
unidiff for removing the troublesome code from mangle_hash.c

This patch removes the test for a leading period from function is_valid_name()
in smbd/mangle_hash.c.

is_valid_name() is a static function that is only called in two places: once by
name_map() and once by is_8_3_w(). In the first case the leading period test
causes bug 2152 (unwanted and unnecessary name mangling is applied to a long
filename which starts with a period). In the second case the leading period
test is redundant: is_8_3_w() breaks an 8.3 name down into "prefix" and
"extension", and rejects as invalid names that have a zero-length prefix (if
statement at line 266).
Comment 2 Björn Jacke 2005-08-19 07:24:56 UTC
this looks reasonable to me, shouldn't this be checked in ?
Comment 3 Gerald (Jerry) Carter (dead mail address) 2005-08-23 08:30:13 UTC
Jeremy,  mind deciding what to do with this patch?
Comment 4 Jeremy Allison 2005-08-23 10:49:44 UTC
Actually, this test is an optimization put in the wrong place. I've moved it
into is_8_3_w() instead which is where it should have been.
Thanks, sorry this didn't make 3.0.20 - it'll definately be in 3.0.21.
Jeremy.
Comment 5 Gerald (Jerry) Carter (dead mail address) 2005-08-24 10:16:46 UTC
sorry for the same, cleaning up the database to prevent unecessary reopens of bugs.