Added some missing parameter checking in m_swhois

[user/henk/code/inspircd.git] / src / inspircd.cpp

diff --git a/src/inspircd.cpp b/src/inspircd.cpp
index 326c209e9a91c4873da0c7f3a58b626e6f163d5b..858862e9dfa79c0ce06d05fbe9d3ddf1096fef84 100644 (file)
--- a/src/inspircd.cpp
+++ b/src/inspircd.cpp
@@ -14,18 +14,21 @@
 #include "inspircd.h"
 #include "configreader.h"
 #include <signal.h>
-#ifndef WIN32
 
+#ifndef WIN32
 #include <dirent.h>
 #include <unistd.h>
 #include <sys/resource.h>
+#include <dlfcn.h>
+#include <getopt.h>
 
-/* This is just to be completely certain that the change which fixed getrusage on RH7 doesn't break anything else -- Om */  
+/* Some systems don't define RUSAGE_SELF. This should fix them. */
 #ifndef RUSAGE_SELF
-#define RUSAGE_SELF 0
+       #define RUSAGE_SELF 0
 #endif
 
 #endif
+
 #include <exception>
 #include <fstream>
 #include "modules.h"
@@ -38,10 +41,9 @@
 #include "command_parse.h"
 #include "exitcodes.h"
 
-#ifndef WIN32
-#include <dlfcn.h>
-#include <getopt.h>
-#else
+#ifdef WIN32
+
+/* This MUST remain static and delcared outside the class, so that WriteProcessMemory can reference it properly */
 static DWORD owner_processid = 0;
 
 DWORD WindowsForkStart(InspIRCd * Instance)
@@ -953,9 +955,11 @@ bool InspIRCd::LoadModule(const char* filename)
                                return false;
                        }
                }
+               Module* m = NULL;
+               ircd_module* a = NULL;
                try
                {
-                       ircd_module* a = new ircd_module(this, modfile);
+                       a = new ircd_module(this, modfile);
                        factory[this->ModCount+1] = a;
                        if (factory[this->ModCount+1]->LastError())
                        {
@@ -965,14 +969,13 @@ bool InspIRCd::LoadModule(const char* filename)
                        }
                        if ((long)factory[this->ModCount+1]->factory != -1)
                        {
-                               Module* m = factory[this->ModCount+1]->factory->CreateModule(this);
+                               m = factory[this->ModCount+1]->factory->CreateModule(this);
 
                                Version v = m->GetVersion();
 
                                if (v.API != API_VERSION)
                                {
                                        delete m;
-                                       delete a;
                                        this->Log(DEFAULT,"Unable to load %s: Incorrect module API version: %d (our version: %d)",modfile,v.API,API_VERSION);
                                        snprintf(MODERR,MAXBUF,"Loader/Linker error: Incorrect module API version: %d (our version: %d)",v.API,API_VERSION);
                                        return false;