Re: patch: tighten make parser and fix bug in makefile

On Sun, Jun 18, 2017 at 3:17 AM, Marc Espie wrote: > While reading thru gnu/usr.bin/cc, I couldn't figure out where DEPENDFILE > comes from. > > Turns out it comes from FreeBSD, which currently defines it in bsd.own.mk. > > The variable is empty on OpenBSD, thus yielding an always-false condition,

Re: Trapsleds

On Mon, Jun 19, 2017 at 09:22:57PM -0400, Todd Mortimer wrote: > Hello tech, > > I have attached a patch that converts NOP padding from the assembler > into INT3 padding on amd64. The idea is to remove potentially conveinent > NOP sleds from programs and libraries, which makes it harder for an > a

Re: Trapsleds

On Mon, Jun 19, 2017 at 09:22:57PM -0400, Todd Mortimer wrote: > Hello tech, > > I have attached a patch that converts NOP padding from the assembler > into INT3 padding on amd64. The idea is to remove potentially conveinent > NOP sleds from programs and libraries, which makes it harder for an > a

Trapsleds

Hello tech, I have attached a patch that converts NOP padding from the assembler into INT3 padding on amd64. The idea is to remove potentially conveinent NOP sleds from programs and libraries, which makes it harder for an attacker to hit any ROP gadgets or other instructions after a NOP sled. NO

switch libkvm bits from hand rolled RBT updates to proper RBT api use

now RBT is in userland, we can actually use the API instead of hand rolling the copying of the tree topology. ok? Index: lib/libkvm/kvm_proc.c === RCS file: /cvs/src/lib/libkvm/kvm_proc.c,v retrieving revision 1.58 diff -u -p -r1.58

optimise uvm_map_fill_vmmap with a binary search

userland can ask for an array of a processes memory mappings via sysctl, starting from a specified point. currently that starting point is found by iterating over the mappings from start to finish, but because the mappings are in an RBT we can do a binary search instead. ok? Index: uvm_map.c

Re: Towards tcp_input() w/o KERNEL_LOCK()

On Mon, Jun 19, 2017 at 03:53:21PM +0200, Martin Pieuchot wrote: > New version including a fix for kqueue filters. OK bluhm@ > Index: kern/uipc_socket.c > === > RCS file: /cvs/src/sys/kern/uipc_socket.c,v > retrieving revision 1.186

Re: Remove conditionals around crypto based free functions on relayd(8)

Hi, After being prodded by benno@ please find below a new diff for relayd(8) due to claudio@'s migration to libtls: Comments? OK? Index: relay.c === RCS file: /cvs/src/usr.sbin/relayd/relay.c,v retrieving revision 1.221 diff -u -p -

Re: Route priority support for ospf6d

