Re-allow PAMServiceName inside a Match block.

This is documented to be allowed, and was up until 10.4p1 when a
transcription error in the config handling rework reverted it to
global-only, ie not allowed inside Match.  bz#3987.

Rerun VM tests on regress Makefile change too

Run tests on regress Makefile changes too.

upstream: disable pubkey-priority test until I can fix it to not pick

up the user's keys from ~/.ssh

OpenBSD-Regress-ID: 5295a8cefe1d2def5c8b10f9d60d9bb7a350d316

upstream: Fix expected output for implicit ordering test. ok djm@

OpenBSD-Regress-ID: 3ba8907d381fb738578ba2090d4b2f22d35a555a

Don't enable Brainpool on AWS-LC or BoringSSL.

These fail at runtime, so disable until we figure out why.

Correct filename in comment.

Fix OpenSSL ECC detection message.

Make EC_KEY_METHOD_new optional again.

The change making ECC required also incorrectly made EC_KEY_METHOD_new
required, causing builds with BoringSSL to fail.

upstream: crank up MaxAuthTries to avoid failures here

OpenBSD-Regress-ID: 052b0c4cc8bd6ea2679fbc9f96f4cb7f48c73035

upstream: regression test for pubkey ordering in the client; ok

dtucker

OpenBSD-Regress-ID: e55fa977be300307e6e03a001716b3dce8faa1a1

upstream: Tweak ordering of certificates tried during pubkey

authentication to prefer FIDO keys that do not require user presence (touch)
first, followed by ones that do and with FIDO keys that require user
verification via PIN or biometrics last.

This effectively tries low-friction authenticators before higher
friction authenticators.

ok dtucker@

OpenBSD-Commit-ID: 067076c652dbb4f2aa89c6e82b19d8cbfec1d1be

upstream: Add a "ssh -Z user@host" mode that dumps the pubkeys that

will be tried for authentication in the order that they will be used.

feedback tb@ ok dtucker@

OpenBSD-Commit-ID: 617a7f149d1410980eac551e4780280190a5f218

upstream: fix ML-KEM/ECDH interop problem (does not affect

mlkem768x25519-sha256) from markus@

OpenBSD-Commit-ID: 9c72e9c0407906bb7d5fb16648b38282676391e9

Require NISTP521 in libcrypto.

Previously we supported building without NISTP521 in libcrypto since
some vendors shipped without it, but this now seems uncommon. This
removes that support, requiring a fully functional libcrypto.

This removes another set of diffs vs upstream and should reduce the
maintenance burden.  ok djm@

Remove support for libcrypto w/out ECC.

Since 2010, we have supported building against an OpenSSL that did not
have any ECC support, since at the time many systems still shipped with
OpenSSL versions prior to ECC being added.  We no longer support OpenSSL
versions that old, and while it's possible to build a modern libcrypto
without ECC, doing so seems very uncommon.

Having OPENSSL_HAS_ECC sprinkled throughout the code increases the
cost of syncing changes and is a maintenance burden, so this has now
outlived its usefulness and is being removed.  It is still possible to
build --without-openssl on such systems if required.

This includes a few whitespace and header order changes to bring
Portable back into sync with OpenBSD in those places, and removes the
corresponding test.

ok djm@

Add includes.h for compat interfaces.

upstream: Use ssh_mldsa44_ed25519_cleanup consistently to avoid

calling freezero with the wrong size.

With help from tb@

ok tb, dtucker@

OpenBSD-Commit-ID: 4de18532e448a62005e12c179786bec512b83fde

upstream: Fix $OpenBSD marker for easier syncing.

OpenBSD-Commit-ID: 1f38203c268007964b0abdfa4e024d572c65a898

upstream: ssh: update kex unit test

OpenBSD-Regress-ID: db6f1a538803716ce225e2128046de0b6ab6d2aa

upstream: Make mlkem768ecdh build with OPENSSL=no. ok djm@

OpenBSD-Commit-ID: b23fc98b885d72b57d9a2b989f6c062751bff52b

upstream: pass back errors from ed25519 key generation, which

theoretically can fail. From Dimitri John Ledkov via GHPR702.

ok deraadt@ dtucker@

OpenBSD-Commit-ID: 1ecdd981c3f89393092949ff5e96662946fc5040

upstream: Move check of public key type against allowed algorithms

to before parsing of the key sent by the peer.

This removes at least some key parsing and verification paths from
the pre-auth attack surface.

Suggested by Christopher Paul Rohlf of Anthropic, ok deraadt@

OpenBSD-Commit-ID: 4560b53b5cd2fe9b875436e48995a1829c2a9dea

upstream: Fix double frees (impossible to reach outside of a test

harness), and also use freezero where possible from Christopher Paul Rohlf at
Anthropic ok djm

