additional compiler hardening flags

Darren Tucker dtucker at zip.com.au
Sat Jan 18 21:23:54 EST 2014


On Sat, Jan 18, 2014 at 5:35 AM, Alan Barrett <apb at cequrux.com> wrote:
> On Fri, 17 Jan 2014, Corinna Vinschen wrote:
>>
>> Here's the first one:
>>
>>   ../../src/openbsd-compat/fmt_scaled.c:84:2: error: array subscript has
>> type ‘char’ [-Werror=char-subscripts]
>>     while (isascii(*p) && isspace(*p))
>>     ^
> If the openssh developers are interested in fixing this bug, I refer them to
> the spatch which I submitted several years ago, in
> <https://lists.mindrot.org/pipermail/openssh-unix-dev/2010-August/028895.html>

Sorry, I don't recall looking at that, or if I did I intended to go
back to it after that particular release and didn't.

Anyway, most of those have been fixed upstream recently and these
changes have been pulled into Portable.  The ones that haven't are in
the library functions we pull from OpenBSD but are not part of the
main OpenSSH source.

These have also have been fixed upstream, but there are other
non-trivial changes in those files too.  The choices are:
 a) pull in all the changes for the release and risk breaking something
 b) pull in just the ctype fixes for the release

I am not inclined to do a) until after the release, but doing b)
before the release makes that messier.  I'm leaning toward leaving
those 3 files until immediately after the release.

-- 
Darren Tucker (dtucker at zip.com.au)
GPG key 8FF4FA69 / D9A3 86E9 7EEE AF4B B2D4  37C9 C982 80C7 8FF4 FA69
    Good judgement comes with experience. Unfortunately, the experience
usually comes from bad judgement.


More information about the openssh-unix-dev mailing list