CVS: cvs.openbsd.org: src

2023-09-09 Thread Jonathan Gray
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2023/09/09 23:22:46

Modified files:
lib/libcrypto/man: EVP_CIPHER_meth_new.3 

Log message:
fix Xr punctuation



CVS: cvs.openbsd.org: src

2023-09-09 Thread Jonathan Gray
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2023/09/09 22:05:26

Modified files:
lib/libcrypto/man: EVP_PKEY_CTX_get_operation.3 
   EVP_PKEY_keygen.3 

Log message:
spelling



CVS: cvs.openbsd.org: src

2023-09-09 Thread Damien Miller
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2023/09/09 21:51:55

Modified files:
usr.bin/ssh: clientloop.c 

Log message:
typo in comment



CVS: cvs.openbsd.org: src

2023-09-09 Thread Damien Miller
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2023/09/09 21:25:53

Modified files:
usr.bin/ssh: clientloop.c 

Log message:
randomise keystroke obfuscation intervals and average interval rate.
ok dtucker@



CVS: cvs.openbsd.org: src

2023-09-09 Thread Scott Soule Cheloha
CVSROOT:/cvs
Module name:src
Changes by: chel...@cvs.openbsd.org 2023/09/09 21:08:05

Modified files:
sys/sys: clockintr.h sched.h time.h resourcevar.h 
sys/kern   : kern_clockintr.c kern_sched.c sched_bsd.c 
 kern_time.c subr_prof.c 

Log message:
clockintr: support an arbitrary callback function argument

Callers can now provide an argument pointer to clockintr_establish().
The pointer is kept in a new struct clockintr member, cl_arg.  The
pointer is passed as the third parameter to clockintr.cl_func when it
is executed during clockintr_dispatch().  Like the callback function,
the callback argument is immutable after the clockintr is established.

At present, nothing uses this.  All current clockintr_establish()
callers pass a NULL arg pointer.  However, I am confident that dt(4)'s
profile provider will need this in the near future.

Requested by dlg@ back in March.



CVS: cvs.openbsd.org: src

2023-09-09 Thread Scott Soule Cheloha
CVSROOT:/cvs
Module name:src
Changes by: chel...@cvs.openbsd.org 2023/09/09 19:41:16

Modified files:
sys/kern   : kern_clockintr.c 

Log message:
clockintr_dispatch: add intermediate pointer for clockintr_queue.cq_shadow

Adding an intermediate pointer lets me shortens "cq->cq_shadow" to
just "shadow".  I think it makes the dispatch loop logic a little
easier to read.

While here, add a clarifying comment.



CVS: cvs.openbsd.org: src

2023-09-09 Thread Todd C . Miller
CVSROOT:/cvs
Module name:src
Changes by: mill...@cvs.openbsd.org 2023/09/09 12:59:44

Modified files:
usr.bin/awk: FIXES lex.c main.c 

Log message:
Update awk to Dec 15, 2022 version.

Force hex escapes in strings to be no more than two characters, as
they already are in regular expressions. This brings internal
consistency, as well as consistency with gawk.



CVS: cvs.openbsd.org: src

2023-09-09 Thread Theo Buehler
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2023/09/09 12:34:53

Modified files:
distrib/sets/lists/comp: mi 

Log message:
sync



CVS: cvs.openbsd.org: src

2023-09-09 Thread Scott Soule Cheloha
CVSROOT:/cvs
Module name:src
Changes by: chel...@cvs.openbsd.org 2023/09/09 12:19:03

Modified files:
sys/kern   : kern_clock.c 
sys/sys: sched.h 

Log message:
kernel: remove schedhz

Now that alpha no longer sets schedhz, schedhz is a dead variable.
Remove it.

For now, leave the schedclock() call in place in statclock().  It
still runs at its default rate of (stathz / 4).

Part of mpi@'s WIP scheduler patch.  Suggested by mpi@.

Thread: https://marc.info/?l=openbsd-tech=169419781317781=2

ok mpi@



CVS: cvs.openbsd.org: src

2023-09-09 Thread Scott Soule Cheloha
CVSROOT:/cvs
Module name:src
Changes by: chel...@cvs.openbsd.org 2023/09/09 11:07:59