OpenBSD-Commit-ID: 50227bff4ffd4a93aad399f21fc20f1c3cbf2c69

upstream: ssh: implement mlkem768brainpoolp256r1-sha256; ok djm@

OpenBSD-Commit-ID: 99a304085f96d9a595b326c06694290c5612620c

upstream: ssh: implement mlkem768nistp256-sha256; ok djm@

OpenBSD-Commit-ID: f5daafce2fcb44684606197873dab4cd189524a9

upstream: use sshpkt_fatal instead of plain fatal() for errors in

the packet code as this provides context of the failing peer (address, port,
user, etc). Based on patch from Dag-Erling Smørgrav
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

OpenBSD-Commit-ID: 2e50ab08ab697722230f5d7dbebc9ea3c4f2931b

upstream: Automatically detect IPv6 support

and test IPv6 parsing if found. This should always be enabled on OpenBSD,
but allows us to use the same test in Portable without modifications
that make syncs harder.

OpenBSD-Regress-ID: 80dce2465e9414695d878a9af18a3d75711f7861

upstream: Factor out all of the IPv6 addresses into variables.

This lets us skip them in Portable on platforms that don't support IPv6
all in one place, removing diffs from within the tests themselves and
making syncs easier.

OpenBSD-Regress-ID: be5d2d67c28f8134f84e8baab09f303be8ceb626

Sync missed case-insensitivity changes.

Add obsd79 target.

Wording.

upstream: verify that ChannelTimeout in a Match block is effective

OpenBSD-Regress-ID: 30770cad71ca060aaaa05e4bdd0ea8941b768c6b

upstream: verify that RekeyLimit in a Match block is effective

OpenBSD-Regress-ID: 23084bcf86071a7fe0c121552ec6e0b208cc7e28

upstream: Use -Ohashalg to restrict output to specfic hash instead

of awk.

OpenBSD-Regress-ID: aa5fac0e3ee8f518794d3361d537090c6e3f0bec

upstream: fix ChannelTimeout and RekeyLimit not being applied in

sshd_config Match blocks; reported by Alex Harrison

OpenBSD-Commit-ID: 2d8866b841fc92e6e079e3f37590ba5948531b3d

upstream: s/= - 1/= -1/; from serity---

OpenBSD-Commit-ID: d003b300b0062d8e5951b84e8e09bd8d98cfe562

upstream: move to a better place

OpenBSD-Commit-ID: 0c70f26de19babb2557a7a95ae7057d996a2c3f8

upstream: use crypto_api.h #define constants for lengths; no

binary change

OpenBSD-Commit-ID: 6527baa1f07b7fdf42ca84531a13ae3ff2c0dbc8

upstream: make authorized_keys "restrict" keyword apply correctly

to tunnel forwarding (which is administratively disabled by default).

Reported by Erichen, Institute of Computing Technology,
Chinese Academy of Sciences

OpenBSD-Commit-ID: 5b3cc987a64749c94b20e12755db32a83f8f01e6

upstream: in sshd config dump mode, write all directives in mixed

case for consistency

ok djm@

OpenBSD-Commit-ID: 55647b13194d0aaa7095b89455d4c44ddeb53e7d

upstream: read ~/.ssh/id_mldsa44_ed25519 private key files by

default

ok djm@

OpenBSD-Commit-ID: c45683d341d7dce6c126903bf9a37393f2b75839

upstream: document mldsa44-ed25519 host key and public key

algorithm

OpenBSD-Commit-ID: c519ba7408cfb2700d184c9441de4ff01ecda726

upstream: when signing hostkey proofs for a client UpdateHostKeys

request, allow each hostkey to perform at most one signature operation. ok
dtucker@

OpenBSD-Commit-ID: ad4149015634f8156ba723656035ec26140875e8

upstream: setproctitle(3) to identify sshd-session when its

acting as a post- authentication monitor; ok dtucker@

OpenBSD-Commit-ID: a3c36a005a61ccaeb974afd7b9290b826e1620ba

upstream: delete list of ssh_config(5) options under

documentation of -o and instead just direct readers to the actual
ssh_config(5) manpage.

ok deraadt@

OpenBSD-Commit-ID: bed2058af847c2149e0e457202a8c4ee7000ad33

upstream: enable the ssh-mldsa44-ed25519@openssh.com signature

scheme (and its corresponding certificate form) in the lowest-priority
position.

"what took you so long" deraadt@

OpenBSD-Commit-ID: b7be74df494323a7021cf230f11eff824d2810d0

upstream: prefer fstat to stat when it's trivial to do so

OpenBSD-Commit-ID: 3af5548ba2112045db392a14c959ca309605bdb9

upstream: Add tests for ecdsa 256, 384 and 521 keys in DNS

