Try -lstdc++ for libcrypto before giving up.

BoringSSL recently added destructors to libcrypto, which requires
linking against libstdc++, so when checking for a working libcrypto if
at first the link fails, try again with -lstdc++ before giving up.

Remove BoringSSL rpath as it's statically linked.

upstream: Use fmprintf instead of logit for challenge-response name and

info to preserve UTF-8 characters where appropriate.  Prompted by github
PR#452, with & ok djm@.

OpenBSD-Commit-ID: e6361242329ec6925571478f60f4739726aad308

Add AWS-LC and BoringSSL as potential libcryptos.

Add self-hosted status to main README now it's public.

Remove anchor to specific release notes version.

Remove potentially leftover include compat shims.

If we don't need a specific shim, ensure it does not exist.  Prevents
confusion if configurations change or the directory is reused across
different platforms.

Check regress passwd is set before enabling kbdint.

upstream: Remove leftover debugging.

OpenBSD-Regress-ID: e778d76b21696a14db80f31b9e79601f2d7a9abf

Activate kbdint test on PAM configs.

Install libaudit-dev for --with-audit=linux test.

Enable BSM audit test on FreeBSD VMs.

Move BSM audit test to selfhosted runner.

The vmactions VM on Github does not have the required libraries
installed.

Increase riscv64 test coverage.

The machine running the tests has been replaced with a faster one.

Whitespace fix.

Add test coverage for all of the --audit= configs.

upstream: same treatment for remote/remote copies (i.e. scp -3):

adjust permissions on destination directory only if we created it or -p was
requested. bz3925

OpenBSD-Commit-ID: d977006df7b8330e06ceaa319383b347f1aca3ef

upstream: when uploading a directory using sftp/sftp (e.g. during a

recursive transfer), don't clobber the remote directory permissions unless
either we created the directory during the transfer or the -p flag was set.
bz3925 ok dtucker@

OpenBSD-Commit-ID: d66f40d01de05c9ec4029fab5413325301039b3a

upstream: make IPQoS first-match-wins in sshd_config as it's

intended to be bz3924

OpenBSD-Commit-ID: 42753eb8400ab09713c69ace6fa8bfdde133f942

upstream: remove duplicate includes; ok dtucker@

OpenBSD-Commit-ID: 6b9191bc1a0f4320c926d5ccd9f36b09f0f3bcaf

Restore utf8.h removed earlier as it's needed.

... for msetlocale prototype.

Removed duplicate includes; spotted by jsg@.

Remove "draining" of PAM prompts.

With the previous commit, both prompts and info/error error messages are
returned to keyboard-interactive immedately and none are accumulated, so
there will never be any un-drained prompts.  ok djm@

auth-pam: Immediately report interactive instructions to clients

SSH keyboard-interactive authentication method supports instructions but
sshd didn't show them until an user prompt was requested.

This is quite inconvenient for various PAM modules that need to notify
an user without requiring for their explicit input.

So, properly implement RFC4256 making instructions to be shown to users
when they are requested from PAM.

Closes: https://bugzilla.mindrot.org/show_bug.cgi?id=2876

fix duplicate PAM msgs, missing loginmsg reset

without this change in mm_answer_pam_account all messages added in
auth-pam.c sshpam_query(...) case PAM_SUCCESS end up sent here, then are
still sitting in the loginmsg buffer and printed a second time in
session.c do_login(...)

upstream: Reorder headers to match KNF and Portable.

ID sync only.

OpenBSD-Commit-ID: b7f9700d07b532eb3720f7bd722b952e31b1752f

upstream: Add basic test for keyboard-interactive auth.

Not enabled by default since it requires some setup on the host.

OpenBSD-Regress-ID: aa8a9608a2ea2e5aaa094c5a5cc453e4797cd902

upstream: remove unneeded includes; ok dtucker@

OpenBSD-Commit-ID: bba6e85492276c30c7a9d27dfd3c4c55fa033335

Remove obsolete comment referencing auth-chall.c.

It was removed in commit 6cb6dcff along with the rest of the SSH1 server
support.

auth-pam: Add an enum to define the PAM done status

