No need to check as->established before calling pfkey_remove().

Remove the extra checks in the caller and simplify some code because of that.
OK tb@

Adding pkg_add from stdin hasn't worked in ages. It was something
we inherited from the initial implemention on FreeBSD which has
made no sense in years.

prompted by a diff by Johannes Thyssen Tishman
from espie

Report shifted keys like S-A as A not as S-A in mode 1 extended keys,
from Stanislav Kljuhhin.

Extend Log->system to support the same features as BaseState->system

In order to support privsep in tags, we need to be able to pass some
code values in child/parent.

from espie, tested by sthen, ok giovanni

Use auth_conf.method instead of auth_state.method. The latter is always 0.

Explicitly set peer.auth_conf.method, the auth_state value is never
initialized in the SE so bgpctl did always see 0, auth_conf.method
is on the other hand properly shared.
OK tb@

In merge_peers also copy over the auth_conf from the new peer to the old
since that one is kept.
OK tb@

Fix typo in comment
OK tb@

Adjust to the pfkey changes in bgpd.
OK tb@

Rework the pfkey and tcp md5 API to not depend on struct peer.

Instead use struct auth_config and struct auth_state in the pfkey calls
and those tcp_md5 calls where it matters.

This is preparation work to allow RTR to use TCP MD5 as well.
OK tb@

Add a way to make the preview larger in tree mode, GitHub issue 4124.

Make it more obvious which ruadd argument is the source and which one is
the destination by adding const to the source pointer.

OK tb@

Adjust ptrace interface to properly suport single threaded continue.

Introduce P_TRACESINGLE flag to instruct the trapped thread to not
wakeup the other threads (via single_thread_clear). This must be done
like this since ptrace must wake just the single thread to ensure it
runs first and gets the ps_xsig value from ptrace.

Modern gdb depends on this for multi-threaded processes, when a breakpoint
is hit gdb fixes up the trapping instruction and then single steps over
it with only that thread. After that single step gdb continues with all
threads. If all threads are run like now it is possible that one of the
other threads hits a breakpoint before the single step is done which results
in an assertion in gdb (because that is not expected).
OK mpi@

Use global cursor style and colour options for modes instead of default,
GitHub issue 4117.

remove unused variable

drm: Expand max DRM device number to full MINORBITS

From Michal Winiarski
e615cd84dcf834e83b333bfb690fc2032b3fdb85 in linux-6.6.y/6.6.53
071d583e01c88272f6ff216d4f867f8f35e94d7d in mainline linux

accel: Use XArray instead of IDR for minors

From Michal Winiarski
f6b589e361538285fdad8cf62143e3cf3b2c8b2a in linux-6.6.y/6.6.53
45c4d994b82b08f0ce5eb50f8da29379c92a391e in mainline linux

drm: Use XArray instead of IDR for minors

From Michal Winiarski
d2e3d344e20e605378556610811549249298d0aa in linux-6.6.y/6.6.53
5fbca8b48b3050ae7fb611a8b09af60012ed6de1 in mainline linux

add DEFINE_XARRAY_ALLOC for 6.6.53 drm

add MINORBITS for 6.6.53 drm

add ACCEL_MAX_MINORS for 6.6.53 drm

Change pasting to bypass the output key processing entirely and write
what was originally received. Fixes problems with pasted text being
interpreted as extended keys reported by Mark Kelly.

change release date

Improve some currently impossible error path in log_ext_subtype().
Mainly handle unknown ext-communities better and handle the special
case of type == -1.
OK tb@

Use ps_ppid instead of ps_pptr->ps_pid in all places.
OK mpi@

handle non xa_limit_32b ranges in xarray

store xarray flags for XA_FLAGS_ALLOC1 test in alloc

Remove code after exit1() and NOTREACHED comment. Nothing will ever get there.
OK mpi@

In bgpd_rtr_connect() do the setsockopt dance for IP_TOS and TCP_NODELAY
before calling connect(). This way it happens for sure and on top the TOS
is already set on the initial SYN.
OK tb@

On some Windows terminals, if TIOCWINSZ does not return xpixel and
ypixel (they are zero), if this is the case then try the query escape
sequences. From Dmitry Galchinsky in GitHub issue 4099.

Return the number of freed pages and handle SHRINK_STOP in drmbackoff().

