summaryrefslogtreecommitdiffstats
path: root/kex.c
diff options
context:
space:
mode:
authorderaadt@openbsd.org <deraadt@openbsd.org>2024-08-23 06:51:00 +0200
committerDamien Miller <djm@mindrot.org>2024-08-27 01:05:43 +0200
commit10ccf611ab8ecba9ce6b0548c5ccd8c1220baf92 (patch)
treece6dc0c5c4f0f624323c54a2b5b260e1c078a673 /kex.c
parentupstream: sntrup761x25519-sha512 now has an IANA codepoint assigned, so (diff)
downloadopenssh-10ccf611ab8ecba9ce6b0548c5ccd8c1220baf92.tar.xz
openssh-10ccf611ab8ecba9ce6b0548c5ccd8c1220baf92.zip
upstream: As defined in the RFC, the SSH protocol has negotiable
compression support (which is requested as the name "zlib"). Compression starts very early in the session. Relative early in OpenSSH lifetime, privsep was added to sshd, and this required a shared-memory hack so the two processes could see what was going on in the dataflow. This shared-memory hack was soon recognized as a tremendous complexity risk, because it put libz (which very much trusts it's memory) in a dangerous place, and a new option ("zlib@openssh.com") was added begins compression after authentication (aka delayed-compression). That change also permitted removal of the shared-memory hack. Despite removal from the server, the old "zlib" support remained in the client, to allow negotiation with non-OpenSSH daemons which lack the delayed-compression option. This commit deletes support for the older "zlib" option in the client. It reduces our featureset in a small way, and encourages other servers to move to a better design. The SSH protocol is different enough that compressed-key-material attacks like BEAST are unlikely, but who wants to take the chance? We encourage other ssh servers who care about optional compression support to add delayed-zlib support. (Some already do "zlib@openssh.com") ok djm markus OpenBSD-Commit-ID: 6df986f38e4ab389f795a6e39e7c6857a763ba72
Diffstat (limited to 'kex.c')
-rw-r--r--kex.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/kex.c b/kex.c
index 63aae5d71..6b957e5e1 100644
--- a/kex.c
+++ b/kex.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: kex.c,v 1.186 2024/05/17 00:30:23 djm Exp $ */
+/* $OpenBSD: kex.c,v 1.187 2024/08/23 04:51:00 deraadt Exp $ */
/*
* Copyright (c) 2000, 2001 Markus Friedl. All rights reserved.
*
@@ -842,8 +842,6 @@ choose_comp(struct sshcomp *comp, char *client, char *server)
#ifdef WITH_ZLIB
if (strcmp(name, "zlib@openssh.com") == 0) {
comp->type = COMP_DELAYED;
- } else if (strcmp(name, "zlib") == 0) {
- comp->type = COMP_ZLIB;
} else
#endif /* WITH_ZLIB */
if (strcmp(name, "none") == 0) {