Re: svn commit: r255439 - head/sys/dev/cpuctl

2013-09-10 Thread John-Mark Gurney
Konstantin Belousov wrote this message on Tue, Sep 10, 2013 at 05:17 +: > Author: kib > Date: Tue Sep 10 05:17:53 2013 > New Revision: 255439 > URL: http://svnweb.freebsd.org/changeset/base/255439 > > Log: > Call free() on the pointer returned from malloc(). We really should add a KASSERT f

Re: svn commit: r255439 - head/sys/dev/cpuctl

2013-09-10 Thread Konstantin Belousov
On Tue, Sep 10, 2013 at 07:29:51AM -0700, Matthew Fleming wrote: > I don't know exactly what the stock malloc(9) will return, but memguard(9), > under its default mode with vm.memguard.options having MG_GUARD_AROUND set > will align the returned pointer to only 16 bytes. When I added that > featur

Re: svn commit: r255439 - head/sys/dev/cpuctl

2013-09-10 Thread Konstantin Belousov
On Tue, Sep 10, 2013 at 06:09:54PM +0300, Konstantin Belousov wrote: > On Tue, Sep 10, 2013 at 07:29:51AM -0700, Matthew Fleming wrote: > > I don't know exactly what the stock malloc(9) will return, but memguard(9), > > under its default mode with vm.memguard.options having MG_GUARD_AROUND set > >

Re: svn commit: r255439 - head/sys/dev/cpuctl

2013-09-10 Thread Matthew Fleming
On Mon, Sep 9, 2013 at 10:17 PM, Konstantin Belousov wrote: > Author: kib > Date: Tue Sep 10 05:17:53 2013 > New Revision: 255439 > URL: http://svnweb.freebsd.org/changeset/base/255439 > > Log: > Call free() on the pointer returned from malloc(). > > Reported and tested by: Oliver Pinter

svn commit: r255439 - head/sys/dev/cpuctl

2013-09-09 Thread Konstantin Belousov
Author: kib Date: Tue Sep 10 05:17:53 2013 New Revision: 255439 URL: http://svnweb.freebsd.org/changeset/base/255439 Log: Call free() on the pointer returned from malloc(). Reported and tested by: Oliver Pinter Sponsored by: The FreeBSD Foundation MFC after:3 days Approved