ok jsg@

Only use default-shell for popups, return to /bin/sh for run-shell,
if-shell and #() - these have been documented as using /bin/sh for a
long time and scripts rely on it. Pointed out by Gregory Pakosz.

init xarray pool in drm_linux_init(), needed for static initialisation

remove WSDISPLAYIO_OGINFO compat ioctl, planned for after 7.3
ok miod@

Fix grey colour, from Magnus Gross.

add some commas, to aid readability;
from thorsten blum

sync synopsis and usage, sort commands, fix their spacing

OK input lucas

remove unused cruft; ok kettenis@

fix HWCAP_ASIMDHP test; ok kettenis@

correct a prototype

catch up with the u-boot-aarch64 package split in install docs. changing
the dtb isn't normally needed (and can cause problems) so don't suggest that.
add some info for rk3588. partly from jsg.

rsync: make blkhash_free() callable with NULL.

It is in principle possible that blkhash_free() (which obviously frees
a struct blktab, sigh) is called with NULL. This would lead to a crash.

ok claudio

rsync: avoid reallocarray() with a nmemb of 0.

bset->blksz can be 0 here and how reallocarray() behaves is implementation
defined.

rsync: fix reallocarray() usage in blkhash_set()

The well-named ERR() macro doesn't error out. Therefore an incorrect use
of reallocarray() is actually a leak that is easily overlooked. Do it the
right way by assigning to a temporary variable and preserve behavior by
freeing and NULL-ing.

ok claudio

rpki-client: free deltas in rrdp_session_parse() as in rrdp_repo_free()

ok claudio

rpki-client: check for getline error when reading the skiplist file

ok claudio

test some more Match syntax, including criteria=arg and negations

Previous pipex.c,v 1.155 was broken if the client was not behind a NAT.

ok mvs

fix previous change to ssh_config Match, which broken on negated
Matches; spotted by phessler@ ok deraadt@

Fix amd64/GENERIC not linking due to missing pmap_shootept symbol.

My recent commit to add an INVEPT ipi missed an ifdef related to
MP machines.

ok krw@

Add an ipi for executing INVEPT to flush EPT on remote cpus.

Similar to how the fast ipi for tlb flush is implemented, this adds
one for calling INVEPT to invalidate EPT caches on the cpu. This
is the first step to allowing guest memory to not be wired by UVM
and decreases the behavioral differences between Intel and AMD's
nested paging in vmm(4) and pmap(9).

This change does not hook EPT ptes into the PV list, so the ipi is
only used during address space teardown and pte removal. (With the
removal of the "mprotect" ioctl, vmm(4) no longer modifies EPT ptes
other than inserting them and removing them.)

ok mlarkin@

remove unneeded psl.h include

Remove debug leftover. There is no need to run tcpdump anymore.

document that there are nvme sensors available and nudge the reader to sysctl;
ok bmercer

remove unneeded includes; ok dv@ mlarkin@

remove some unused defines; ok djm@

Correct an indexing error that may leave stale data in the mousecfg buffer.

ok mpi@

Do not crash in witness code if interrupt fires early.

It can happen that an interrupt handler is called immediately on a
secondary CPU when it is spun up during boot.  At that moment no
process is schduled yet, so curproc is NULL.  To prevent a crash
when p->p_sleeplocks is dereferenced, skip witness check in this
case.

OK visa@

Improve description of -f.

ok sthen, florian

Committing on behalf of jmc as requested.

Extend maxattr test to also include IPv6 and with that test for the
up_generate_mp_reach() failure.

correct macro for -R entry;

When generating UPDATE handle the message size limit better.

First of all warn that a prefix was dropped. In the generate an update
code handle possible overflows of attributes and NLRI and withdraw the
affected prefix. This way the peer will not have stale data.
OK tb@

Add a maxattr regress test that causes the rdomain1 bgpd instance to trip
over 4096 bytes for a BGP UPDATE message. Check on the 3rd system that the
affected prefixes are properly withdrawn.

sysupgrade: add -R #.# to try to use a specific release version rather
than the immediate +0.1. print an https://ftp.openbsd.org/... URL where
the new signify pubkey can be found if not present.

no guarantees: we only test +0.1, but jumping further does work quite
often (and if tight on disk, can work better than multiple steps) -
this avoids editing the script if you're going to do it anyway.