Makes things more readable and easier to extend

auth-pam: Add debugging information when we receive PAM messages

Remove DragonFlyBSD workaround for sys/mount.h.

... since we're not not including it at all any more.

upstream: Replace <sys/mount.h> with <limits.h>

The former is a portability hassle, but it turns out the only thing we
need from it is PATH_MAX which we can get directly from limits.h.

OpenBSD-Commit-ID: ccfbbd678bef3a3930ae89da456645c3ee5f83c0

upstream: remove unneeded forward struct declaration ok djm@

OpenBSD-Commit-ID: a0c97e919667394bef8dbf31df72af3ba07542e9

upstream: very basic testing of multiple files in RevokedKeys and

RevokedHostkeys

OpenBSD-Regress-ID: 6cee76bcc4bd6840bc8d39dd0d32d724e1427aa7

upstream: support multiple files in a ssh_config RevokedHostKeys

directive bz3918; ok dtucker

OpenBSD-Commit-ID: 0ad2eacf836f912f347846ab84760799033dd348

upstream: support multiple files in a sshd_config RevokedKeys

directive bz3918; ok dtucker

OpenBSD-Commit-ID: 9fc58c4e676f8e9ed2e3a0da666242a17b8a55b2

upstream: Add includes used in Portable to reduce diffs.

OpenBSD-Commit-ID: 186c60cf2da0ddb075d5bc4879e87bbd8779b7e4

upstream: Remove unused sys/queue.h include.

OpenBSD-Commit-ID: 564f75672e27f1006f280614934eb304abe69167

upstream: Reorder includes and defines to match both KNF and

Portable.

OpenBSD-Commit-ID: f3f179c095f8e4787ded5f450e2842881f6b8ab2

Factor out RNG reseeding in to a single function.

sshd and sshd-session both reseed the RNG after a fork.  Move the
existing reseed_prngs() function into entropy.c and use for both.
Clean up entropy.h too.  ok djm@

Remove do_pam_chauthtok since it's no longer used.

upstream: Pass actual size of the buffer to hostname() instead of a

define that's probably the same.  ok millert@ djm@

OpenBSD-Commit-ID: 7c97b22439100b4193404ccfa1e5f539c5a8d039

upstream: De-underscore __inline__ to match -portable

(and every other use of it in ssh).  ID sync only.

OpenBSD-Commit-ID: 83c913d5e2345635bc5434167ed67cec5409d494

upstream: Remove references to skey auth which is long gone.

ID sync only.

OpenBSD-Commit-ID: 0c2340566c399f7f74fe4c5366394974cd6fd122

upstream: Remove unused OpenSSL includes,

that are no longer used, even when building with OPENSSL=yes.

OpenBSD-Commit-ID: e97e3e551ade9aee994b80a1d5851be6f32288e3

upstream: Use https for URLs.

ID sync only.

OpenBSD-Commit-ID: 85b2919e95e6d2bfdeddf5e3b0709fb5b6b4c438

upstream: Remove unused OpenSSL includes,

which are no longer used even when building with OPENSSL=yes.

OpenBSD-Commit-ID: 31adb21bf3f8f5c13cde59229f1b85c20f19a858

upstream: Remove now-unused SKEYQUERY enums from monitor_reqtype.

ID sync only.

OpenBSD-Commit-ID: dab93b58e69c754887507e5557a81a0b5b84d734

upstream: Remove now-unused openssl includes since sshd.c no longer

needs them, even when built with OpenSSL.

OpenBSD-Commit-ID: ceaa0394db1520e92d75c37eea58130d44ba93c9

Don't create sys/mount.h shim except on DragonFly.

Fixes build on Mac OS X.

Minor resync with upstream

Reorder definitions add whitespace to eliminate diffs vs upstream.

Factor out COMPATINCLUDES into its own variable.

Provide compat shims for sys/{mount.h,statvfs.h).

In addition to shimming on platforms that don't have them, we also need to
shim sys/mount.h on DragonFlyBSD since it uses its native STAILQ_ENTRYs
which our compat queues.h does not have, which causes sftp-server.o to
not build.  This is a little icky, but it limits the blast radius to
just one source file on only DragonFly.  ok djm@

