[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#5408) back-ldif bugs



hyc@symas.com writes:
> h.b.furuseth@usit.uio.no wrote:
>>    - back-ldif escapes the directory separator as \<hex value>.  But
>>      Windows uses "\" as directory separator, so that doesn't help.  It
>>      needs another escape character.
>>
>>      To avoid double hex-escaping of DNs that contain "\", we can pick
>>      another escape char, e.g. "^", escape that too, and translate "\" to
>>      it.  Thus DN "cn=x^y\2Cz" gets filename "cn=x^5Ey^2Cz.ldif".
>
> That sounds fine. Is there any reason not to use %, which is already
> used for URL encoding?

Good point.  The filenames I suggested will not be valid URLs but will
resemble them, but that can be fixed by using proper URL-encoding of
RDNs.  Filenames for RDNs with \ in them get a bit longer though.

>>    - More characters should likely be hex-escaped:
>>      + ":" (as in "C:\") and "/" on Windows?  I've seen programs use the
>>        latter as directory separator even on Windows.  Others?
>>      + 8-bit chars in case the OS gets clever about charset handling.
>>      + Control chars.
>>
>>      I don't use Windows myself though.  Nor Mac...
>
> Again, the rules for URL encoding probably already cover any cases we
> should worry about.

I don't think so.  At least "/" is not "unsafe", if I read it right.

>>    - RDNs ending with ".ldif" should be escaped.  Currently "cn=foo.ldif"
>>      is the name of both RDN "cn=foo"'s entry file and RDN "cn=foo.ldif"'s
>>      non-leaf directory.
>
> Perhaps we should escape '.'

Hm, true.  "." is (was?) a special character on some filesystems.
E.g. a file can only have one of them, or the 2nd is followed by a
version number.

>>    - Sorted-value RDNs:

No reply? :-)  Ok, sort order can be adjusted separately anyway.
One error in my previous attempt: bconfig.c mentions
	"pseudo-indexed" entries (cn=Include{xx}, cn=Module{xx})
which I presume must be sorted by {xx} too, so back-ldif can't
treat only "attr={number}val" as indexed.  I guess I'll either check
both variants or allow {number} anywhere, depending on convenience.

-- 
Hallvard