"Only upgrades from one version to the next are tested. Skipping
versions may work. Downgrading is unlikely to work."

discussed with deraadt chris florian, ok deraadt

remove some unneeded Xo/Xc calls; from evan silberman
the original diff had a couple of errors, which i've fixed

document why sys/param.h is used

fix regression introduced when I switched the "Match" criteria tokeniser
to a more shell-like one. Apparently the old tokeniser (accidentally?)
allowed "Match criteria=argument" as well as the "Match criteria argument"
syntax that we tested for.

People were using this syntax so this adds back support for
"Match criteria=argument"

bz3739 ok dtucker

- VM(s) -> VMs
- simpler tense
- fix the -width parameter
- add -nosplit to AUTHORS

vmd(8) manpage change for shutdown behaviour

Document the shutdown behaviour for vmd(8). Suggested via bugs@
via eric at mulh.net.

ok jmc@, bluhm@

backout scanner.l strtonum commits; they restrict numbers to INT_MAX,
breaking filter expressions like "ether[14:4]=0xaaaa0300". ok op@ deraadt@

-/--------------------------
revision 1.32
date: 2024/08/29 07:33:50;  author: anton;  state: Exp;  lines: +2 -2;  commitid: ZYtBn8qrrNNh87on;
Fix regression introduced in previous migration to strtol() by
correcting the error condition, making it on par with the example in the
manual.

ok op@
-/--------------------------
revision 1.31
date: 2024/08/28 11:40:33;  author: op;  state: Exp;  lines: +13 -31;  commitid: RirJPBPfGo9NeSE6;
libpcap: replace hand-rolled number parser with strtol

can't use strtonum here since it needs to handle octal and hex
notations as well.  Part of a larger diff that's ok beck@
-/--------------------------

Fix sleeping race during malloc in sysctl hw.disknames.

When mallocarray(9) sleeps, disk_count can change, and diskstatslen
gets inconsistent.  This caused free(9) to panic.

Reported-by: syzbot+36e1f3b306f721f90c72@syzkaller.appspotmail.com
OK deraadt@ mpi@

Remove -r toggle and generally be less smart.

The default is to install the next release. Snapshots are only
installed when invoked with -s.

The logic on what to do per default got out of hand and it was very
difficult to reason about what sysupgrade(8) actually did. deraadt@
then suggested that we should dumb it all down, sysupgrade(8) is there
to upgrade from one release to the next. More advance usage needs to
be requested by the user.

With all this simplification we can now be a bit more smart to work
out what the next release is. With that, snapshots right before a
release can be sysupgrade(8)'ed to the official release.

OK sthen on a previous version that was much more complicated but
allowed shortly-before-release -> release upgrade

testing sthen on this version

Guidance, prodding & OK deraadt

some extra paranoia, reminded by jsg@

___linux__ -> __linux__

for sysctl on arm64, handle CPU_ID_AA64ISAR1 same as CPU_ID_AA64ISAR0
ok sthen

If during parsing lines in the script, ksh finds a NUL byte on the
line, it should abort ("syntax error: NUL byte unexpected").  There
appears to be one piece of software which is misinterpreting guidance
of this, and trying to depend upon embedded NUL.  During research,
every shell we tested has one or more cases where a NUL byte in the
input or inside variable contents will create divergent behaviour from
other shells.  (ie. gets converted to a space, is silently skipped, or
aborts script parsing or later execution).  All the shells are written
in C, and majority of them use C strings for everything, which means
they cannot embed a NUL, so this is not surprising.  It is quite
unbelievable there are people trying to rewrite history on a lark, and
expecting the world to follow alone.

If there is ONE THING the Unix world needs, it is for bash/ksh/sh to
stop diverging further by permitting STUPID INPUT that cannot
plausibly work in all other shells.  We are in a post-Postel world.

It remains possible to put arbitrary bytes *AFTER* the parts of the
shell script that get parsed & executed (like some Solaris patch files
do).  But you can't put arbirary bytes in the middle, ahead of shell
script parsed lines, because shells can't jump to arbitrary offsets
inside the input file, they go THROUGH all the 'valid shell script
text lines' to get there.

This was in snapshots for more than 2 months, and only spotted one
other program depending on the behaviour (and that test program did
not observe that it was therefore depending in incorrect behaviour!!)