Modified files:
sys/kern   : kern_clockintr.c 

Log message:
clockintr_advance: tweak logic to eliminate early-return

With the switch to clockintr_schedule_locked(), clockintr_advance() is
now much shorter and the early-return from the non-mutex path doesn't
make the function any easier to read.  Move the mutex path into the else
branch and always return 'count' at the end of the function.



CVS: cvs.openbsd.org: src

2023-09-09 Thread Scott Soule Cheloha
CVSROOT:/cvs
Module name:src
Changes by: chel...@cvs.openbsd.org 2023/09/09 10:59:01

Modified files:
sys/kern   : kern_clockintr.c 

Log message:
clockintr_schedule: tweak logic to eliminate early return

With the introduction of clockintr_schedule_locked(), clockintr_schedule()
is short enough that the early-return from the non-mutex path doesn't make
the function easier to read.  Move the mutex path into the else branch.



CVS: cvs.openbsd.org: src

2023-09-09 Thread Scott Soule Cheloha
CVSROOT:/cvs
Module name:src
Changes by: chel...@cvs.openbsd.org 2023/09/09 10:34:39

Modified files:
sys/kern   : kern_clockintr.c 

Log message:
clockintr_advance: delete duplicate code, call clockintr_schedule_locked()

Replace a bunch of code in clockintr_advance() with just one call to
clockintr_schedule_locked() -- everyone's favorite hot new subroutine.



CVS: cvs.openbsd.org: src

2023-09-09 Thread Scott Soule Cheloha
CVSROOT:/cvs
Module name:src
Changes by: chel...@cvs.openbsd.org 2023/09/09 10:20:48

Modified files:
sys/kern   : kern_clockintr.c 

Log message:
clockintr_schedule: refactor reusable code into clockintr_schedule_locked()

Move the code common to clockintr_advance() and clockintr_schedule()
from clockintr_schedule() into a new function, clockintr_schedule_locked().
In a subsequent commit we will replace that code in clockintr_advance()
with a call to clockintr_schedule_locked().



CVS: cvs.openbsd.org: src

2023-09-09 Thread Claudio Jeker
CVSROOT:/cvs
Module name:src
Changes by: clau...@cvs.openbsd.org 2023/09/09 08:50:09

Modified files:
sys/kern   : kern_sig.c 

Log message:
Fix SCHED_LOCK() leak in single_thread_set()

In the (q->p_flag & P_WEXIT) branch is a continue that did not release
the SCHED_LOCK. Refactor the code a bit to simplify the places SCHED_LOCK
is grabbed and released.

Reported-by: syzbot+ea26d351acfad3bb3...@syzkaller.appspotmail.com
OK kettenis@



CVS: cvs.openbsd.org: src

2023-09-09 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2023/09/09 08:39:09

Modified files:
lib/libcrypto/man: EVP_PKEY_CTX_new.3 evp.3 Makefile 
Added files:
lib/libcrypto/man: EVP_PKEY_CTX_get_operation.3 

Log message:
new manual page EVP_PKEY_CTX_get_operation(3),
also documenting EVP_PKEY_CTX_get0_pkey(3)



CVS: cvs.openbsd.org: src

2023-09-09 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2023/09/09 08:35:23

Modified files:
lib/libcrypto/man: EVP_PKEY_meth_new.3 

Log message:
document EVP_PKEY_CTX_get_data(3) and EVP_PKEY_CTX_set_data(3)



CVS: cvs.openbsd.org: src

2023-09-09 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2023/09/09 08:31:38

Modified files:
lib/libcrypto/man: EVP_PKEY_keygen.3 

Log message:
Document EVP_PKEY_CTX_set0_keygen_info(3).
While here, also add the missing RETURN VALUES entries
for EVP_PKEY_gen_cb(3), EVP_PKEY_CTX_get_cb(3),
and EVP_PKEY_CTX_get_keygen_info(3).



CVS: cvs.openbsd.org: src

2023-09-09 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2023/09/09 08:26:35

Modified files:
lib/libcrypto/man: EVP_PKEY_derive.3 

Log message:
Document EVP_PKEY_CTX_get0_peerkey(3).
While here, also make the descriptions of the other functions more precise.



CVS: cvs.openbsd.org: src