Shim <sys/queue.h> and <sys/tree.h>.

This significantly reduces the diff vs upstream making future syncs
less painful.  ok djm@

upstream: Reorder headers according to KNF,

and pull in a few we don't have from Portable.

OpenBSD-Commit-ID: d83f6c75da7bfb16bbff40fd2133d6eba4aba272

upstream: Include sys/socket.h to match -portable,

eliminating one diff.

OpenBSD-Commit-ID: 7670fdf35b0c7aee41cd0d6ded86b4792e261f36

upstream: Reorder headers as per KNF.

OpenBSD-Commit-ID: 3e29fabe20422454fd5d77f85c853e1e557f2181

Remove openindiana VM test.

When it works it's by far the slowest (>1h to install packages) and the
package installation is flaky.  We can bring it back if their infra ever
improves.

Sync header order with upstream and KNF.

Sync whitespace with upstream.

Sync header order with upstream.

Remove generic check for getpagesize.

We have a more specific check later.

Test KERBEROS5=yes builds on OpenBSD.

upstream: Make ssh optionally build with Kerberos 5 against the

Heimdal port. This updates the Makefiles and repairs some bitrot in headers,
resyncing them against Portable.  To do this, "pkg_add heimdal" then "make
KERBEROS5=yes".  ok djm@

(ID sync only)

OpenBSD-Commit-ID: 31f95c9ba58aa7ba89264f1d80c79106042b1095

upstream: Remove sys/poll.h since we also have poll.h.

Also removes one line of diff vs portable.
(ID sync only).

OpenBSD-Commit-ID: 461bd0cd35bfad82bd06892ccb0ff0fac15d1d27

upstream: Move setting of user, service and style earlier since

-portable needs to use these when setting up PAM.  Removes two diffs vs
portable.

OpenBSD-Commit-ID: 8db130d42a3581b7a1eaed65917673d4474fc4fe

Move USE_SYSTEM_GLOB into a glob.h compat shim.

This moves the logic for selecting whether or not we can use the system
glob into configure, and if either don't have glob or can't use it, we
create the shim.  Removes several diffs vs upstream.

upstream: misc.h is needed for ForwardOptions in servconf.h.

OpenBSD-Commit-ID: b241d81c499e273fc2d81c82d5b7c7b280827416

Remove unused <netinet/in_systm.h> includes.

netinet/in_systm.h is no longer in upstream and anything that actually
needs it will get it from includes.h.

upstream: Also check for EWOULDBLOCK on system error. This is the

same as EAGAIN on OpenBSD so is a no-op but removes a diff making portable
syncs easier.  (ID sync only).

OpenBSD-Commit-ID: 68a5dcc5e2a506208c40396c6366f67bbf3b1dbe

upstream: Move ssherr.h to where portable needs it.

(ID sync only)

OpenBSD-Commit-ID: 0488ce85f24864186678dcac7c9973ca44bd2cd5

Move paths.h and poll.h includes to resync with upstream.

Move poll.h include to resync with upstream.

Resync minor format diffs with upstream.

Resync headers with upstream.

Resync with upstream (unused header and whitespace).

Fix ut_type for btmp records

According to man utmp ut_type is supposed to be only switched from
LOGIN_PROCESS to USER_PROCESS after succesfull authentication and this
is how sshd behaved before 671c44078.

Fixes: 671c44078 ("use construct_utmp to construct btmp records")
Signed-off-by: Artem Savkov <artem.savkov@gmail.com>

upstream: bit of webauthn support missed in previous commit

OpenBSD-Commit-ID: 9768454543ded01b7c61567fc5b3e78664346be2

upstream: Adjust Makefiles to include just-added

ssherr_libcrypto where necessary.

OpenBSD-Regress-ID: 53d179a2db3ab931f2aa0e5447cf20cb9787a8bb

upstream: Fetch the error reason from libcrypto

if available, append it to the corresponding ssh error message and
optionall print the libcrypto full error stack (at debug1).  with &
ok tb@ djm@ millert@ schwarze@