ok ingo.  Softer ok's from various others.

now hacking on 7.6-current
(corrected)

Replace `&&' with `if' for proper $? handling;  OK lucas

iked and isakmpd guard against themselves with "return 0" as rc.subr(8)
checks rc_pre()'s return code and aborts daemo start iff non-zero, but
that isn't needed if we use ksh properly.

now hacking on 7.6-current

zap redundant "|| return 1";  OK lucas

unbound-checkconf(8) itself exits 1 on error already.

document SIOCSIFMTU;  OK jmc

sync struct defintions from headers; OK jmc

recognise Neoverse V3AE (Poseidon AE)

regen

add more Navi 32 and Navi 33 device ids

7460 rev 00 is Radeon Pro V710
7461 rev 00 is Radeon Pro V710
7499 rev 00 is Radeon Pro W7400
7499 rev c0 is Radeon RX 7400
7499 rev c1 is Radeon RX 7300
found in ROCm 6.2.1 libdrm-amdgpu-common

add device tree mapping for HP OmniBook X 14
ok patrick@ kettenis@

Reinstate bounds check accidentally disabled when defining OPENSSL_NO_DTLS1

From Kenjiro Nakayama
Closes https://github.com/libressl/portable/issues/1097

remove some unused defines; ok djm@

For compatibility with groff, the .In macro has been callable and parsed
since mandoc-1.7.24 = mdoc_macro.c rev. 1.11 (July 12, 2009).
Match the documentation to the implementation.
Patch from Evan Silberman <evan at jklol dot net>.

Talk less about "Single UNIX" because it has a confusing numbering
scheme.  Instead, stick to the XPG/POSIX numbering scheme that has
clearly won: earlier this year, POSIX Issue 8 has been released,
and nobody calls it "Single UNIX version 5".

The XPG/POSIX numbering scheme is also preferable because it can
be used for the entire history leading up to the current POSIX
standard (1988-2024), whereas the Single UNIX numbering scheme
only works for the period from XPG4.2 to POSIX Issue 7 (1994-2008).

This patch mostly changes (informational) paragraph headers
and corrects two instances of the incorrect term "XPG version"
to the correct term "XPG Issue".  The (normative) .St argument
definitions remain unchanged, and all supported .St arguments
remain documented, including the deprecated arguments -susv1 to -susv4.

No objections when shown on tech@ a few months ago.

Increase the default buffer size for AF_UNIX from 8192 to 32768.

Using 8k for socketpairs was always on the low end side. Also this
avoid a fatal error in sshd that can be triggered when the network
stack is pushed hard enough to consume most of the allowed memory.
By increasing the default buffer size a bit the error in sshd is
avoided which is good enough for now.

Long term a better solution for sonewconn() and especially sbchecklowmem()
needs to be found. m_pool_used() returns not the right information for
them.

OK deraadt@ otto@

remove BACK_CHANNEL define, unused since YP bits removed in rev 1.20

We aren't ready to choose S0-over-S3 based upon the S0ix bit in FADT.
Some machines which work great in S3, don't work great in S0.

Some people want to be able to force S0, mostly for testing purposes
(or to notice improvements as changes are made in the tree).  Provide
a TEMPORARY method via machdep.lidaction=-1 which will be S0-suspend
while =1 remains S3 suspend.  This button will not remain long-term,
but for now, and during 7.6, it will be better than nothing.
ok ratchov kettenis

Check that .In is callable and parsed.
Triggered by a documentation patch from Evan Silberman <evan at jklol dot net>.

I'm not testing yet whether the same applies in the SYNOPSIS section
because there are multiple parsing and formatting differences with groff
in that section.

add route-to example

ok benno@

vmm(4): remove EPT mprotect ioctl

This old ioctl isn't used by vmd(8) and is getting in the way of some
improvements we want to do. It was used by solo5 but the person who was
helping maintain this is no longer involved with that project.

ok dv

vmm.h is a needed include now

Add device tree mapping for Lenovo ThinkPad T14s.

ok tobhe@, mlarkin@

Avoid use after free when retrying the -o file

This is horrible code and at least file leaks in various paths, but that's
for someone else to fix.

found by & ok jsg

gdb: fix path to ex

from Nir Lichtman
ok pascal sthen
(they okayed the ports version which I take to be an agreement with this).