X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fdynamic.cpp;h=c75371a39b33cdbf73d3c1684dab430ea4a9e0fa;hb=eb28eaea35d9d109a0b7b890de9d957d562da675;hp=0e207dbd9513d61c46baf67d7896d4e053baa03f;hpb=6fa35bdb1814ef4eab42d050e283d05910dbf3d2;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/dynamic.cpp b/src/dynamic.cpp
index 0e207dbd9..c75371a39 100644
--- a/src/dynamic.cpp
+++ b/src/dynamic.cpp
@@ -33,9 +33,9 @@ using namespace std;
 #include <sys/types.h>
 #include <stdio.h>
 
-extern ServerConfig* Config;
+extern InspIRCd* ServerInstance;
 
-DLLManager::DLLManager(char *fname)
+DLLManager::DLLManager(const char *fname)
 {
 	err = NULL;
 
@@ -67,12 +67,13 @@ DLLManager::DLLManager(char *fname)
 	FILE* x = fopen(fname,"rb");
 	if (!x)
 	{
-		err = "Module file not found or cannot access, game over man!";
+		err = strerror(errno);
 		return;
 	}
+	log(DEBUG,"Opened module file %s",fname);
 	char tmpfile_template[255];
 	char buffer[65536];
-	snprintf(tmpfile_template, 255, "%s/inspircd_file.so.%d.XXXXXXXXXX",Config->TempDir,getpid());
+	snprintf(tmpfile_template, 255, "%s/inspircd_file.so.%d.XXXXXXXXXX",ServerInstance->Config->TempDir,getpid());
 	int fd = mkstemp(tmpfile_template);
 	if (fd == -1)
 	{
@@ -80,6 +81,7 @@ DLLManager::DLLManager(char *fname)
 		err = strerror(errno);
 		return;
 	}
+	log(DEBUG,"Copying %s to %s",fname, tmpfile_template);
 	while (!feof(x))
 	{
 		int n = fread(buffer, 1, 65535, x);
@@ -94,15 +96,31 @@ DLLManager::DLLManager(char *fname)
 			}
 		}
 	}
+	log(DEBUG,"Copied entire file.");
 	// Try to open the library now and get any error message.
-	
-	h = dlopen(tmpfile_template, RTLD_NOW );
-	err = (char*)dlerror();
-	close(fd);
-	fclose(x);
+
+	if (close(fd) == -1)
+		err = strerror(errno);
+	if (fclose(x) == EOF)
+		err = strerror(errno);
+
+	h = dlopen(fname, RTLD_NOW|RTLD_LOCAL);
+	if (!h)
+	{
+		log(DEBUG,"dlerror occured!");
+		err = (char*)dlerror();
+		return;
+	}
+
+	log(DEBUG,"Finished loading '%s': %0x",tmpfile_template, h);
+
 	// We can delete the tempfile once it's loaded, leaving just the inode.
-	if (!Config->debugging)
-		unlink(tmpfile_template);
+	if (!err && !ServerInstance->Config->debugging)
+	{
+		log(DEBUG,"Deleteting %s",tmpfile_template);
+		if (unlink(tmpfile_template) == -1)
+			err = strerror(errno);
+	}
 #endif
 }
 
@@ -110,8 +128,8 @@ DLLManager::~DLLManager()
 {
 #ifndef STATIC_LINK
 	// close the library if it isn't null
-	if (h != 0)
-	dlclose(h);
+	if (h)
+		dlclose(h);
 #endif
 }
 
@@ -119,7 +137,7 @@ DLLManager::~DLLManager()
 
 #ifdef STATIC_LINK
 
-bool DLLManager::GetSymbol(initfunc* &v, char *sym_name)
+bool DLLManager::GetSymbol(initfunc* &v, const char *sym_name)
 {
 	log(DEBUG,"Symbol search...");
 	for (int j = 0; modsyms[j].name; j++)
@@ -138,45 +156,52 @@ bool DLLManager::GetSymbol(initfunc* &v, char *sym_name)
 
 #else
 
-bool DLLManager::GetSymbol(void **v, char *sym_name)
+bool DLLManager::GetSymbol(void** v, const char* sym_name)
 {
 	// try extract a symbol from the library
 	// get any error message is there is any
 	
-	if(h != 0)
+	if (h)
 	{
-		*v = dlsym( h, sym_name );
+		log(DEBUG,"Found symbol %s", sym_name);
+		dlerror(); // clear value
+		*v = dlsym(h, sym_name);
 		err = (char*)dlerror();
-		if( err == 0 )
-			return true;
-	    	else
+		if (!*v || err)
 			return false;
 	}
+	
+	if (err)
+	{
+		return false;
+	}
 	else
 	{	
-		return false;
+		return true;
 	}
 }
 
 #endif
 
-DLLFactoryBase::DLLFactoryBase(char *fname, char *factory) : DLLManager(fname)
+DLLFactoryBase::DLLFactoryBase(const char* fname, const char* symbol) : DLLManager(fname)
 {
 	// try get the factory function if there is no error yet
-	
 	factory_func = 0;
 	
-	if(LastError() == 0)
+	if (!LastError())
 	{
 #ifdef STATIC_LINK
-		GetSymbol( factory_func, factory ? factory : (char*)"init_module" );
+		if (!GetSymbol( factory_func, symbol ? symbol : "init_module"))
 #else
-		GetSymbol( (void **)&factory_func, factory ? factory : (char*)"init_module" );
+		if (!GetSymbol( (void **)&factory_func, symbol ? symbol : "init_module"))
 #endif
+		{
+			throw ModuleException("Missing init_module() entrypoint!");
+		}
 	}
 }
 
-
 DLLFactoryBase::~DLLFactoryBase()
 {
 }
+