Thanks, I commited it to move forward. On 06/09/17 15:41, Claudio Jeker wrote: > On Fri, Jun 09, 2017 at 03:28:07PM +0200, Alexander Bluhm wrote: >> On Wed, May 31, 2017 at 02:29:03PM +0200, Florian Riehm wrote: >>> @@ -359,6 +333,7 @@ kr_fib_decouple(void) >>> void >>> kr_dispatch_msg(int fd,

Re: freetype-config.in

Il 19 giu 2017 22:32, "Stuart Henderson" ha scritto: On 2017/06/19 22:18, David Coppa wrote: > Il 19 giu 2017 21:23, "Stuart Henderson" ha scritto: > > On 2017/06/19 17:46, David Coppa wrote: > > > > > > Il 19 giu 2017 16:19, "Stuart Henderson" ha > > scritto: > > > > I'm looking at updatin

Re: freetype-config.in

On 2017/06/19 22:18, David Coppa wrote: > Il 19 giu 2017 21:23, "Stuart Henderson" ha scritto: > > On 2017/06/19 17:46, David Coppa wrote: > > > > > > Il 19 giu 2017 16:19, "Stuart Henderson" ha > > scritto: > > > > I'm looking at updating freetype, mostly to unbreak PCF font names > > w

crypt_checkpass.3: clarify cetrain points

I lost the old thread. tedu@ said that we musn't recommend a buffer size of 61 (current underlying code), but instead suggested using _PASSWORD_LEN or 128. _PASSWORD_LEN is used in /usr/src and /usr/ports. Index: lib/libc/crypt/crypt_checkpass.3 ==

Re: freetype-config.in

Il 19 giu 2017 21:23, "Stuart Henderson" ha scritto: On 2017/06/19 17:46, David Coppa wrote: > > > Il 19 giu 2017 16:19, "Stuart Henderson" ha > scritto: > > I'm looking at updating freetype, mostly to unbreak PCF font names > which are currently different on different machines (e.g. I h

Re: freetype-config.in

On 2017/06/19 17:46, David Coppa wrote: > > > Il 19 giu 2017 16:19, "Stuart Henderson" ha > scritto: > > I'm looking at updating freetype, mostly to unbreak PCF font names > which are currently different on different machines (e.g. I have > "xos4 > Terminus" on a more recently i

Re: arp: Fix synopsis

> On Mon, Jun 19, 2017 at 12:15:49PM +0200, Klemens Nanni wrote: > > 'arp -[d]a' do not require hostname as told in the manual page. > > > > A single fprintf() is both shorter and cleaner. > > > > this diff suggest that running arp without arguments should work, but it > does not. > > there are

Re: arp: Fix synopsis

On Mon, Jun 19, 2017 at 12:15:49PM +0200, Klemens Nanni wrote: > 'arp -[d]a' do not require hostname as told in the manual page. > > A single fprintf() is both shorter and cleaner. > this diff suggest that running arp without arguments should work, but it does not. there are a few approaches i

Re: vi(1): documenting :s

On Mon, Jun 19, 2017 at 03:57:35AM -0600, Anthony J. Bentley wrote: > Hi, > > Jason McIntyre writes: > > ok by me. note that posix ex(1) does detail a working [s]ubstitute command, > > so i'm not sure whether we should support this or not. > > Hm, so it does. I think I would prefer to follow POSI

Re: freetype-config.in

Il 19 giu 2017 16:19, "Stuart Henderson" ha scritto: I'm looking at updating freetype, mostly to unbreak PCF font names which are currently different on different machines (e.g. I have "xos4 Terminus" on a more recently installed laptop but just "Terminus" on other machines that have been upgrade

Re: vi(1): documenting :s

Theo Buehler writes: > This looks like a reasonable approach and it appears to work. When I > looked at this after jmc's question, I was scared off by the comment > > > * Adding new commands starting with 's' may break the substitute command > code > > * in ex_cmd() (the ex parser). Read thr

Re: vi(1): documenting :s

On Mon, Jun 19, 2017 at 03:57:35AM -0600, Anthony J. Bentley wrote: > Hi, > > Jason McIntyre writes: > > ok by me. note that posix ex(1) does detail a working [s]ubstitute command, > > so i'm not sure whether we should support this or not. > > Hm, so it does. I think I would prefer to follow POSI

freetype-config.in

I'm looking at updating freetype, mostly to unbreak PCF font names which are currently different on different machines (e.g. I have "xos4 Terminus" on a more recently installed laptop but just "Terminus" on other machines that have been upgraded from older versions). This is a problem for ports whi

Re: Towards tcp_input() w/o KERNEL_LOCK()

On 09/06/17(Fri) 15:54, Martin Pieuchot wrote: > On 09/06/17(Fri) 00:32, Alexander Bluhm wrote: > > On Tue, Jun 06, 2017 at 05:15:40PM +0200, Martin Pieuchot wrote: > > > TCP/UDP are almost ready to run without KERNEL_LOCK() because accesses > > > to their sockets are serialized via the NET_LOCK().

Re: remove redundant flag from iwm(4)

> Date: Mon, 19 Jun 2017 13:02:58 +0200 > From: Stefan Sperling > > On Mon, Jun 19, 2017 at 11:57:36AM +0200, Mark Kettenis wrote: > > > @@ -7450,7 +7444,7 @@ iwm_init_task(void *arg1) > > > } > > > s = splnet(); > > > > > > - if (sc->sc_flags & IWM_FLAG_HW_INITED) > > > + if (sc->sc_flags

Re: radeondrm: add a handler for the WSDISPLAYIO_GINFO ioctl

> DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; > d=statdns-com.20150623.gappssmtp.com; s=20150623; > h=date:from:to:subject:message-id:mime-version:content-disposition > :user-agent; > bh=V9zHsQIE9vOgAyjMqPGAM0Ewu2IvCE/CHFNKt6wyxNQ=; > b=MkKZ9WE9lfj

Re: inteldrm: add a handler for the WSDISPLAYIO_GINFO ioctl

> Date: Sat, 17 Jun 2017 21:57:08 +0200 > From: Frederic Cambus > > Hi tech@, > > Add a handler for the WSDISPLAYIO_GINFO ioctl in inteldrm, allowing > to retrieve basic information about a framebuffer display. > > When running wsconsctl on a machine with inteldrm support, this now > returns th

Re: remove redundant flag from iwm(4)

On Mon, Jun 19, 2017 at 11:57:36AM +0200, Mark Kettenis wrote: > > @@ -7450,7 +7444,7 @@ iwm_init_task(void *arg1) > > } > > s = splnet(); > > > > - if (sc->sc_flags & IWM_FLAG_HW_INITED) > > + if (sc->sc_flags & IFF_RUNNING) > > iwm_stop(ifp, 0); > > This looks wrong to

