http://defect.opensolaris.org/bz/show_bug.cgi?id=4307
--- Comment #3 from Eric Saxe <eric.saxe at sun.com> 2008-10-28 22:40:41 --- (In reply to comment #2) > Here is a patch for it. > > > diff -r b19ac1d32eaa usr/src/uts/i86pc/io/cpudrv/cpudrv_mach.c > --- a/usr/src/uts/i86pc/io/cpudrv/cpudrv_mach.c Mon Oct 27 15:08:24 2008 -0700 > +++ b/usr/src/uts/i86pc/io/cpudrv/cpudrv_mach.c Wed Oct 29 13:24:13 2008 +0800 > @@ -739,7 +739,12 @@ > return (CPUPM_NO_DOMAIN); > > mach_state = cpudsp->mach_state; > - domain = mach_state->pstate_domain_node->pdn_domain; > + > + if (mach_state->pstate_domain_node == NULL) > + return (CPUPM_NO_DOMAIN); > + > + if ((domain = mach_state->pstate_domain_node->pdn_domain) == NULL) > + return (CPUPM_NO_DOMAIN); > > return ((id_t)domain->pd_domain); > } Thanks Aubrey! Did it trip the NULL pointer via the dereference of pstate_domain_node, or pdn_domain? I haven't looked in the code to see if it would be possible for the domain node to be created, but not the actual domain...but from your patch it looks like you're saying that's a possibility... -- Configure bugmail: http://defect.opensolaris.org/bz/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
