CVS: cvs.openbsd.org: src

2021-04-07 Thread Theo Buehler
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2021/04/07 15:48:23

Modified files:
lib/libssl : tls13_lib.c 

Log message:
Avoid clobbering the error code when sending an alert

In order to fail gracefully on encountering a self-signed cert, curl looks
at the top-most error on the stack and needs specific SSL_R_ error codes.
This mechanism was broken when the tls13_alert_sent_cb() was added after
people complained about unhelpful unknown errors. Fix this by only setting
the error code from a fatal alert if no error has been set previously.

Issue reported by Christopher Reid

ok jsing



CVS: cvs.openbsd.org: www

2021-04-07 Thread Sebastian Benoit
CVSROOT:/cvs
Module name:www
Changes by: be...@cvs.openbsd.org   2021/04/07 14:26:13

Modified files:
.  : 69.html 

Log message:
changes up to 2021-01-01, more to come



CVS: cvs.openbsd.org: src

2021-04-07 Thread Theo Buehler
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2021/04/07 11:21:40

Modified files:
regress/lib/libcrypto/pkcs7: pkcs7test.c 

Log message:
Use ERR_print_error_fp() to avoid leaking a BIO in fatal()



CVS: cvs.openbsd.org: src

2021-04-07 Thread Mark Kettenis
CVSROOT:/cvs
Module name:src
Changes by: kette...@cvs.openbsd.org2021/04/07 11:13:38

Modified files:
share/man/man4 : Makefile simpleaudio.4 
Added files:
share/man/man4 : graphaudio.4 

Log message:
graphaudio(4)



CVS: cvs.openbsd.org: src

2021-04-07 Thread Mark Kettenis
CVSROOT:/cvs
Module name:src
Changes by: kette...@cvs.openbsd.org2021/04/07 11:12:22

Modified files:
sys/arch/arm64/conf: GENERIC 
sys/dev/fdt: files.fdt 
sys/dev/ofw: ofw_misc.c ofw_misc.h 
Added files:
sys/dev/fdt: graphaudio.c 

Log message:
Add graphaudio(4), a driver to support linking together audio components
based on the audio graph description that uses ports and endpoints in
the device tree.

ok patrick@



CVS: cvs.openbsd.org: src

2021-04-07 Thread Claudio Jeker
CVSROOT:/cvs
Module name:src
Changes by: clau...@cvs.openbsd.org 2021/04/07 10:40:39

Modified files:
usr.sbin/rpki-client: http.c 

Log message:
In some cases the http process terminates and Theo's and my theory is that
this is caused by a SIGPIPE. So add a handler that will abort the process
and dump core.
OK deraadt@



CVS: cvs.openbsd.org: src

2021-04-07 Thread Mark Kettenis
CVSROOT:/cvs
Module name:src
Changes by: kette...@cvs.openbsd.org2021/04/07 10:35:02

Modified files:
sys/dev/fdt: rkclock.c rkclock_clocks.h 

Log message:
Add support for the fractional dividers for the i2s clocks.
Fixes audio on the pinebook pro.

ok kn@, patrick@



CVS: cvs.openbsd.org: src

2021-04-07 Thread Claudio Jeker
CVSROOT:/cvs
Module name:src
Changes by: clau...@cvs.openbsd.org 2021/04/07 10:29:14

Modified files:
usr.sbin/rpki-client: rrdp.c 

Log message:
Free the parser then close the file. Seems like the better order.



CVS: cvs.openbsd.org: src

2021-04-07 Thread Kenneth R Westerback
CVSROOT:/cvs
Module name:src
Changes by: k...@cvs.openbsd.org2021/04/07 10:09:40

Modified files:
sbin/dhclient  : dhclient.c 

Log message:
"oldlladddr" -> "oldlladdr" in fatal() verbiage.



CVS: cvs.openbsd.org: src

2021-04-07 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2021/04/07 10:06:37

Modified files:
usr.sbin/rpki-client: http.c main.c 

Log message:
minor KNF found while hunting for a bug



CVS: cvs.openbsd.org: src

