Re: Call for testing: OpenSSH 6.9

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

 



Hi Darren,

On Jun  2 11:39, Darren Tucker wrote:
> On Mon, Jun 01, 2015 at 03:30:38PM +0200, Corinna Vinschen wrote:
> > Hi,
> > 
> > On May 29 17:12, Damien Miller wrote:
> > > Hi,
> > > 
> > > OpenSSH 6.9 is almost ready for release, so we would appreciate testing
> > > on as many platforms and systems as possible. This release contains
> > > some substantial new features and a number of bugfixes.
> > 
> > I tested git master HEAD on Cygwin 2.0.2 x86_64.
> 
> Thanks!  I'd like to add this small Cygwin change, could you please
> sanity-check?
> 
> diff --git a/openbsd-compat/bsd-cygwin_util.c b/openbsd-compat/bsd-cygwin_util.c
> index a2d8212..8672ccf 100644
> --- a/openbsd-compat/bsd-cygwin_util.c
> +++ b/openbsd-compat/bsd-cygwin_util.c
> @@ -68,7 +68,7 @@ cygwin_ssh_privsep_user()
>        if (cygwin_internal (CW_CYGNAME_FROM_WINNAME, "sshd", cyg_privsep_user,
>  			   sizeof cyg_privsep_user) != 0)
>  #endif
> -	strcpy (cyg_privsep_user, "sshd");
> +	strlcpy(cyg_privsep_user, "sshd", sizeof(cyg_privsep_user));
>      }
>    return cyg_privsep_user;
>  }

that patch is fine, albeit not really necessary.  The source string is a
constant string of 5 bytes and the target buffer size is guaranteed to
be larger than 5 bytes (DNLEN = 15, UNLEN = 256).


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat

Attachment: pgpyRYHZoaJJt.pgp
Description: PGP signature

_______________________________________________
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