From 417d1c1138a4fecc4a80ad5dbdbe542f7079e535 Mon Sep 17 00:00:00 2001
From: buccella <buccella>
Date: Tue, 29 Jun 2010 22:54:33 +0000
Subject: [PATCH] [ 3022934 ] SLP code calls setupControl more than once

---
 ChangeLog         | 3 +++
 NEWS              | 1 +
 cimslpCMPI.c      | 2 --
 profileProvider.c | 2 +-
 4 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 68a2681..8754339 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -6,6 +6,9 @@
 	* cimslp.c, cimslpSLP.c:
 	[ 3022941 ] Remove HAVE_SLP_ALONE sections
 
+	* profileProvider.c, cimslpCMPI.c:
+	[ 3022934 ] SLP code calls setupControl more than once
+
 2010-06-17  Chris Buccella  <buccella@linux.vnet.ibm.com>
 
 	* providerDrv.c:
diff --git a/NEWS b/NEWS
index 69955b2..10f623b 100644
--- a/NEWS
+++ b/NEWS
@@ -10,6 +10,7 @@ Bugs fixed:
 - 2950521 Provider operations are wrongly traced
 - 3016863 Data corruption when replacing array properties
 - 3022941 Remove HAVE_SLP_ALONE sections
+- 3022934 SLP code calls setupControl more than once
 
 Changes in 1.3.8
 ================
diff --git a/cimslpCMPI.c b/cimslpCMPI.c
index d69bf15..86e0a32 100644
--- a/cimslpCMPI.c
+++ b/cimslpCMPI.c
@@ -275,7 +275,6 @@ getSLPData(cimomConfig cfg, const CMPIBroker *_broker,
   int             err;
 
   err = 1;
-  setupControl(configfile);
   if (getControlChars("slpHostnamelib", &ln) == 0) {
     libraryName(NULL, ln, dlName, 512);
     if ((hostnameLib = dlopen(dlName, RTLD_LAZY))) {
@@ -284,7 +283,6 @@ getSLPData(cimomConfig cfg, const CMPIBroker *_broker,
         err = 0;
     }
   }
-  sunsetControl();
   if (err)
     mlogf(M_ERROR, M_SHOW,
           "--- SLP Hostname exit %s not found. Defaulting to system hostname.\n",
diff --git a/profileProvider.c b/profileProvider.c
index a67fb6d..048f967 100644
--- a/profileProvider.c
+++ b/profileProvider.c
@@ -328,7 +328,6 @@ updateSLPReg(const CMPIContext *ctx, int slpLifeTime)
   _SFCB_ENTER(TRACE_SLP, "updateSLPReg");
 
   pthread_mutex_lock(&slpUpdateMtx);
-  setupControl(configfile);
 
   getControlBool("enableSlp", &enableSlp);
   if(!enableSlp) {
@@ -440,6 +439,7 @@ slpUpdate(void *args)
             timeLeft, slp_shutting_down ? "true" : "false"));
   }
   //End loop
+  sunsetControl();
   if(http_url) {
     _SFCB_TRACE(2, ("--- Deregistering http advertisement"));
     deregisterCIMService(http_url);