cdio read_track() malloc/free

2017-06-18 Thread Michael W. Bombardieri
Hello, In cdio's read_track() the error condition around sio_write() doesn't free local storage "sec" before it returns. The other error cases appear to free it. - Michael Index: rip.c === RCS file: /cvs/src/usr.bin/cdio/rip.c,v r

Re: inteldrm: add a handler for the WSDISPLAYIO_GINFO ioctl

2017-06-18 Thread Ted Unangst
Frederic Cambus wrote: > On Sat, Jun 17, 2017 at 05:27:38PM -0400, Ted Unangst wrote: > > > > Add a handler for the WSDISPLAYIO_GINFO ioctl in inteldrm, allowing > > > to retrieve basic information about a framebuffer display. > > [...] > > > why? what program will use this information? > > The

radeondrm: add a handler for the WSDISPLAYIO_GINFO ioctl

2017-06-18 Thread Frederic Cambus
Hi tech@, Add a handler for the WSDISPLAYIO_GINFO ioctl in radeondrm, allowing to retrieve basic information about a framebuffer display. Same rationale as the diff I sent for inteldrm, and it can also be tested with wsconsctl which will return new display values for width, height and depth. Com

[patch] mg: fix overflow on vteeol() (resend/bump)

2017-06-18 Thread Hiltjo Posthuma
Hey, This is a resend/bump of a patch about a month ago, can it get applied? Original message below: mg crashes with certain (unicode) characters and moving the cursor to the end of the line. The characters are printed to the screen as \nnn in vtpute() and vtcol is updated, however vteeol() wi

wsfontload(8): close file descriptors on all exit paths

2017-06-18 Thread Frederic Cambus
Hi tech@, Close file descriptors on all exit paths in wsfontload(8). Comments? OK? Index: usr.sbin/wsfontload/wsfontload.c === RCS file: /cvs/src/usr.sbin/wsfontload/wsfontload.c,v retrieving revision 1.18 diff -u -p -r1.18 wsfontlo

Re: update logging in ifstated

2017-06-18 Thread Sebastian Benoit
Rob Pierce(r...@2keys.ca) on 2017.06.13 19:12:18 -0400: > On Tue, Jun 13, 2017 at 08:44:46AM +0200, Sebastian Benoit wrote: > > Rob Pierce(r...@2keys.ca) on 2017.06.11 18:04:31 -0400: > > > This minimizes differences with the latest log.c. > > > > > > I was not sure how to handle verbosity, as the

Re: faster timecounters for kvm/xen

2017-06-18 Thread Mike Belopuhov
On Sun, Jun 18, 2017 at 21:20 +1000, Jonathan Matthew wrote: > On Fri, Jun 16, 2017 at 10:25:29AM +0200, Mike Belopuhov wrote: > > Now regarding the diff. pvbus_init_vcpu. Ah yes, please. > > It was a chicken and the egg problem for me: I didn't have > > Xen, but wanted a callback from cpu_hatch

Re: faster timecounters for kvm/xen

2017-06-18 Thread Jonathan Matthew
On Fri, Jun 16, 2017 at 10:25:29AM +0200, Mike Belopuhov wrote: > On Fri, Jun 16, 2017 at 16:31 +1000, Jonathan Matthew wrote: > > Recently I updated the kernel lock profiling stuff I've been working on, > > since > > it had been rotting a bit since witness was introduced. Running my diff > > o

patch: tighten make parser and fix bug in makefile

2017-06-18 Thread Marc Espie
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, which was definitely not the intent. This does put the right

Re: inteldrm: add a handler for the WSDISPLAYIO_GINFO ioctl

2017-06-18 Thread Frederic Cambus
On Sat, Jun 17, 2017 at 05:27:38PM -0400, Ted Unangst wrote: > > Add a handler for the WSDISPLAYIO_GINFO ioctl in inteldrm, allowing > > to retrieve basic information about a framebuffer display. [...] > why? what program will use this information? The reason I would like to add this handler is