[wip] merge#842
Open
tgauth wants to merge 475 commits into
Open
Conversation
Remove #ifdef HAVE_SYS_TIME_H wrapper. ok djm@
Remove #ifdef HAVE_SYS_STAT_H wrapper. ok djm@
Remove #ifdef HAVE_TIME_H wrapper. ok djm@
Remove #ifdef HAVE_SYS_UN_H wrapper. ok djm@
Remove #ifdef HAVE_LIBGEN_H wrapper. ok djm@
Remove #ifdef HAVE_NETGROUP_H wrapper. ok djm@
Removes diffs vs upstream.
Remove #ifdef HAVE_ENDIAN_H wrapper. ok djm@
Remove #ifdef HAVE_NLIST_H wrapper. ok djm@
removed in 2017. OpenBSD-Commit-ID: 937fca21498b921adf6e04bac120f4a2e7975b3c
-portable. OpenBSD-Commit-ID: 2061307dc938712e524bc9da48a52f545e43670e
OpenBSD-Commit-ID: db72be57429418f6a4319bbe34c98fc103e11ce0
OpenBSD-Commit-ID: 7156b69b0364c68e181e0f6fa17c0f05c72e8670
OpenBSD-Commit-ID: 4df519fd9fa13ce9653adf7a3d1076e20591d886
vs portable. OpenBSD-Commit-ID: 38043f0bfa17c48ef6d1a744c2834b4405bc9311
Check for nlist function presence before attenmpting to use it instead of relying on the presence of the nlist.h header. Mac OS X, in particular has the header, but only has the function in the 32bit libraries.
Otherwise tun forwarding from other OSes fails as soon as the first IPv6 message is sent by the other side (which is usually a Router Solicitation ICMPv6 message which is sent as soon as the interface is up): all other OS'es use SSH_TUN_COMPAT_AF or SSH_TUN_PREPEND_AF which effectively uses OpenBSD AF_INET/AF_INET6 values.
They seem to work, at least for now.
Fix for when building out of tree.
OpenBSD-Commit-ID: 06598021a9f08188dab29ac956b2baa002a0ff85
OpenBSD-Commit-ID: 7e8178786157e863f6ff63c5d55200d7b6b04f9e
the certificate expiry time plus a short (5 min) grace period. This will cause the agent to automtically remove certificates shortly after they expire. A new ssh-add -N option disables this behaviour. Feedback/ok deraadt@ OpenBSD-Commit-ID: 92fed1bba1025069ad45deebb534be7530e181df
…merge-v10.3P1-20260420
…merge-v10.3P1-20260420
…merge-v10.3P1-20260420
…merge-v10.3P1-20260420
…merge-v10.3P1-20260420
Previously Get-CommitCIStatus/Get-CommitDetails caught all exceptions (including HTTP 403 rate-limit) and returned unknown/null, so a rate-limited scan silently continued and emitted a batch from incomplete CI data. Added Test-IsRateLimitError + Invoke-GitHubApi wrapper that throws a marked terminating error on rate-limit, a proactive /rate_limit budget pre-check, and a per-commit loop guard that exits non-zero on mid-scan exhaustion.
…merge-v10.3P1-20260420
Upstream's KNF header-reordering commits removed #ifdef HAVE_*_H guards around system headers (paths.h, util.h, ifaddrs.h, netgroup.h, nlist.h, endian.h), making the includes unconditional. These headers do not exist on Windows, causing C1083 build errors. Restore the guards so the includes are skipped on Windows where the macros are undefined.
Upstream's unit-test benchmark framework (test_helper.c) and misc-agent.c call clock_gettime() unconditionally. Windows win32compat provided no clock_gettime, so the linker resolved the symbol from the vcpkg-auto-linked fido2.lib, pulling in fido2.lib(clock_gettime.c.obj) and emitting 'LNK4099: PDB fido2.pdb was not found' in every unit test (e.g. unittest-bitmap). Provide a real clock_gettime() in win32compat (plus clockid_t / CLOCK_REALTIME / CLOCK_MONOTONIC) so the symbol resolves from posix_compat.lib instead, removing the accidental fido2 dependency and the warning.
Collaborator
Author
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
Author
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Document the risk that upstream additions/modifications to a regress/*.sh file already containing 'if [ os == windows ]' blocks merge verbatim (no Git conflict) and can reintroduce Unix-only assumptions such as a plain diff instead of diff --strip-trailing-cr. Adds Pattern 5 and a Common Conflict Patterns bullet, using the cfgparse.sh MaxStartups case from the 10.3 merge as the example.
…merge-v10.3P1-20260420
…merge-v10.3P1-20260420
…merge-v10.3P1-20260420
…merge-v10.3P1-20260420
Normalize the 6 vm-*.yml workflows added in batch 80 to dispatch-only, matching the existing fork policy (c-cpp.yml, selfhosted.yml, upstream.yml). Comment out push/pull_request triggers and leave workflow_dispatch enabled.
…merge-v10.3P1-20260420
…merge-v10.3P1-20260420
…merge-v10.3P1-20260420
…merge-v10.3P1-20260420
Collaborator
Author
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Summary
PR Context