OpenBSD cvs log

created 2018-12-30T11:08:57Z
begin 2018-12-17T00:00:00Z
end 2018-12-18T00:00:00Z
path src/sys
commits 9

date 2018-12-17T09:11:10Z
author claudio
files src/sys/net/pf.c log diff annotate
message Use timeout_add_sec() instead of timeout_add() with a multiplication with hz
OK kn@, florian@, visa@, cheloha@

date 2018-12-17T09:11:11Z
author claudio
files src/sys/net/pf_ioctl.c log diff annotate
message Use timeout_add_sec() instead of timeout_add() with a multiplication with hz
OK kn@, florian@, visa@, cheloha@

date 2018-12-17T09:17:30Z
author claudio
files src/sys/netinet/ip_carp.c log diff annotate
message Switch from timeout_add with tvtohz to just timeout_add_tv. Now this change
will reduce the sleep time by one tick which doesn't matter in the common
case. The code never passes a true 0 timeval to timeout_add_tv so the code
will always sleep for at least 1 tick which is good enough.
OK kn@, florian@, visa@, cheloha@

date 2018-12-17T14:51:57Z
author visa
files src/sys/kern/kern_sig.c log diff annotate
src/sys/sys/signalvar.h log diff annotate
message Remove unused function gsignal().

OK deraadt@ anton@

date 2018-12-17T15:37:41Z
author kn
files src/sys/net/pf_ioctl.c log diff annotate
src/sys/net/pf_ruleset.c log diff annotate
src/sys/net/pfvar.h log diff annotate
message Rename pf_anchor_remove() to pf_remove_anchor()

For semantic consistency with pf_{create,find,remove}_{anchor,ruleset}().

Simplify logic by squashing the if/else block while here.
No functional change.

Feedback jca and mikeb, OK mikeb

date 2018-12-17T16:46:59Z
author bluhm
files src/sys/kern/uipc_socket.c log diff annotate
message When using MSG_WAITALL, soreceive() can sleep while processing the
receive buffer of a stream socket. Then a new pair of control and
data mbuf can be appended to the mbuf queue. In this case, terminate
the loop with a short read to prevent a panic. Userland should
read the control message with the next system call.
OK claudio@ deraadt@

date 2018-12-17T19:14:59Z
author anton
files src/sys/dev/wscons/wsmux.c log diff annotate
message When no child devices are attached to a wsmux device, make sure to return an
error for inappropriate ioctl commands. Prevents a panic caused by using a wsmux
device as the controlling terminal.

ok deraadt@ visa@

Reported-by: syzbot+44bab40fc5a11357d774@syzkaller.appspotmail.com

date 2018-12-17T23:08:36Z
author dlg
files src/sys/net/if_etherip.c log diff annotate
src/sys/net/if_gre.c log diff annotate
message If something like ether_ioctl returns ENETRESET, don't let it bubble out.

ENETRESET in hardware drivers means you should reprogram the hardware.
There's no hardware to reprogram, so just turn it into 0 on the way
out.

date 2018-12-17T23:42:47Z
author dlg
files src/sys/net/if_etherip.c log diff annotate
src/sys/net/if_gre.c log diff annotate
message Handle SIOCADDMULTI and SIOCDELMULTI as nops in ioctl code.

There's nothing underneath the tunnels that needs configuration,
so there's no point in keepign track of configured multicast
addresses. We will at least save a bit of memory.