fingerprints. Dynamically generate the required zone file.

OpenBSD-Regress-ID: 61acdf25efc8c5d3bb0156fd3a53bf8159d3e13a

upstream: check sshd_config output case insensitively

OpenBSD-Regress-ID: dafa6c3e723cf39ca5e552304372bc085c348102

upstream: Add SSHFP tests for ed25519 key type.

OpenBSD-Regress-ID: a4bc60d0f398b148a29df3594d5f36de0e4e5ea2

upstream: Restructure the SSHFP test in preparation for adding

other key types:  - change the DNS names to be rsa.* for the existing RSA
fingerprints.  - verify that all required SSHFP records exist in DNS.  - only
run the RSA tests if the build supports RSA.

OpenBSD-Regress-ID: 03e4087c3bd09ad8bede788f76f0ab59b732639e

remove README.tun

it's badly out of date, has already been deleted upstream and is not
well-adapted for portable.

add some parentheses to clarify a && || condition

Spotted by Harish Yadav

upstream: unused variables

OpenBSD-Commit-ID: 03fc22fb427b7547ee7844907cf3257bce7fdc3c

upstream: fix GSSAPI option names, that I somehow screwed up while

refactoring servconf.c bz3974 patch from Colin Watson

OpenBSD-Commit-ID: be39ad3dbe36d9ecdb86f3811da5dfbdc9bcb1e6

Add 10.4 branch to status page.

depend

crank version numbers

upstream: openssh-10.4

OpenBSD-Commit-ID: ce7b0749e5139c70410ee92a13d368d7d34262b5

upstream: Fix multiple RFC 4462 (GSSAPIAuthentication) compliance

problems

1) Remove an early failure return for GSSAPI authentication attempts
made for invalid accounts that yielded different behaviour for
valid vs invalid accounts.

2) Fix a situation where some GSSAPI requestes were not correctly
subjected to MaxAuthTries.

3) Fix a moderate pre-authentication resource DoS related to #2.

Add missing logging for error cases.

Report and fixes from Manfred Kaiser, milCERT AT

OpenBSD-Commit-ID: ca0acdd64eea435d6f89534538a9eb404a5629d3

upstream: fix ownership and lifetime of several bits of client

state that need to persist for the life of the connection, especially the
cached hostkey that was being incorrectly freed early on some paths, possibly
allowing its use after free.

Reported by Zhenpeng (Leo) Lin from depthfirst.com

OpenBSD-Commit-ID: faaa6ad72e7d69d41fa8b197b606265b7d9bc73f

upstream: Fix cases in GSSAPI and keyboard-interactive

authentication where the minimum per-attempt delay was not being enforced.

Reported by Orange Cyberdefense Vulnerability Team

OpenBSD-Commit-ID: c40bd35cc2428fcaccad7a141703c28baa6da01e

upstream: void functions should not return anything. Patch from Tim

Rice.

OpenBSD-Commit-ID: bb5021b2b45d9d3f54a012d569872805d107f59c

upstream: fix inverted test that broke ssh-add with keys on stdin. From

Laurence Tratt

OpenBSD-Commit-ID: bcef522cfa587c7cf035660bb347cff36b65bfbd

grammar fix; from Daniel O'Connor

more config option details in README.privsep

more README.privsep polish

tweak; from dlg@

tweak previous

revise README.privsep for multi-binary model

This rewrites most of the privsep description to more accurately capture
recent changes in how sshd managed privilege across its lifecycle,
including describing the roles of the sshd-session and sshd-auth
helper binaries.

Also 100% more ASCII art

Need clang >= 19 for constexpr in hardened_malloc.

Tabs -> spaces.

Provide better error for non-supported private keys

Signed-off-by: Zoltan Fridrich <zfridric@redhat.com>

upstream: more missing mldsa44-ed25519, based on GHPR696 from Loganaden

Velvindron

OpenBSD-Commit-ID: f32e17df54fa66b0e936f1aeb3963f949e4a9bb8

upstream: whitespace

OpenBSD-Commit-ID: 730e54174b2182c7011cc87a0fdae595cd9bcc2d

upstream: simplify SIGINFO output: remove list of active channels (too

verbose) and just display destination and connection duration; requested
deraadt@

OpenBSD-Commit-ID: cb36192cd53483f8e452ea91bc42be7bbb2fffa3

upstream: Tighten up the introduction a little:

Mention Match as a conditional directive (previously it only
mentioned Host)

Try to use consistent language in the introduction to refer to
configuration directives (previously it used "parameters" and
"keywords" interchangeably).

Mention that comments may appear at the end of the line too, and that
whitespace at the beginning/end of lines is not significant.

OpenBSD-Commit-ID: e08f34413eeced47478c14d5005726f9bdd5d80e