Re: net80211: initialize link state during attach

> X-Envelope-From: s...@stsp.name > X-Envelope-To: > Date: Fri, 16 Jun 2017 15:50:24 +0200 > From: Stefan Sperling > Mail-Followup-To: tech@openbsd.org > Content-Disposition: inline > List-Owner: > X-Loop: tech@openbsd.org > Sender: owner-t...@openbsd.org > X-CNFS-

arp: Fix synopsis

'arp -[d]a' do not require hostname as told in the manual page. A single fprintf() is both shorter and cleaner. Index: arp.8 === RCS file: /cvs/src/usr.sbin/arp/arp.8,v retrieving revision 1.39 diff -u -p -r1.39 arp.8 --- arp.8

Re: vi(1): documenting :s

Hi, Jason McIntyre writes: > ok by me. note that posix ex(1) does detail a working [s]ubstitute command, > so i'm not sure whether we should support this or not. Hm, so it does. I think I would prefer to follow POSIX in this case. Here's a diff to allow "substitute" to work. Annoyingly, there's

Re: remove redundant flag from iwm(4)

> Date: Fri, 16 Jun 2017 15:39:30 +0200 > From: Stefan Sperling > > The HW_INITED flag has a grammatically dubious name and is unnecessary > because it duplicates the purpose of the IFF_RUNNING flag. > > ok? I don't think so... > Index: sys/dev/pci/if_iwm.c > ==

Re: smtpd session hang

On Fri, Jun 16, 2017 at 07:12:43PM +0300, Henri Kemppainen wrote: > > > Nice catch, the diff reads fine to me, I'll commit later today when I > > > have another ok from eric@ > > > Yes, this looks correct. But, I would rather move the resume test before > > the EOM test, to avoid touching the sess

manpage for pool_cache_init(9)

mikeb@ suggested that it would be nice to know what the numbers provided by the pool cache sysctls mean, which reminded me that i write a manpage for the pool cache stuff. the following adds a manpage that documents pool_cache_init(9) and pool_cache_destroy(9), and also the information that mikeb@