[PATCH] Allow . to end a filename, use real unicode for
abartlet at samba.org
Sun Jul 27 22:56:16 GMT 2003
On Mon, 2003-07-28 at 08:48, Jeremy Allison wrote:
> On Mon, Jul 28, 2003 at 08:43:39AM +1000, Andrew Bartlett wrote:
> > Given all that, I think it's a reasonable approach (and that's the
> > reason I raised it on the list). And I would never have disagreed with
> > it if masktest hadn't tripped over it - I'm much more worried about the
> > fnmatch code...
> Yes - me too. Can you give me an explaination of what your changes
> are trying to fix (or a bugid) ?
It started with #205, and the fact that the 'unicode' conversion in
mask_match is by null-padding, not by actual use of iconv().
Given the problem on expanding/contracting strings and strlower_m(), I
looked at how far down I could push the problem - and realised that we
could just do the case-sensitive part in the fnmatch code itself,
without ever lowercasing the entire string.
It also allowed me to say 'invalid strings don't match', rather than the
smb_panic() in unix_strlower().
BTW, why does the comparison occur in lowercase? For 'weird' languages
we should determine exactly which case the case-insensitve stuff
actually occurs in.
Andrew Bartlett abartlet at pcug.org.au
Manager, Authentication Subsystems, Samba Team abartlet at samba.org
Student Network Administrator, Hawker College abartlet at hawkerc.net
http://samba.org http://build.samba.org http://hawkerc.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.samba.org/archive/samba-technical/attachments/20030727/1ee35b7d/attachment.bin
More information about the samba-technical