Note that the quality of errors obtainable from libcrypto is somewhat
variable, so these may be any of: useful, misleading, incomplete
or missing entirely. As a result we reserve the right to change
what is returned or even stop returning it if it does more harm than
good.

OpenBSD-Commit-ID: 1ad599ac3eeddbe254fec6b9c1cf658fa70d572e

Typographical Fixes

upstream: Typo fixes, mostly in comments.

From THE-Spellchecker via github PR#620.

OpenBSD-Commit-ID: 64929fafa3caae5a162f23257917ecf33f8a3764

Enable gss-auth tests on Kerberos test configs.

Add a GSSAPI authentication test

Split sudo out to its own install line.

Remove obsolete comments.

Run tests on older OmniOS version too.

Add OpenIndiana VM test target.

upstream: remove vestige of when we supported running without privsep

OpenBSD-Commit-ID: 5342c24d2330ef5ce357c294056f72b8123122c0

upstream: Implement missing pieces of FIDO/webauthn signature support,

mostly related to certificate handling and enable acceptance of this
signature format by default.  bz3748 GHPR624 GHPR625

Feedback tb / James Zhang; ok tb

OpenBSD-Commit-ID: ce3327b508086b24a3f7a6507aa5c49d8e9505e6

upstream: Implement "query" extension from

draft-ietf-sshm-ssh-agent

feedback jsg@, tb@; ok tb@

OpenBSD-Commit-ID: adb2b79473ff86ba781ed5ab2735c1437b590f07

upstream: Make it clear that DenyUsers/DenyGroups overrides

AllowUsers/AllowGroups. Previously we specified the order in which the
directives are processed but it was ambiguous as to what happened if both
matched. OK djm@

OpenBSD-Commit-ID: 6ae0ab52ff796b78486b92a45cd7ec9310e20f4e

upstream: In ssh(1), don't try to match certificates held in an

agent to private keys.

This matching is done to support certificates that were
loaded without their private key material, but is unnecessary for
agent-hosted certificate which always have private key material
loaded in the agent. Worse, this matching would mess up the
request sent to the agent in such a way as to break usage of these
keys when the key usage was restricted in the agent.

Patch from Thibault Cools via bz3752, ok dtucker@

OpenBSD-Commit-ID: ebfe37817dad4841c53339930565242ec683d726

upstream: If editline has been switched to vi mode (i.e. via "bind

-v" in .editrc), setup a keybinding so that command mode can be entered. Diff
originally from Walter Alejandro Iglesias with tweaks. Feedback from Crystal
Kolipe. ok djm

OpenBSD-Commit-ID: 5786e17ccd83573e2d86418023f9bc768223336a

upstream: Fill entropy in a single operation instead of hundreds.

The sntrup761 code we use from SUPERCOP fills entropy arrays 4 bytes at
a time.  On some platforms each of these operations has a significant
overhead, so instead fill it in a single operation and as a precaution
zero that array after it's used.

Analysis and code change is from Mike Frysinger via Github PR#621 with
feedback from djm@ and sed-ification from me.  ok djm@ beck@.

This change was submitted by Mike to SUPERCOP upstream so hopefully
future versions will already have it.

OpenBSD-Commit-ID: 0e85c82f79b1b396facac59e05b288c08048f15c

upstream: rewrite SOCKS4/4A/5 parsing code to use sshbuf functions

instead of manual pointer fiddling. Should make the code safer and easier to
read. feedback/ok markus@

OpenBSD-Commit-ID: 5ebd841fbd78d8395774f002a19c1ddcf91ad047

upstream: test the right thing, doofus

OpenBSD-Commit-ID: 31b2ec6e0b3dbd08c60ba2d969dd687cd80c25fd

upstream: avoid possible NULL deref if

ssh_packet_check_rekey_blocklimit() called before the encrypted transport is
brought up.

OpenBSD-Commit-ID: fb998ccbe59865e33a8ab6a6577f254d39bdc72f

upstream: unit tests for sshbuf_consume_upto_child()

OpenBSD-Regress-ID: 13cbd0370ebca7c61c35346b3e0356517719a447