2021-04-07 Thread Nicholas Marriott
CVSROOT:/cvs
Module name:src
Changes by: n...@cvs.openbsd.org2021/04/07 09:46:12

Modified files:
usr.bin/tmux   : key-string.c tty-keys.c 

Log message:
Restore previous behaviour so that C-X remains the same as C-x. Instead,
translate incoming extended keys so that they are consistent.



CVS: cvs.openbsd.org: src

2021-04-07 Thread Claudio Jeker
CVSROOT:/cvs
Module name:src
Changes by: clau...@cvs.openbsd.org 2021/04/07 08:26:21

Modified files:
usr.sbin/rpki-client: http.c 

Log message:
Handle bind() failure like connect() or socket() failure and try next
address if available. No other tools consider bind() errors as non-fatal
warnings so rpki-client should not behave different.
OK tb@



CVS: cvs.openbsd.org: src

2021-04-07 Thread Claudio Jeker
CVSROOT:/cvs
Module name:src
Changes by: clau...@cvs.openbsd.org 2021/04/07 08:19:31

Modified files:
usr.sbin/rpki-client: repo.c 

Log message:
When merging a repo even files to delete can be part of the temporary
work dir. So unlink can return an ENOENT error for the main repo. In
which case the temp dir should be tried.
Refactor this code a bit since there is no way rrdp_filename() should
fail in this part of the code.
OK tb@



CVS: cvs.openbsd.org: www

2021-04-07 Thread Theo Buehler
CVSROOT:/cvs
Module name:www
Changes by: t...@cvs.openbsd.org2021/04/07 07:00:39

Modified files:
faq: current.html 

Log message:
use  instead of 



Re: CVS: cvs.openbsd.org: src

2021-04-07 Thread Nicholas Marriott
Hi

Thanks but I was thinking about this a bit more and I think it is better
to translate the keys that come in with extended keys, so C-S-X is
converted to C-S-x. I had originally worried about different keyboard
layouts but I don't think many people will be affected and I will worry
about that if they complain, so I'm going to change it to do that later
on, which should mean this is unnecessary.


On Wed, Apr 07, 2021 at 01:32:23PM +0100, Stuart Henderson wrote:
> On 2021/04/07 01:30, Nicholas Marriott wrote:
> > CVSROOT:/cvs
> > Module name:src
> > Changes by: n...@cvs.openbsd.org2021/04/07 01:30:02
> > 
> > Modified files:
> > usr.bin/tmux   : key-string.c tty-keys.c 
> > 
> > Log message:
> > Fixes for extended keys: 1) allow C-x and C-X to be bound separately
> > since some terminals report them differently 2) use the "backspace"
> > option to translate backspace 3) map ctrl which are have the ctrl
> > implied (such as C-x) properly when the terminal reports both the key
> > and the modifier.
> > 
> > Note that any key bindings for C-X where C-x is meant must now be
> > changed.
> > 
> 
> I think this is going to affect a lot of people.
> Does this sound about right for an upgrade note?
> 
> Index: current.html
> ===
> RCS file: /cvs/www/faq/current.html,v
> retrieving revision 1.1068
> diff -u -p -r1.1068 current.html
> --- current.html  29 Mar 2021 12:11:22 -  1.1068
> +++ current.html  7 Apr 2021 12:30:48 -
> @@ -289,6 +289,19 @@ relayd or nginx). It no longer uses a co
>  line arguments.
>  
>  
> +2021/04/07 - tmux: ^X / C-X binding changes
> +
> +Some terminals report differently for uppercase and lowercase control
> +character sequences.
> +https://man.openbsd.org/tmux;>tmux(1) now handles these 
> separately;
> +if you previously had bindings or prefix using an upper-case variant
> +and need them to continue working with lower-case or with a terminal
> +that does not distinguish between them, you will need to adjust your
> +configuration.
> +This applies to bindings using C-letter or
> +^letter.
> +
> +
>  

CVS: cvs.openbsd.org: src

2021-04-07 Thread Nicholas Marriott
CVSROOT:/cvs
Module name:src
Changes by: n...@cvs.openbsd.org2021/04/07 06:50:12

