On Tue, Sep 10, 2013 at 09:48:57AM -0700, Jeremy Allison wrote:
> It's an old, old check back from when SJIS and EUC were
> common multi-byte systems.
> 
> SJIS especially has the property that the second byte
> can contain a value <127 as part of the 2-byte char
> set. So if CH_UNIX is set to a char set with such a
> property we can't walk it as bytes, but must see if
> a pair of values [0] (> 0x80) [1] (any value) can be
> converted into a valid multi-byte char, in which case
> we ignore it (otherwise we might look at the second
> byte value of ':' or something and consider it invalid).
> 
> I thought about removing this and re-writing it, but
> it made my brain hurt (and might break some very old
> systems :-). So moving to next_codepoint() which checks
> the next char len without causing the conversion error
> messages seemed the simplest fix :-).

Thanks! +1 from me.

Volker

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kont...@sernet.de

*****************************************************************
visit us on it-sa:IT security exhibitions in Nürnberg, Germany
October 8th - 10th 2013, hall 12, booth 333
free tickets available via code 270691 on: www.it-sa.de/gutschein
******************************************************************
-- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/options/samba

Reply via email to