Call for testing: OpenSSH 6.6

Kevin Brott kevin.brott at gmail.com
Sat Mar 1 21:24:06 EST 2014


After manually tweaking the dhgex.sh file (gnupatch even un-wrapped still
won't apply), and pointing the make test sysconfdir to the installed config
directory on my Debian box - all tests do pass.  But I'm not sure the
config suite should assume there's already a working ssh config somewhere,
and go looking for it (HP-UX uses a non-std location, so do our localize
builds), as some baseline systems won't even have it.

This bit works with gnupatch to fix dhgex.sh for the CR issue ... but
fixing the CR issue in the test file might make more sense and avoid
needing this.

*** openssh/regress/dhgex.sh.orig       Thu Feb 27 15:21:30 2014
--- openssh/regress/dhgex.sh    Sat Mar  1 02:00:42 2014
***************
*** 32 ****
!       gotbits="`awk '/bits set:/{print $4}' ${LOG} | head -1 | cut -f2
-d/`"
--- 32 ----
!       gotbits="`awk '/bits set:/{print $4}' ${LOG} | head -1 | cut -f2
-d/ | sed -e 's/\r$//'`"



On Sat, Mar 1, 2014 at 1:54 AM, mancha <mancha1 at hush.com> wrote:

> On Fri, 28 Feb 2014 22:41:37 +0000 "Damien Miller" wrote:
> >Hi,
> >
> >OpenSSH 6.6 is almost ready for release, so we would appreciate
> >testing on as many platforms and systems as possible. This is a
> >small release mostly to fix some minor but annoying bugs in
> >openssh-6.5.
>
> >Running the regression tests supplied with Portable OpenSSH does
> >not require installation and is a simply:
> >
> >$ ./configure && make tests
>
> Hi.
>
> After configure && make tests, sshd defaults to looking for the
> system moduli file at /usr/local/etc/moduli. If it doesn't find
> it there, the fallback is using dh group-14 (2048-bit modulus).
>
> This is causing the the dhgex.sh test errors reported by Hisashi
> & Loganaden (i.e. 3072 != 2048).
>
> To resolve this on my system (where moduli file is at /etc/ssh
> /moduli), I use:
>
> $ ./configure && make tests sysconfdir=/etc/ssh
>
> Also, on the system I tested (Slackware Linux), the client logfile
> has CRLF line terminators so $gotbits contains a trailing ^M and
> the comparison fails. Patch below is one way to fix this:
>
> --- a/dhgex.sh
> +++ b/dhgex.sh
> @@ -29,7 +29,7 @@ ssh_test_dhgex()
>                 fail "$tid unexpected GEX sizes, expected $groupsz,
> got $got"
>         fi
>         # check what we got (depends on contents of system moduli
> file)
> -       gotbits="`awk '/bits set:/{print $4}' ${LOG} | head -1 |
> cut -f2 -d/`"
> +       gotbits="`awk '/bits set:/{print $4}' ${LOG} | head -1 |
> cut -f2 -d/ | sed -e 's/\r$//'`"
>         if [ "$gotbits" -lt "$bits" ]; then
>                 fatal "$tid expected $bits bit group, got $gotbits"
>         fi
>
> With these two changes, all tests pass on Slackware Linux 14.1.
>
> --mancha
>
> _______________________________________________
> openssh-unix-dev mailing list
> openssh-unix-dev at mindrot.org
> https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev
>



-- 
# include <stddisclaimer.h>
/* Kevin  Brott <Kevin.Brott at gmail.com> */


More information about the openssh-unix-dev mailing list