2023-09-09 Thread Kevin Lo
CVSROOT:/cvs
Module name:src
Changes by: ke...@cvs.openbsd.org   2023/09/09 08:24:07

Modified files:
sys/dev/usb: usbdevs.h usbdevs_data.h 

Log message:
Regen



CVS: cvs.openbsd.org: src

2023-09-09 Thread Kevin Lo
CVSROOT:/cvs
Module name:src
Changes by: ke...@cvs.openbsd.org   2023/09/09 08:23:37

Modified files:
sys/dev/usb: if_ure.c usbdevs 

Log message:
Correct TP-LINK bluetooth ID.
Reported on bugs@ by Douglas Silva .

ok jsg@



CVS: cvs.openbsd.org: src

2023-09-09 Thread Anthony J . Bentley
CVSROOT:/cvs
Module name:src
Changes by: bent...@cvs.openbsd.org 2023/09/09 08:15:42

Modified files:
share/man/man5 : bsd.port.mk.5 

Log message:
an PT_OPENBSD_* -> a PT_OPENBSD_*

ok jmc@



CVS: cvs.openbsd.org: src

2023-09-09 Thread Marc Espie
CVSROOT:/cvs
Module name:src
Changes by: es...@cvs.openbsd.org   2023/09/09 04:56:22

Modified files:
share/man/man5 : bsd.port.mk.5 

Log message:
gc WRKDIR_LINK_NAME



CVS: cvs.openbsd.org: src

2023-09-09 Thread Marc Espie
CVSROOT:/cvs
Module name:src
Changes by: es...@cvs.openbsd.org   2023/09/09 04:07:00

Modified files:
share/man/man5 : bsd.port.mk.5 

Log message:
remove PATCH_DEBUG documentation, since it has been always on since 2016



CVS: cvs.openbsd.org: src

2023-09-09 Thread Theo Buehler
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2023/09/09 03:12:01

Modified files:
distrib/sets/lists/base: mi 

Log message:
sync



Re: CVS: cvs.openbsd.org: src

2023-09-09 Thread Saša Nedvědický
Hello,

I was about to start looking into it. however there was another mail
in my inbox where I learned naddy@ has backed out the commit.
bluhm@ and I agree with him. the change attempts to fix awkward
corner case doing more harm than good.

thanks and
regards
sashan

pá 8. 9. 2023 v 12:59 odesílatel Anton Lindqvist  napsal:
>
> On Thu, Sep 07, 2023 at 03:59:43AM -0600, Alexandr Nedvedicky wrote:
> > CVSROOT:  /cvs
> > Module name:  src
> > Changes by:   sas...@cvs.openbsd.org  2023/09/07 03:59:43
> >
> > Modified files:
> >   sys/net: pf.c
> >
> > Log message:
> > pf(4) ignores 'keep state' and 'nat-to' actions for unsolicited
> > icmp error responses. Fix tightens rule matching logic so icmp
> > error responses no longer match 'keep state' rule. In typical
> > scenarios icmp errors (if solicited) should match existing state.
> > The change is going to bite firewalls which deal with asymmetric
> > routes. In those cases the 'keep state' action should be relaxed
> > to sloppy or new 'no state' rule to explicitly match icmp
> > errors should be added.
> >
> > The issue has been reported by Peter J. Philip (pjp _at_ delphinusdns.org).
> >
> > Discussed with bluhm@ and florian@
> >
> > OK bluhm@
>
> Couple of new failures. Reverting this commit makes the problem go away.
>
> https://regress.basename.se/amd64/2023-09-08.1/296-sys-net-pair.log
> https://regress.basename.se/amd64/2023-09-08.1/310-sys-net-vxlan.log
> https://regress.basename.se/amd64/2023-09-08.1/397-usr.sbin-bgpd.log
> https://regress.basename.se/amd64/2023-09-08.1/405-usr.sbin-ospf6d.log



CVS: cvs.openbsd.org: src

2023-09-09 Thread ASOU Masato
CVSROOT:/cvs
Module name:src
Changes by: a...@cvs.openbsd.org2023/09/09 00:52:41

Modified files:
lib/libc/stdlib: malloc.c 

Log message:
Print waring message when not allocated memory in putleakinfo().

ok otto.