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 k...@freebsd.orgwrote: 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:

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 will

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 feature I

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 for

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 oliver.p...@gmail.com Sponsored by: The FreeBSD Foundation MFC after: