On Tue, Jun 23, 2015 at 10:41:42AM +0000, Paul Goyette wrote: > Committed By: pgoyette > Date: Tue Jun 23 10:41:42 UTC 2015 > > Modified Files: > src/sys/dev/sysmon: sysmon_envsys_events.c > > Log Message: > Fix the KASSERT - we want to make sure that _both_ pointers are non-NULL, > not just that one or the other is non-NULL!
Wouldn't it be better written as: KASSERT(sme != NULL); KASSERT(edata != NULL); Same effect, but when one of the triggers you know immediately which pointer was NULL. --chris