From nobody Mon Jan 6 00:44:06 2025 X-Original-To: dev-commits-ports-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4YRFmk4ftcz5j49H; Mon, 06 Jan 2025 00:44:06 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R11" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4YRFmk28J1z4l6n; Mon, 6 Jan 2025 00:44:06 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1736124246; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=aevyrDFTEcfO7dSbFVTQHbpVjoe31GS6em8uQg68frU=; b=hK5CXviAPRsYlYoTGtwLcPHmvq3bPELGbF1RwnT2eG3auSv8KzesbTki8PpFmAaQgL2XJv 2j8Vw43Tbnswz4pk2PgYd9/s1rfGQos9C5UPfLwIyfU94SGJmtP8UOwtwwDg32epiNXZxu VtfN5dwRHTe5QukNyiUyEcOjRTNjT289Jp+RrnOa5JYeJJglsYpI1/FgnkwJjUHGV3fqwt kpNU54oVIqTEXoT1Jqv2uY7uq/RCP3rokyJIXFRrlEi87RGEmul30BDTSeEyoy3zj04w+3 e4KI+nt21z9Bx2ACCv/JkepcKBRRuOY8lfQs29D5gHris5/TKKe2msBBaTulMw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1736124246; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=aevyrDFTEcfO7dSbFVTQHbpVjoe31GS6em8uQg68frU=; b=Z8B29j9pxW+zPh09zR5B0hAWm70KNe2hXGgowL3i+zQURkEocXp/TFjuotlkOo0XYbMmQd Gv8czGIoN0ys/PFCB2msM4oYsHY54jLaDuD9pIhHzA2ZC8IgvqrQy1NyW7ScHsficAUWWs lp7Gk13r+K92LET2IOXo4X5YP/aMllZo6584e34XJedsP6L2fJVGv9Eua23Vup+h7/IPyl XIAzuN5PdBoVCIlyX7Gyj6OAeTFvfShn3KoEGWiZgpUGFrQeeWjzkqUbSnR3N8WorUVywh 0Tzj5yIDp7fk0ljr/qP1lA+tbHGvI62P/PdgPP6ICgPw63YhEnVKvEtcDnMJlQ== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1736124246; a=rsa-sha256; cv=none; b=BKtXA+37HERITgTLcn2Bn8BiS8xTVHY+vGykdrNyaM9FMD6EimUhAooA9Vs4OhsBZBUAde kyUEildsV877LNg24cYM1fOIEfuTD2m+knutHEoyDwjaoIa9ZWOM1HjjEfE0/yQz+raHFj D82rQ1OKz8SHJJ+WGykndXK6fIHbknVcz7EvdGedrcoNc0fVGar+06OQnK8tLhfRt18rCf 0xV3nKhZj0hcytv9eYhL19o2Bx8v0ayQ+BYxR+MvTTmYSmsz2DtBO7RkdDB6oWZzlKyV3N jMTQhn7kqnHmmuOqIpFT2rq7tKmKCrS8yCZiPMgHmMtnekH4l4QODdmw8pLoKA== ARC-Authentication-Results: i=1; mx1.freebsd.org; none Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4YRFmk1lp0z12MJ; Mon, 06 Jan 2025 00:44:06 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.18.1/8.18.1) with ESMTP id 5060i6jA059845; Mon, 6 Jan 2025 00:44:06 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.18.1/8.18.1/Submit) id 5060i6bQ059842; Mon, 6 Jan 2025 00:44:06 GMT (envelope-from git) Date: Mon, 6 Jan 2025 00:44:06 GMT Message-Id: <202501060044.5060i6bQ059842@gitrepo.freebsd.org> To: ports-committers@FreeBSD.org, dev-commits-ports-all@FreeBSD.org, dev-commits-ports-main@FreeBSD.org From: Matthias Andree Subject: git: e7afbce7472c - main - security/putty: update to pre-0.83 as of 2025-01-05 List-Id: Commit messages for all branches of the ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-ports-all@freebsd.org Sender: owner-dev-commits-ports-all@FreeBSD.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: mandree X-Git-Repository: ports X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: e7afbce7472c2f071ca94ae856b246b9a6667d76 Auto-Submitted: auto-generated The branch main has been updated by mandree: URL: https://cgit.FreeBSD.org/ports/commit/?id=e7afbce7472c2f071ca94ae856b246b9a6667d76 commit e7afbce7472c2f071ca94ae856b246b9a6667d76 Author: Matthias Andree AuthorDate: 2025-01-06 00:39:35 +0000 Commit: Matthias Andree CommitDate: 2025-01-06 00:43:34 +0000 security/putty: update to pre-0.83 as of 2025-01-05 changes by Simon Tatham unless otherwise stated, newest first: * 1e451997 2024-12-26 | Treat SOS and PM terminal escape sequences like APC (HEAD -> pre-0.83, origin/pre-0.83) * 98200d1b 2024-12-19 | Arm: turn on PSTATE.DIT if available and needed. * c2077f88 2024-12-19 | Fix compile warnings in tree234 tests. * 27550b02 2024-12-16 | Windows: inhibit all default application manifests. * 363debc7 2024-12-15 | lineedit: make both ^M and ^J terminate a line. * 1fc5f4af 2024-12-15 | wm_size_resize_term: update conf unconditionally. * 11c7c760 2024-12-15 | Remove bit-rotted RDB_DEBUG_PATCH. * c91437ba 2024-12-15 | Update cmake_minimum_required to avoid warnings on sid. * 7802932e 2024-12-15 | Document how to set GIT_SSH_COMMAND to plink -batch. * 09095a7d 2023-07-15 | Avoid treating non-X GDK display names as X ones [Ben Harris] * 1ce8ec9c 2024-12-08 | lineedit_send_line: batch up output characters. * edd5e13f 2024-12-14 | Fix assertion failure on Restart Session. * f8e1a2b3 2024-12-13 | Windows: rewrite request_file() to support Unicode. * 22dfc46f 2024-12-13 | Windows: add filename_to_wstr(). * 1ef0fbaa 2024-12-13 | Add helper function dupwcscat(). * 897ecf46 2024-12-11 | SUPDUP: make the TDCRL command clear to end of line. * 3c6a5139 2024-12-08 | Minimally document ML-KEM key exchange methods. [Jacob Nevins] * a3f22a2c 2024-12-08 | Use the new 'HYBRID' names for the hybrid KEX packets. * e98615f0 2024-12-07 | New post-quantum kex: ML-KEM, and three hybrids of it. * b36d490b 2024-12-07 | Give the kex selection list box a fixed height. * 16629d3b 2024-12-07 | Add more variants of SHAKE. * f08da2b6 2024-12-07 | Separate NTRU Prime from the hybridisation layer. * fcdc804b 2024-12-01 | Move some NTRU helper routines into a header file. * c2d7ea8e 2024-12-04 | Fix use of aligned_alloc() to be ASan-clean. * 7da34495 2024-12-07 | Fix error message when KEXINIT negotiation fails. * 296b6291 2024-12-07 | GTK: fix a crash when clicking Cancel on Change Settings. * 6a88b294 2024-12-04 | Unix PuTTY/pterm: fix UB with small keypad. [Jacob Nevins] * b97f20d0 2024-11-30 | release.pl: Adjust pscp/plink transcript updater. [Jacob Nevins] * 54f6fefe 2024-11-30 | Docs: pscp/plink now need -h/--help to print usage. [Jacob Nevins] * ebe24534 2024-11-28 | psftp: use cmdline_arg_to_filename for batch files. * d4e848a9 2024-11-28 | CHECKLST: update for some extra test builds. * 948a4c8e 2024-11-28 | Fix a compile warning when building with GTK 1. * 8805cf3d 2024-11-28 | Fix a build failure with NO_GSSAPI defined. * c72a8627 2024-11-28 | Fix build failures with NO_IPV6 defined. --- security/putty/Makefile | 12 +++--- security/putty/distinfo | 6 +-- .../patch-296b6291d39c0cf118cd3081c3ab86a5889eb4d9 | 33 -------------- .../patch-6a88b294276b9c24584efa18b9a37f437fa46712 | 50 ---------------------- .../patch-7da3449586ea3e6faaa92663d32774e28cf4e2e3 | 36 ---------------- .../patch-c72a86272446c0e4fb33c68601563549044b29e6 | 40 ----------------- .../patch-f8e1a2b3a934d750aba7c26d182f52d71952c529 | 43 ------------------- 7 files changed, 10 insertions(+), 210 deletions(-) diff --git a/security/putty/Makefile b/security/putty/Makefile index a7ae097d6c5e..da57463800da 100644 --- a/security/putty/Makefile +++ b/security/putty/Makefile @@ -1,12 +1,14 @@ PORTNAME= putty -DISTVERSION= 0.82 +# https://tartarus.org/~simon/putty-prerel-snapshots/putty-0.83~pre20250105.1e45199.tar.gz +DISTVERSION= 0.83~pre20250105 PORTREVISION= 0 -#DISTVERSIONSUFFIX= .0c59d49 +DISTVERSIONSUFFIX= .1e45199 CATEGORIES= security -MASTER_SITES= http://the.earth.li/~sgtatham/putty/${PORTVERSION}/ \ - ftp://ftp.chiark.greenend.org.uk/users/sgtatham/putty-latest/ \ +#MASTER_SITES= http://the.earth.li/~sgtatham/putty/${PORTVERSION}/ \ +# ftp://ftp.chiark.greenend.org.uk/users/sgtatham/putty-latest/ \ +# LOCAL/mandree/ +MASTER_SITES= https://tartarus.org/~simon/putty-prerel-snapshots/ \ LOCAL/mandree/ -#MASTER_SITES= https://tartarus.org/~simon/putty-prerel-snapshots/ MAINTAINER= mandree@FreeBSD.org COMMENT= Secure shell and telnet client including xterm emulator diff --git a/security/putty/distinfo b/security/putty/distinfo index 541b081ab0de..36dff882a785 100644 --- a/security/putty/distinfo +++ b/security/putty/distinfo @@ -1,3 +1,3 @@ -TIMESTAMP = 1734704014 -SHA256 (putty-0.82.tar.gz) = 195621638bb6b33784b4e96cdc296f332991b5244968dc623521c3703097b5d9 -SIZE (putty-0.82.tar.gz) = 2944148 +TIMESTAMP = 1736119462 +SHA256 (putty-0.83~pre20250105.1e45199.tar.gz) = 87a51fdfd06ec78f1ca8759cbb81da4e05ca4e497d6127fa32143709648f4923 +SIZE (putty-0.83~pre20250105.1e45199.tar.gz) = 3009307 diff --git a/security/putty/files/patch-296b6291d39c0cf118cd3081c3ab86a5889eb4d9 b/security/putty/files/patch-296b6291d39c0cf118cd3081c3ab86a5889eb4d9 deleted file mode 100644 index 30de85c82802..000000000000 --- a/security/putty/files/patch-296b6291d39c0cf118cd3081c3ab86a5889eb4d9 +++ /dev/null @@ -1,33 +0,0 @@ -From: Simon Tatham -Date: Sat, 7 Dec 2024 19:28:44 +0000 (+0000) -Subject: Fix error message when KEXINIT negotiation fails. -X-Git-Url: https://git.tartarus.org/?p=simon%2Fputty.git;a=commitdiff_plain;h=7da3449586ea3e6faaa92663d32774e28cf4e2e3;hp=296b6291d39c0cf118cd3081c3ab86a5889eb4d9 - -Fix error message when KEXINIT negotiation fails. - -By putting the wrong error-type enum value in a ScanKexinitsResult, I -accidentally caused nonsense messages of the form - - Selected key exchange algorithm "foo,bar,baz" does not correspond to any supported algorithm - -where "foo,bar,baz" is the full comma-separated list sent by the -server, so it's not even _an_ algorithm as the message suggests. - -Now the message is the one it should have been all along: - - Couldn't agree a key exchange algorithm (available: foo,bar,baz) ---- - -diff --git a/ssh/transport2.c b/ssh/transport2.c -index 5dd73cfe..b8e0d1c6 100644 ---- a/ssh/transport2.c -+++ b/ssh/transport2.c -@@ -1166,7 +1166,7 @@ static ScanKexinitsResult ssh2_scan_kexinits( - * Otherwise, any match failure _is_ a fatal error. - */ - ScanKexinitsResult skr = { -- .success = false, .error = SKR_UNKNOWN_ID, -+ .success = false, .error = SKR_NO_AGREEMENT, - .kind = kexlist_descr[i], .desc = slists[i], - }; - return skr; diff --git a/security/putty/files/patch-6a88b294276b9c24584efa18b9a37f437fa46712 b/security/putty/files/patch-6a88b294276b9c24584efa18b9a37f437fa46712 deleted file mode 100644 index 37bda1c8a77a..000000000000 --- a/security/putty/files/patch-6a88b294276b9c24584efa18b9a37f437fa46712 +++ /dev/null @@ -1,50 +0,0 @@ -From: Simon Tatham -Date: Sat, 7 Dec 2024 09:37:15 +0000 (+0000) -Subject: GTK: fix a crash when clicking Cancel on Change Settings. -X-Git-Url: https://git.tartarus.org/?p=simon%2Fputty.git;a=commitdiff_plain;h=296b6291d39c0cf118cd3081c3ab86a5889eb4d9;hp=6a88b294276b9c24584efa18b9a37f437fa46712 - -GTK: fix a crash when clicking Cancel on Change Settings. - -I only observed this in the GTK1 build, but I don't know for sure it -can't happen in other situations, so there's no reason not to be -careful. - -What seems to happen is that when the user clicks Cancel on the Change -Settings dialog box, we call gtk_widget_destroy on the window, which -emits the "destroy" signal on the window, our handler for which frees -the whole dlgparam. But _then_ GTK goes through and cleans up all the -sub-widgets of the dialog box, and some of those generate extra -events. In particular, destroying a list box is done by first deleting -all the list entries - and if one of those is selected, the list box's -selection changes, triggering an event which calls our callback that -tries to look up the control in the dlgparam we just freed. - -My simple workaround is to defer actually freeing the dlgparam, via a -toplevel callback. Then it's still lying around empty while all those -random events are firing. ---- - -diff --git a/unix/dialog.c b/unix/dialog.c -index 835ad978..fa645b3a 100644 ---- a/unix/dialog.c -+++ b/unix/dialog.c -@@ -3345,9 +3345,18 @@ static void dlgparam_destroy(GtkWidget *widget, gpointer data) - sfree(dp->selparams[i]); - } - sfree(dp->selparams); -+ dp->selparams = NULL; - } - #endif -- sfree(dp); -+ /* -+ * Instead of freeing dp right now, defer it until we return to -+ * the GTK main loop. Then if any other last-minute GTK events -+ * happen while the rest of the widgets are being cleaned up, our -+ * handlers will still be able to try to look things up in dp. -+ * (They won't find anything - we've just emptied it - but at -+ * least they won't crash while trying.) -+ */ -+ queue_toplevel_callback(sfree, dp); - } - - static void messagebox_handler(dlgcontrol *ctrl, dlgparam *dp, diff --git a/security/putty/files/patch-7da3449586ea3e6faaa92663d32774e28cf4e2e3 b/security/putty/files/patch-7da3449586ea3e6faaa92663d32774e28cf4e2e3 deleted file mode 100644 index 8c2427e006f9..000000000000 --- a/security/putty/files/patch-7da3449586ea3e6faaa92663d32774e28cf4e2e3 +++ /dev/null @@ -1,36 +0,0 @@ -From: Simon Tatham -Date: Wed, 4 Dec 2024 12:02:05 +0000 (+0100) -Subject: Fix use of aligned_alloc() to be ASan-clean. -X-Git-Url: https://git.tartarus.org/?p=simon%2Fputty.git;a=commitdiff_plain;h=c2d7ea8e67c462341e16d74e7a0ea42edd514635;hp=7da3449586ea3e6faaa92663d32774e28cf4e2e3 - -Fix use of aligned_alloc() to be ASan-clean. - -aligned_alloc() is used by testsc for all its memory allocation, to -avoid false-positive timing variations that depend on memory alignment -rather than actual secret data. But I'd forgotten that aligned_alloc -requires the allocation size to be a multiple of the requested -alignment. - -This showed up when I ran testsc in dry-run mode, and my normal build -happened to be using ASan, which complains at the invalid allocation -size. But it was theoretically a problem in all builds of -testsc. (Though, as far as I'm aware, not practically; and it _only_ -affected testsc.) ---- - -diff --git a/utils/memory.c b/utils/memory.c -index 0ba791ad..590be002 100644 ---- a/utils/memory.c -+++ b/utils/memory.c -@@ -35,7 +35,10 @@ void *safemalloc(size_t factor1, size_t factor2, size_t addend) - #ifdef MINEFIELD - p = minefield_c_malloc(size); - #elif defined ALLOCATION_ALIGNMENT -- p = aligned_alloc(ALLOCATION_ALIGNMENT, size); -+ /* aligned_alloc requires the allocation size to be rounded up */ -+ p = aligned_alloc( -+ ALLOCATION_ALIGNMENT, -+ (size + ALLOCATION_ALIGNMENT - 1) & ~(ALLOCATION_ALIGNMENT-1)); - #else - p = malloc(size); - #endif diff --git a/security/putty/files/patch-c72a86272446c0e4fb33c68601563549044b29e6 b/security/putty/files/patch-c72a86272446c0e4fb33c68601563549044b29e6 deleted file mode 100644 index ca26d6758f9d..000000000000 --- a/security/putty/files/patch-c72a86272446c0e4fb33c68601563549044b29e6 +++ /dev/null @@ -1,40 +0,0 @@ -From: Simon Tatham -Date: Thu, 28 Nov 2024 18:30:48 +0000 (+0000) -Subject: Fix a build failure with NO_GSSAPI defined. -X-Git-Url: https://git.tartarus.org/?p=simon%2Fputty.git;a=commitdiff_plain;h=8805cf3d9a1bb39c190345b9820ecefa9cfe801d;hp=c72a86272446c0e4fb33c68601563549044b29e6 - -Fix a build failure with NO_GSSAPI defined. - -The stub no-gss.c still wanted to know the layout of the -ssh_gss_liblist structure, in order to fill it in with nothing. ---- - -diff --git a/ssh/gss.h b/ssh/gss.h -index c819d48b..d11a359f 100644 ---- a/ssh/gss.h -+++ b/ssh/gss.h -@@ -3,6 +3,13 @@ - #include "putty.h" - #include "pgssapi.h" - -+/* This struct is defined even in NO_GSSAPI mode, so that stubs/no-gss.c can -+ * return an instance of it containing no libraries */ -+struct ssh_gss_liblist { -+ struct ssh_gss_library *libraries; -+ int nlibraries; -+}; -+ - #ifndef NO_GSSAPI - - #define SSH2_GSS_OIDTYPE 0x06 -@@ -49,10 +56,6 @@ struct ssh_gss_library; - * The free function cleans up the structure, and its associated - * libraries (if any). - */ --struct ssh_gss_liblist { -- struct ssh_gss_library *libraries; -- int nlibraries; --}; - struct ssh_gss_liblist *ssh_gss_setup(Conf *conf); - void ssh_gss_cleanup(struct ssh_gss_liblist *list); - diff --git a/security/putty/files/patch-f8e1a2b3a934d750aba7c26d182f52d71952c529 b/security/putty/files/patch-f8e1a2b3a934d750aba7c26d182f52d71952c529 deleted file mode 100644 index b2b67f41e4f7..000000000000 --- a/security/putty/files/patch-f8e1a2b3a934d750aba7c26d182f52d71952c529 +++ /dev/null @@ -1,43 +0,0 @@ -From: Simon Tatham -Date: Sat, 14 Dec 2024 11:44:28 +0000 (+0000) -Subject: Fix assertion failure on Restart Session. -X-Git-Url: https://git.tartarus.org/?p=simon%2Fputty.git;a=commitdiff_plain;h=edd5e13ffc976025443e0b9d75888249aa3325a9;hp=f8e1a2b3a934d750aba7c26d182f52d71952c529 - -Fix assertion failure on Restart Session. - -This occurred if the SSH server closed the connection for any -reason (in practice usually a timeout, but reproducible more easily by -manually killing a test server process) while the user was in the -middle of any kind of interactive prompt-based login in the GUI PuTTY -terminal (be it simple password, k-i, private key passphrase, -whatever). - -The problem was that term->userpass_state wasn't cleaned up when the -connection died, and then if you started a fresh SSH session in the -same terminal, the attempt to create a new term->userpass_state would -find there was one already there. - -The simplest place to insert the missing cleanup is the call to -term_provide_backend(), because that's a terminal API function which -is already called to notify the terminal that one backend has gone -away and the next one has turned up. - -(In fact, it's called twice, once to set term->backend to NULL when -the first session closes, and again when the session is restarted. I -see no harm in making the cleanup unconditional, not bothering to tell -the difference between the two cases.) ---- - -diff --git a/terminal/terminal.c b/terminal/terminal.c -index e127ff6e..2db81c9a 100644 ---- a/terminal/terminal.c -+++ b/terminal/terminal.c -@@ -2374,6 +2374,8 @@ void term_resize_request_completed(Terminal *term) - void term_provide_backend(Terminal *term, Backend *backend) - { - term->backend = backend; -+ if (term->userpass_state) -+ term_userpass_state_free(term->userpass_state); - if (term->backend && term->cols > 0 && term->rows > 0) - backend_size(term->backend, term->cols, term->rows); - }