Re: CVS commit: src/external/cddl/osnet

[Adam Hoka <adam.hoka%gmail.com@localhost>]

On Sat, 24 Apr 2010 12:18:59 +0200
haad <haaaad%gmail.com@localhost> wrote:

> On Fri, Apr 23, 2010 at 1:39 PM, Adam Hoka <ahoka%netbsd.org@localhost> wrote:
> > Module Name:    src
> > Committed By:   ahoka
> > Date:           Fri Apr 23 11:39:53 UTC 2010
> >
> > Modified Files:
> >        src/external/cddl/osnet/dev/dtrace: dtrace_sysctl.c dtrace_unload.c
> >        src/external/cddl/osnet/dev/dtrace/amd64: dtrace_subr.c
> >        src/external/cddl/osnet/dev/dtrace/i386: dtrace_subr.c
> >        src/external/cddl/osnet/dist/uts/common/dtrace: dtrace.c
> >
> > Log Message:
> > Remove a couple of zero length kmem_frees.
> >
> > It should fix at least one crash when unloading the dtrace module,
> > possibly many others.
> >
> 
> This should be reverted and properly reviewed. Adding more diffs
> against solaris code base is wrong.

That code is not from Solaris. Those wouldn't work there either...

-- 
NetBSD - Simplicity is prerequisite for reliability