Windows: PerformanceTuningInterval Merge error
authorJeffrey Altman <jaltman@your-file-system.com>
Sat, 11 Dec 2010 12:19:54 +0000 (06:19 -0600)
committerJeffrey Altman <jaltman@openafs.org>
Sun, 12 Dec 2010 04:35:04 +0000 (20:35 -0800)
Due to a merge error, if RankServerInterval was set via the
registry, its value would also be used for the
PerformanceTuningInterval.  Fix it.

Change-Id: I6605146db0c25ecbf0d20f9d7198934f5bf411ad
Reviewed-on: http://gerrit.openafs.org/3500
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>

src/WINNT/afsd/cm_daemon.c

index 0d79f20..a31766b 100644 (file)
@@ -343,6 +343,10 @@ cm_DaemonCheckInit(void)
     dummyLen = sizeof(DWORD);
     code = RegQueryValueEx(parmKey, "daemonPerformanceTuningInterval", NULL, NULL,
                            (BYTE *) &dummy, &dummyLen);
+    if (code == ERROR_SUCCESS)
+       cm_daemonPerformanceTuningInterval = dummy;
+    afsi_log("daemonPerformanceTuningInterval is %d", cm_daemonPerformanceTuningInterval);
+
     dummyLen = sizeof(DWORD);
     code = RegQueryValueEx(parmKey, "daemonRankServerInterval", NULL, NULL,
                            (BYTE *) &dummy, &dummyLen);
@@ -350,10 +354,6 @@ cm_DaemonCheckInit(void)
        cm_daemonRankServerInterval = dummy;
     afsi_log("daemonRankServerInterval is %d", cm_daemonRankServerInterval);
 
-    if (code == ERROR_SUCCESS)
-       cm_daemonPerformanceTuningInterval = dummy;
-    afsi_log("daemonPerformanceTuningInterval is %d", cm_daemonPerformanceTuningInterval);
-    
     RegCloseKey(parmKey);
 
     if (cm_daemonPerformanceTuningInterval)