upstream: Move negative-FD checks to before first use. CID 909998,

ok djm@

OpenBSD-Commit-ID: f0208911d27fd5bd66cd608c7b6fb5a36002d6a5

upstream: ssh -o doesn't support Host or Include options, they are only

valid in the config file. bz3968 from xspielinbox

OpenBSD-Commit-ID: cbecf11f2eea63e3cf6752e9bbe59091ba60cd32

upstream: mention mldsa44-ed25519 in usage(); based on GHPR695 from

Loganaden Velvindron

OpenBSD-Commit-ID: 4af2400312b6ca232a68a758160ca5c64ca6a168

upstream: ssherr-libcrypto: avoid use of deprecated

ERR_load_crypto_strings()

Follow regress and use unchecked OPENSSL_init_crypto() because
ERR_load_crypto_strings() has been deprecated in OpenSSL 1.1.0.
This call can in principle fail, in which case there's a fallback
to generic error strings.

There is still use of OpenSSL 3.x-deprecated API in here. That's
a problem for someone else to solve.

ok djm

OpenBSD-Commit-ID: 3e9dc94caa83b361775b83a92425e858f8680c05

upstream: another ruser_name/ruser_group vs attrib_to_stat() ordering

screwup. Coverity CID 910530 via dtucker@

OpenBSD-Commit-ID: d8c4656119f09304e79fcf2ab32299ed68006a29

upstream: set FD_CLOEXEC on the fds between sftp and its ssh

process, avoids risk of subcommands that write on odd fds breaking the
connection. GHPR693 from Manuel Einfalt; feedback deraadt@ ok dtucker@

OpenBSD-Commit-ID: 91b12cd66580dab6b3bfe5e507a1a8310c8f9149

upstream: check key and IV length received in privsep state

transfer exactly match the expected sizes for the selected cipher; partially
redundant to similar checks in cipher_init(), but nice to be more exact.

GHPR from jmestwa-coder; ok dtucker@

OpenBSD-Commit-ID: d4a9deef194cc44189e835ac59b7b800b9b5c281

upstream: revert bits that weren't ready for commit yet

OpenBSD-Commit-ID: ee8a219f02db32778444356ad2d93b983a38a704

upstream: Move user/group name lookup to correct place; coverity

CID 910530 via dtucker@

OpenBSD-Commit-ID: 03ba9f8c720eea38436e4fef4c40814eae1b1fe3

upstream: fix ineffective max file size check when loading

blobs/keys from files and add another one on a patch that was not covered by
the existing ones. From Tess Gauthier via bz3969 and bz3970

OpenBSD-Commit-ID: c0dec6c587853349113df85b6dc528dc15079af0

upstream: check strdup() return to avoid NULL deref on failure.

bz3948 from RuiHe-MO

OpenBSD-Commit-ID: 2b1fbfb2e1f3359150feadfdf05acaa1c7d211e8

upstream: s/calloc/xcalloc/ to reduce noise from AI bug detectors

that don't understand context

OpenBSD-Commit-ID: dcef5b1804620f2aed108267bbf5023a81230e14

upstream: fix "ls -n", which was still displaying user/group names

rather than numeric uids/gids. Based on patch from Tamilan in bz3953

OpenBSD-Commit-ID: 65dd2ecb870b727e872cf9df544f8767426d2cc0

upstream: move documentation of the Include directive to near the

start of the options list, alongside that for Match and Host which are
similar insofar as they all affect configuration parsing rather than altering
the configuration directly. from xspielinbox via bz3968

OpenBSD-Commit-ID: 1cde8af8d40dbbe3ea49cf56c5d408946c875230

upstream: mention RefuseConnection, VersionAddendum and

WarnWeakCrypto along with other options. from xspielinbox via bz3968

OpenBSD-Commit-ID: 5d4ef72dc703a74e64175e4611961ea338379d66

upstream: correct directive name (s/Host/Match) in error message

from xspielinbox via bz3968

OpenBSD-Commit-ID: 3aec5cc9d349bcef12abc951490474f1217b4aeb

upstream: report errors in fill_default_options() properly, based on

GHPR649 by Zoltán Fridrich
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

OpenBSD-Commit-ID: 10137e31df74c3100e3abc7d008b1645ca5b45af

upstream: don't use deprecated ERR_load_crypto_strings()

GHPR650 from Mike Frysinger

OpenBSD-Regress-ID: bfe86d9be1db4569c091f3d763cba04ddbb86ec0

don't leak rrset on fail; GHPR681 from metsw24-max

ok dtucker

check sockaddr length; GHPR681 from metsw24-max

ok dtucker

return result of raise(2); GHPR681 from metsw24-max

ok dtucker

use size_t for lengths; GHPR681 from metsw24-max

ok dtucker