Re: [patch] scp + UTF-8

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

 



Hi Michael,

Michael Stone wrote on Wed, Jan 20, 2016 at 07:50:03AM -0500:
> On Wed, Jan 20, 2016 at 01:13:10AM +0100, Roland Mainz wrote:

>> 3. I am not sure whether there is a specific byte limit for UTF-8 in
>> any of the standards, e.g. "- To support terminals larger then
>> MAX_WINSIZE and still be properly indented I increased the buf size to
>> 4x the size of MAX_WINSIZE, since the maximum size of an UTF-8 char
>> <should> be 4 bytes." might not be a portable assumption and I would
>> at least safeguard it.

> Isn't that assumption completely broken in the presence of combining
> characters?

It is.  As far as i understand, given any natural number N > 0,
you can construct a valid Unicode string of display width 1
such that its UTF-8 encoding consists of N bytes.

Not that such strings are terribly useful for large N, but we do
indeed have to keep in mind that the code must not break when it
encouters them.  Martijn's code already seems safe in *that*
respect, it just cuts the string early and pads with blanks if
the buffer is too short, but i'll re-check before deciding on a
final version.

Thanks for the reminder!

Yours,
  Ingo
_______________________________________________
openssh-unix-dev mailing list
openssh-unix-dev@xxxxxxxxxxx
https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev



[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux