Re: loading new envstat properties reboots system

[Nicolas Joly <njoly%pasteur.fr@localhost>]

On Thu, Dec 16, 2010 at 06:43:00AM -0800, Paul Goyette wrote:
> 
> Very strange.  It would seem that something is not being initialized 
> properly.  I will investigate further.

Got it. In sme_remove_userprops(), props variable is only initialised
when sme_get_limits is not NULL, but always checked later to register
the initial limits if exists.

The attached patch do fix it.

-- 
Nicolas Joly

Biological Software and Databanks.
Institut Pasteur, Paris.

Index: sys/dev/sysmon/sysmon_envsys.c
===================================================================
RCS file: /cvsroot/src/sys/dev/sysmon/sysmon_envsys.c,v
retrieving revision 1.110
diff -u -p -r1.110 sysmon_envsys.c
--- sys/dev/sysmon/sysmon_envsys.c      15 Dec 2010 17:17:16 -0000      1.110
+++ sys/dev/sysmon/sysmon_envsys.c      16 Dec 2010 15:01:55 -0000
@@ -1136,6 +1136,7 @@ sme_remove_userprops(void)
                        /*
                         * Next, we need to retrieve those initial limits.
                         */
+                       props = 0;
                        edata->upropset &= ~PROP_LIMITS;
                        if (sme->sme_get_limits) {
                                DPRINTF(("%s: retrieve limits for %s %s\n",