forked from gentoo/gentoo
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
net-misc/openssh: Security bump for CVE-2016-0777 (bug #571892).
Package-Manager: portage-2.2.26 Signed-off-by: Lars Wendler <[email protected]>
- Loading branch information
Lars Wendler
committed
Jan 14, 2016
1 parent
bfd3423
commit ad9f88e
Showing
2 changed files
with
360 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
--- openssh-7.1p1/readconf.c | ||
+++ openssh-7.1p1/readconf.c | ||
@@ -1660,7 +1660,7 @@ | ||
options->tun_remote = -1; | ||
options->local_command = NULL; | ||
options->permit_local_command = -1; | ||
- options->use_roaming = -1; | ||
+ options->use_roaming = 0; | ||
options->visual_host_key = -1; | ||
options->ip_qos_interactive = -1; | ||
options->ip_qos_bulk = -1; | ||
@@ -1833,8 +1833,7 @@ | ||
options->tun_remote = SSH_TUNID_ANY; | ||
if (options->permit_local_command == -1) | ||
options->permit_local_command = 0; | ||
- if (options->use_roaming == -1) | ||
- options->use_roaming = 1; | ||
+ options->use_roaming = 0; | ||
if (options->visual_host_key == -1) | ||
options->visual_host_key = 0; | ||
if (options->ip_qos_interactive == -1) | ||
--- openssh-7.1p1/ssh.c | ||
+++ openssh-7.1p1/ssh.c | ||
@@ -1932,9 +1932,6 @@ | ||
fork_postauth(); | ||
} | ||
|
||
- if (options.use_roaming) | ||
- request_roaming(); | ||
- | ||
return client_loop(tty_flag, tty_flag ? | ||
options.escape_char : SSH_ESCAPECHAR_NONE, id); | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,327 @@ | ||
# Copyright 1999-2016 Gentoo Foundation | ||
# Distributed under the terms of the GNU General Public License v2 | ||
# $Id$ | ||
|
||
EAPI=5 | ||
inherit eutils user flag-o-matic multilib autotools pam systemd versionator | ||
|
||
# Make it more portable between straight releases | ||
# and _p? releases. | ||
PARCH=${P/_} | ||
|
||
HPN_PATCH="${PARCH}-hpnssh14v9.tar.xz" | ||
LDAP_PATCH="${PN}-lpk-6.8p1-0.3.14.patch.xz" | ||
X509_VER="8.6" X509_PATCH="${PN}-${PV//_}+x509-${X509_VER}.diff.gz" | ||
|
||
DESCRIPTION="Port of OpenBSD's free SSH release" | ||
HOMEPAGE="http://www.openssh.org/" | ||
SRC_URI="mirror://openbsd/OpenSSH/portable/${PARCH}.tar.gz | ||
mirror://gentoo/${PN}-6.8_p1-sctp.patch.xz | ||
${HPN_PATCH:+hpn? ( | ||
mirror://gentoo/${HPN_PATCH} | ||
mirror://sourceforge/hpnssh/${HPN_PATCH} | ||
)} | ||
${LDAP_PATCH:+ldap? ( mirror://gentoo/${LDAP_PATCH} )} | ||
${X509_PATCH:+X509? ( http://roumenpetrov.info/openssh/x509-${X509_VER}/${X509_PATCH} )} | ||
" | ||
|
||
LICENSE="BSD GPL-2" | ||
SLOT="0" | ||
KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~m68k ~mips ~ppc ~ppc64 ~s390 ~sh ~sparc ~x86 ~amd64-fbsd ~sparc-fbsd ~x86-fbsd ~arm-linux ~x86-linux" | ||
# Probably want to drop ssl defaulting to on in a future version. | ||
IUSE="bindist debug ${HPN_PATCH:++}hpn kerberos kernel_linux ldap ldns libedit libressl pam +pie sctp selinux skey ssh1 +ssl static X X509" | ||
REQUIRED_USE="ldns? ( ssl ) | ||
pie? ( !static ) | ||
ssh1? ( ssl ) | ||
static? ( !kerberos !pam ) | ||
X509? ( !ldap ssl )" | ||
|
||
LIB_DEPEND=" | ||
ldns? ( | ||
net-libs/ldns[static-libs(+)] | ||
!bindist? ( net-libs/ldns[ecdsa,ssl] ) | ||
bindist? ( net-libs/ldns[-ecdsa,ssl] ) | ||
) | ||
libedit? ( dev-libs/libedit[static-libs(+)] ) | ||
sctp? ( net-misc/lksctp-tools[static-libs(+)] ) | ||
selinux? ( >=sys-libs/libselinux-1.28[static-libs(+)] ) | ||
skey? ( >=sys-auth/skey-1.1.5-r1[static-libs(+)] ) | ||
ssl? ( | ||
!libressl? ( | ||
>=dev-libs/openssl-0.9.8f:0[bindist=] | ||
dev-libs/openssl:0[static-libs(+)] | ||
) | ||
libressl? ( dev-libs/libressl[static-libs(+)] ) | ||
) | ||
>=sys-libs/zlib-1.2.3[static-libs(+)]" | ||
RDEPEND=" | ||
!static? ( ${LIB_DEPEND//\[static-libs(+)]} ) | ||
pam? ( virtual/pam ) | ||
kerberos? ( virtual/krb5 ) | ||
ldap? ( net-nds/openldap )" | ||
DEPEND="${RDEPEND} | ||
static? ( ${LIB_DEPEND} ) | ||
virtual/pkgconfig | ||
virtual/os-headers | ||
sys-devel/autoconf" | ||
RDEPEND="${RDEPEND} | ||
pam? ( >=sys-auth/pambase-20081028 ) | ||
userland_GNU? ( virtual/shadow ) | ||
X? ( x11-apps/xauth )" | ||
|
||
S=${WORKDIR}/${PARCH} | ||
|
||
pkg_setup() { | ||
# this sucks, but i'd rather have people unable to `emerge -u openssh` | ||
# than not be able to log in to their server any more | ||
maybe_fail() { [[ -z ${!2} ]] && echo "$1" ; } | ||
local fail=" | ||
$(use X509 && maybe_fail X509 X509_PATCH) | ||
$(use ldap && maybe_fail ldap LDAP_PATCH) | ||
$(use hpn && maybe_fail hpn HPN_PATCH) | ||
" | ||
fail=$(echo ${fail}) | ||
if [[ -n ${fail} ]] ; then | ||
eerror "Sorry, but this version does not yet support features" | ||
eerror "that you requested: ${fail}" | ||
eerror "Please mask ${PF} for now and check back later:" | ||
eerror " # echo '=${CATEGORY}/${PF}' >> /etc/portage/package.mask" | ||
die "booooo" | ||
fi | ||
|
||
# Make sure people who are using tcp wrappers are notified of its removal. #531156 | ||
if grep -qs '^ *sshd *:' "${EROOT}"/etc/hosts.{allow,deny} ; then | ||
ewarn "Sorry, but openssh no longer supports tcp-wrappers, and it seems like" | ||
ewarn "you're trying to use it. Update your ${EROOT}etc/hosts.{allow,deny} please." | ||
fi | ||
} | ||
|
||
save_version() { | ||
# version.h patch conflict avoidence | ||
mv version.h version.h.$1 | ||
cp -f version.h.pristine version.h | ||
} | ||
|
||
src_prepare() { | ||
sed -i \ | ||
-e "/_PATH_XAUTH/s:/usr/X11R6/bin/xauth:${EPREFIX}/usr/bin/xauth:" \ | ||
pathnames.h || die | ||
# keep this as we need it to avoid the conflict between LPK and HPN changing | ||
# this file. | ||
cp version.h version.h.pristine | ||
|
||
# don't break .ssh/authorized_keys2 for fun | ||
sed -i '/^AuthorizedKeysFile/s:^:#:' sshd_config || die | ||
|
||
if use X509 ; then | ||
pushd .. >/dev/null | ||
pushd ${HPN_PATCH%.*.*} >/dev/null | ||
epatch "${FILESDIR}"/${PN}-7.1_p1-hpn-x509-glue.patch | ||
popd >/dev/null | ||
epatch "${FILESDIR}"/${PN}-7.0_p1-sctp-x509-glue.patch | ||
popd >/dev/null | ||
epatch "${WORKDIR}"/${X509_PATCH%.*} | ||
epatch "${FILESDIR}"/${PN}-6.3_p1-x509-hpn14v2-glue.patch | ||
epatch "${FILESDIR}"/${PN}-6.9_p1-x509-warnings.patch | ||
save_version X509 | ||
fi | ||
if use ldap ; then | ||
epatch "${WORKDIR}"/${LDAP_PATCH%.*} | ||
save_version LPK | ||
fi | ||
epatch "${FILESDIR}"/${PN}-4.7_p1-GSSAPI-dns.patch #165444 integrated into gsskex | ||
epatch "${FILESDIR}"/${PN}-6.7_p1-openssl-ignore-status.patch | ||
# The X509 patchset fixes this independently. | ||
use X509 || epatch "${FILESDIR}"/${PN}-6.8_p1-ssl-engine-configure.patch | ||
epatch "${WORKDIR}"/${PN}-6.8_p1-sctp.patch | ||
if use hpn ; then | ||
EPATCH_FORCE="yes" EPATCH_SUFFIX="patch" \ | ||
EPATCH_MULTI_MSG="Applying HPN patchset ..." \ | ||
epatch "${WORKDIR}"/${HPN_PATCH%.*.*} | ||
save_version HPN | ||
fi | ||
|
||
epatch "${FILESDIR}"/${PN}-7.1_p1-CVE-2016-0777.patch | ||
|
||
tc-export PKG_CONFIG | ||
local sed_args=( | ||
-e "s:-lcrypto:$(${PKG_CONFIG} --libs openssl):" | ||
# Disable PATH reset, trust what portage gives us #254615 | ||
-e 's:^PATH=/:#PATH=/:' | ||
# Disable fortify flags ... our gcc does this for us | ||
-e 's:-D_FORTIFY_SOURCE=2::' | ||
) | ||
# The -ftrapv flag ICEs on hppa #505182 | ||
use hppa && sed_args+=( | ||
-e '/CFLAGS/s:-ftrapv:-fdisable-this-test:' | ||
-e '/OSSH_CHECK_CFLAG_LINK.*-ftrapv/d' | ||
) | ||
sed -i "${sed_args[@]}" configure{.ac,} || die | ||
|
||
epatch_user #473004 | ||
|
||
# Now we can build a sane merged version.h | ||
( | ||
sed '/^#define SSH_RELEASE/d' version.h.* | sort -u | ||
macros=() | ||
for p in HPN LPK X509 ; do [ -e version.h.${p} ] && macros+=( SSH_${p} ) ; done | ||
printf '#define SSH_RELEASE SSH_VERSION SSH_PORTABLE %s\n' "${macros}" | ||
) > version.h | ||
|
||
eautoreconf | ||
} | ||
|
||
src_configure() { | ||
addwrite /dev/ptmx | ||
|
||
use debug && append-cppflags -DSANDBOX_SECCOMP_FILTER_DEBUG | ||
use static && append-ldflags -static | ||
|
||
local myconf=( | ||
--with-ldflags="${LDFLAGS}" | ||
--disable-strip | ||
--with-pid-dir="${EPREFIX}"$(usex kernel_linux '' '/var')/run | ||
--sysconfdir="${EPREFIX}"/etc/ssh | ||
--libexecdir="${EPREFIX}"/usr/$(get_libdir)/misc | ||
--datadir="${EPREFIX}"/usr/share/openssh | ||
--with-privsep-path="${EPREFIX}"/var/empty | ||
--with-privsep-user=sshd | ||
$(use_with kerberos kerberos5 "${EPREFIX}"/usr) | ||
# We apply the ldap patch conditionally, so can't pass --without-ldap | ||
# unconditionally else we get unknown flag warnings. | ||
$(use ldap && use_with ldap) | ||
$(use_with ldns) | ||
$(use_with libedit) | ||
$(use_with pam) | ||
$(use_with pie) | ||
$(use_with sctp) | ||
$(use_with selinux) | ||
$(use_with skey) | ||
$(use_with ssh1) | ||
# The X509 patch deletes this option entirely. | ||
$(use X509 || use_with ssl openssl) | ||
$(use_with ssl md5-passwords) | ||
$(use_with ssl ssl-engine) | ||
) | ||
|
||
# The seccomp sandbox is broken on x32, so use the older method for now. #553748 | ||
use amd64 && [[ ${ABI} == "x32" ]] && myconf+=( --with-sandbox=rlimit ) | ||
|
||
econf "${myconf[@]}" | ||
} | ||
|
||
src_install() { | ||
emake install-nokeys DESTDIR="${D}" | ||
fperms 600 /etc/ssh/sshd_config | ||
dobin contrib/ssh-copy-id | ||
newinitd "${FILESDIR}"/sshd.rc6.4 sshd | ||
newconfd "${FILESDIR}"/sshd.confd sshd | ||
keepdir /var/empty | ||
|
||
newpamd "${FILESDIR}"/sshd.pam_include.2 sshd | ||
if use pam ; then | ||
sed -i \ | ||
-e "/^#UsePAM /s:.*:UsePAM yes:" \ | ||
-e "/^#PasswordAuthentication /s:.*:PasswordAuthentication no:" \ | ||
-e "/^#PrintMotd /s:.*:PrintMotd no:" \ | ||
-e "/^#PrintLastLog /s:.*:PrintLastLog no:" \ | ||
"${ED}"/etc/ssh/sshd_config || die | ||
fi | ||
|
||
# Gentoo tweaks to default config files | ||
cat <<-EOF >> "${ED}"/etc/ssh/sshd_config | ||
# Allow client to pass locale environment variables #367017 | ||
AcceptEnv LANG LC_* | ||
EOF | ||
cat <<-EOF >> "${ED}"/etc/ssh/ssh_config | ||
# Send locale environment variables #367017 | ||
SendEnv LANG LC_* | ||
EOF | ||
|
||
if ! use X509 && [[ -n ${LDAP_PATCH} ]] && use ldap ; then | ||
insinto /etc/openldap/schema/ | ||
newins openssh-lpk_openldap.schema openssh-lpk.schema | ||
fi | ||
|
||
doman contrib/ssh-copy-id.1 | ||
dodoc ChangeLog CREDITS OVERVIEW README* TODO sshd_config | ||
|
||
diropts -m 0700 | ||
dodir /etc/skel/.ssh | ||
|
||
systemd_dounit "${FILESDIR}"/sshd.{service,socket} | ||
systemd_newunit "${FILESDIR}"/sshd_at.service '[email protected]' | ||
} | ||
|
||
src_test() { | ||
local t tests skipped failed passed shell | ||
tests="interop-tests compat-tests" | ||
skipped="" | ||
shell=$(egetshell ${UID}) | ||
if [[ ${shell} == */nologin ]] || [[ ${shell} == */false ]] ; then | ||
elog "Running the full OpenSSH testsuite" | ||
elog "requires a usable shell for the 'portage'" | ||
elog "user, so we will run a subset only." | ||
skipped="${skipped} tests" | ||
else | ||
tests="${tests} tests" | ||
fi | ||
# It will also attempt to write to the homedir .ssh | ||
local sshhome=${T}/homedir | ||
mkdir -p "${sshhome}"/.ssh | ||
for t in ${tests} ; do | ||
# Some tests read from stdin ... | ||
HOMEDIR="${sshhome}" \ | ||
emake -k -j1 ${t} </dev/null \ | ||
&& passed="${passed}${t} " \ | ||
|| failed="${failed}${t} " | ||
done | ||
einfo "Passed tests: ${passed}" | ||
ewarn "Skipped tests: ${skipped}" | ||
if [[ -n ${failed} ]] ; then | ||
ewarn "Failed tests: ${failed}" | ||
die "Some tests failed: ${failed}" | ||
else | ||
einfo "Failed tests: ${failed}" | ||
return 0 | ||
fi | ||
} | ||
|
||
pkg_preinst() { | ||
enewgroup sshd 22 | ||
enewuser sshd 22 -1 /var/empty sshd | ||
} | ||
|
||
pkg_postinst() { | ||
if has_version "<${CATEGORY}/${PN}-5.8_p1" ; then | ||
elog "Starting with openssh-5.8p1, the server will default to a newer key" | ||
elog "algorithm (ECDSA). You are encouraged to manually update your stored" | ||
elog "keys list as servers update theirs. See ssh-keyscan(1) for more info." | ||
fi | ||
if has_version "<${CATEGORY}/${PN}-6.9_p1" ; then | ||
elog "Starting with openssh-6.9p1, ssh1 support is disabled by default." | ||
fi | ||
if has_version "<${CATEGORY}/${PN}-7.0_p1" ; then | ||
elog "Starting with openssh-6.7, support for USE=tcpd has been dropped by upstream." | ||
elog "Make sure to update any configs that you might have. Note that xinetd might" | ||
elog "be an alternative for you as it supports USE=tcpd." | ||
fi | ||
if has_version "<${CATEGORY}/${PN}-7.1_p1" ; then #557388 #555518 | ||
elog "Starting with openssh-7.0, support for ssh-dss keys were disabled due to their" | ||
elog "weak sizes. If you rely on these key types, you can re-enable the key types by" | ||
elog "adding to your sshd_config or ~/.ssh/config files:" | ||
elog " PubkeyAcceptedKeyTypes=+ssh-dss" | ||
elog "You should however generate new keys using rsa or ed25519." | ||
|
||
elog "Starting with openssh-7.0, the default for PermitRootLogin changed from 'yes'" | ||
elog "to 'prohibit-password'. That means password auth for root users no longer works" | ||
elog "out of the box. If you need this, please update your sshd_config explicitly." | ||
fi | ||
if ! use ssl && has_version "${CATEGORY}/${PN}[ssl]" ; then | ||
elog "Be aware that by disabling openssl support in openssh, the server and clients" | ||
elog "no longer support dss/rsa/ecdsa keys. You will need to generate ed25519 keys" | ||
elog "and update all clients/servers that utilize them." | ||
fi | ||
} |