Modified files:
usr.bin/tmux   : cfg.c cmd-queue.c tmux.1 

Log message:
Add a current_file format for the config file being parsed. Originally
suggested by kn@, also GitHub issue 2638.



CVS: cvs.openbsd.org: src

2021-04-07 Thread Nicholas Marriott
CVSROOT:/cvs
Module name:src
Changes by: n...@cvs.openbsd.org2021/04/07 06:49:33

Modified files:
usr.bin/tmux   : cmd-display-message.c 

Log message:
When display-message used in config file, show the message after the
config file finishes. GitHub issue 2637.



Re: CVS: cvs.openbsd.org: src

2021-04-07 Thread Stuart Henderson
On 2021/04/07 01:30, Nicholas Marriott wrote:
> CVSROOT:  /cvs
> Module name:  src
> Changes by:   n...@cvs.openbsd.org2021/04/07 01:30:02
> 
> Modified files:
>   usr.bin/tmux   : key-string.c tty-keys.c 
> 
> Log message:
> Fixes for extended keys: 1) allow C-x and C-X to be bound separately
> since some terminals report them differently 2) use the "backspace"
> option to translate backspace 3) map ctrl which are have the ctrl
> implied (such as C-x) properly when the terminal reports both the key
> and the modifier.
> 
> Note that any key bindings for C-X where C-x is meant must now be
> changed.
> 

I think this is going to affect a lot of people.
Does this sound about right for an upgrade note?

Index: current.html
===
RCS file: /cvs/www/faq/current.html,v
retrieving revision 1.1068
diff -u -p -r1.1068 current.html
--- current.html29 Mar 2021 12:11:22 -  1.1068
+++ current.html7 Apr 2021 12:30:48 -
@@ -289,6 +289,19 @@ relayd or nginx). It no longer uses a co
 line arguments.
 
 
+2021/04/07 - tmux: ^X / C-X binding changes
+
+Some terminals report differently for uppercase and lowercase control
+character sequences.
+https://man.openbsd.org/tmux;>tmux(1) now handles these 
separately;
+if you previously had bindings or prefix using an upper-case variant
+and need them to continue working with lower-case or with a terminal
+that does not distinguish between them, you will need to adjust your
+configuration.
+This applies to bindings using C-letter or
+^letter.
+
+
 

CVS: cvs.openbsd.org: src

2021-04-07 Thread Inoguchi Kinichiro
CVSROOT:/cvs
Module name:src
Changes by: inogu...@cvs.openbsd.org2021/04/07 04:44:03

Modified files:
usr.bin/openssl: x509.c 

Log message:
Check function return value in openssl(1) x509.c

input from bcook@, ok and comments from tb@



CVS: cvs.openbsd.org: src

2021-04-07 Thread Inoguchi Kinichiro
CVSROOT:/cvs
Module name:src
Changes by: inogu...@cvs.openbsd.org2021/04/07 04:29:58

Modified files:
usr.bin/openssl: x509.c 

Log message:
Avoid leak in error path

ok and input from tb@



CVS: cvs.openbsd.org: src

2021-04-07 Thread Nicholas Marriott
CVSROOT:/cvs
Module name:src
Changes by: n...@cvs.openbsd.org2021/04/07 01:30:02

Modified files:
usr.bin/tmux   : key-string.c tty-keys.c 

Log message:
Fixes for extended keys: 1) allow C-x and C-X to be bound separately
since some terminals report them differently 2) use the "backspace"
option to translate backspace 3) map ctrl which are have the ctrl
implied (such as C-x) properly when the terminal reports both the key
and the modifier.

Note that any key bindings for C-X where C-x is meant must now be
changed.



CVS: cvs.openbsd.org: src

2021-04-07 Thread David Gwynne
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2021/04/07 00:52:22

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

Log message:
back out r1.22, ie, don't blindly trim the FCS on rxed packets.

there's been multiple reports of severely reduced throughput using
ure(4) since 1.22 was committed, but all the gear we have to play
with is fine with it. i'm backing the diff out until we can get a
better handle on the problem

discussed with and ok kevlo@ jmatthew@