Merge pull request #44 from fireice-uk/fix-uninit-access

Make sure all singletons are set to null and make env a global ptr

xmrstak/backend/amd/minethd.cpp

@@ -65,7 +65,7 @@ __declspec(dllexport)
 #endif
 std::vector<iBackend*>* xmrstak_start_backend(uint32_t threadOffset, miner_work& pWork, environment& env)
 {
-	environment::inst() = env;
+	environment::inst(&env);
 	return amd::minethd::thread_starter(threadOffset, pWork);
 }
 } // extern "C"

xmrstak/backend/nvidia/minethd.cpp

@@ -112,7 +112,7 @@ __declspec(dllexport)
 #endif
 std::vector<iBackend*>* xmrstak_start_backend(uint32_t threadOffset, miner_work& pWork, environment& env)
 {
-	environment::inst() = env;
+	environment::inst(&env);
 	return nvidia::minethd::thread_starter(threadOffset, pWork);
 }
 } // extern "C"

xmrstak/misc/environment.hpp

@@ -12,35 +12,30 @@ struct params;
 
 struct environment
 {
-
-	static environment& inst()
-	{
-		static environment env;
-		return env;
-	}
-	
-	environment& operator=(const environment& env)
+	static inline environment& inst(environment* init = nullptr)
 	{
-		this->pPrinter = env.pPrinter;
-		this->pglobalStates = env.pglobalStates;
-		this->pJconfConfig = env.pJconfConfig;
-		this->pExecutor = env.pExecutor;
-		this->pParams = env.pParams;
-		return *this;
-	}
+		static environment* env = nullptr;
 
+		if(env == nullptr)
+		{
+			if(init == nullptr)
+				env = new environment;
+			else
+				env = init;
+		}
 
-	environment() : pPrinter(nullptr), pglobalStates(nullptr)
-	{
+		return *env;
 	}
 
+	environment()
+	{
+	}
 
-	printer* pPrinter;
-	globalStates* pglobalStates;
-	jconf* pJconfConfig;
-	executor* pExecutor;
-	params* pParams;
-
+	printer* pPrinter = nullptr;
+	globalStates* pglobalStates = nullptr;
+	jconf* pJconfConfig = nullptr;
+	executor* pExecutor = nullptr;
+	params* pParams = nullptr;
 };
 
 } // namepsace xmrstak