]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules.cpp
Decide that it wasn't quite appropriate :(
[user/henk/code/inspircd.git] / src / modules.cpp
index 62e84f218781386da802463cf138546c3bbc32b3..9b6a0d826a5afdc3d10c8adb6035c2ffce3dfa42 100644 (file)
@@ -2,7 +2,7 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  Inspire is copyright (C) 2002-2004 ChatSpike-Dev.
+ *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
  *                       E-mail:
  *                <brain@chatspike.net>
  *               <Craig@chatspike.net>
@@ -19,7 +19,6 @@ using namespace std;
 #include "inspircd_config.h"
 #include "inspircd.h"
 #include "inspircd_io.h"
-#include "inspircd_util.h"
 #include <unistd.h>
 #include <sys/errno.h>
 #include <time.h>
@@ -49,35 +48,23 @@ using namespace std;
 #include "socket.h"
 #include "socketengine.h"
 #include "typedefs.h"
+#include "modules.h"
+#include "command_parse.h"
 
-extern SocketEngine* SE;
 extern ServerConfig *Config;
+extern InspIRCd* ServerInstance;
 extern int MODCOUNT;
 extern std::vector<Module*> modules;
 extern std::vector<ircd_module*> factory;
-extern std::vector<std::string> include_stack;
 extern std::vector<InspSocket*> module_sockets;
-
+extern std::vector<userrec*> local_users;
 extern time_t TIME;
-extern int WHOWAS_STALE;
-extern int WHOWAS_MAX;
-extern time_t startup_time;
-extern std::vector<std::string> module_names;
-extern int boundPortCount;
-extern int portCount;
-extern int ports[MAXSOCKS];
-
 class Server;
-
-extern userrec* fd_ref_table[65536];
+extern userrec* fd_ref_table[MAX_DESCRIPTORS];
 
 extern user_hash clientlist;
 extern chan_hash chanlist;
-extern whowas_hash whowas;
 extern command_table cmdlist;
-extern file_cache MOTD;
-extern file_cache RULES;
-extern address_cache IP;                                     
 ExtModeList EMode;
 
 // returns true if an extended mode character is in use
@@ -218,7 +205,7 @@ Module* Event::GetSource()
 
 char* Event::Send()
 {
-        FOREACH_MOD OnEvent(this);
+        FOREACH_MOD(I_OnEvent,OnEvent(this));
         return NULL;
 }
 
@@ -236,7 +223,7 @@ void                Module::OnUserConnect(userrec* user) { }
 void           Module::OnUserQuit(userrec* user, std::string message) { }
 void           Module::OnUserDisconnect(userrec* user) { }
 void           Module::OnUserJoin(userrec* user, chanrec* channel) { }
-void           Module::OnUserPart(userrec* user, chanrec* channel) { }
+void           Module::OnUserPart(userrec* user, chanrec* channel, std::string partmessage) { }
 void           Module::OnRehash(std::string parameter) { }
 void           Module::OnServerRaw(std::string &raw, bool inbound, userrec* user) { }
 int            Module::OnUserPreJoin(userrec* user, chanrec* chan, const char* cname) { return 0; }
@@ -244,11 +231,12 @@ int               Module::OnExtendedMode(userrec* user, void* target, char modechar, int type
 void           Module::OnMode(userrec* user, void* dest, int target_type, std::string text) { };
 Version                Module::GetVersion() { return Version(1,0,0,0,VF_VENDOR); }
 void           Module::OnOper(userrec* user, std::string opertype) { };
+void           Module::OnPostOper(userrec* user, std::string opertype) { };
 void           Module::OnInfo(userrec* user) { };
 void           Module::OnWhois(userrec* source, userrec* dest) { };
 int            Module::OnUserPreInvite(userrec* source,userrec* dest,chanrec* channel) { return 0; };
-int            Module::OnUserPreMessage(userrec* user,void* dest,int target_type, std::string &text) { return 0; };
-int            Module::OnUserPreNotice(userrec* user,void* dest,int target_type, std::string &text) { return 0; };
+int            Module::OnUserPreMessage(userrec* user,void* dest,int target_type, std::string &text,char status) { return 0; };
+int            Module::OnUserPreNotice(userrec* user,void* dest,int target_type, std::string &text,char status) { return 0; };
 int            Module::OnUserPreNick(userrec* user, std::string newnick) { return 0; };
 void           Module::OnUserPostNick(userrec* user, std::string oldnick) { };
 int            Module::OnAccessCheck(userrec* source,userrec* dest,chanrec* channel,int access_type) { return ACR_DEFAULT; };
@@ -258,7 +246,7 @@ void                Module::OnLoadModule(Module* mod,std::string name) { };
 void           Module::OnUnloadModule(Module* mod,std::string name) { };
 void           Module::OnBackgroundTimer(time_t curtime) { };
 void           Module::OnSendList(userrec* user, chanrec* channel, char mode) { };
-int            Module::OnPreCommand(std::string command, char **parameters, int pcnt, userrec *user) { return 0; };
+int            Module::OnPreCommand(std::string command, char **parameters, int pcnt, userrec *user, bool validated) { return 0; };
 bool           Module::OnCheckReady(userrec* user) { return true; };
 void           Module::OnUserRegister(userrec* user) { };
 int            Module::OnUserPreKick(userrec* source, userrec* user, chanrec* chan, std::string reason) { return 0; };
@@ -268,7 +256,7 @@ int         Module::OnCheckInvite(userrec* user, chanrec* chan) { return 0; };
 int            Module::OnCheckKey(userrec* user, chanrec* chan, std::string keygiven) { return 0; };
 int            Module::OnCheckLimit(userrec* user, chanrec* chan) { return 0; };
 int            Module::OnCheckBan(userrec* user, chanrec* chan) { return 0; };
-void           Module::OnStats(char symbol) { };
+int            Module::OnStats(char symbol, userrec* user) { return 0; };
 int            Module::OnChangeLocalUserHost(userrec* user, std::string newhost) { return 0; };
 int            Module::OnChangeLocalUserGECOS(userrec* user, std::string newhost) { return 0; };
 int            Module::OnLocalTopicChange(userrec* user, chanrec* chan, std::string topic) { return 0; };
@@ -283,8 +271,8 @@ void                Module::OnRawSocketAccept(int fd, std::string ip, int localport) { };
 int            Module::OnRawSocketWrite(int fd, char* buffer, int count) { return 0; };
 void           Module::OnRawSocketClose(int fd) { };
 int            Module::OnRawSocketRead(int fd, char* buffer, unsigned int count, int &readresult) { return 0; };
-void           Module::OnUserMessage(userrec* user, void* dest, int target_type, std::string text) { };
-void           Module::OnUserNotice(userrec* user, void* dest, int target_type, std::string text) { };
+void           Module::OnUserMessage(userrec* user, void* dest, int target_type, std::string text, char status) { };
+void           Module::OnUserNotice(userrec* user, void* dest, int target_type, std::string text, char status) { };
 void           Module::OnRemoteKill(userrec* source, userrec* dest, std::string reason) { };
 void           Module::OnUserInvite(userrec* source,userrec* dest,chanrec* channel) { };
 void           Module::OnPostLocalTopicChange(userrec* user, chanrec* chan, std::string topic) { };
@@ -294,6 +282,7 @@ void                Module::OnSyncChannel(chanrec* chan, Module* proto, void* opaque) { };
 void           Module::ProtoSendMode(void* opaque, int target_type, void* target, std::string modeline) { };
 void           Module::OnSyncChannelMetaData(chanrec* chan, Module* proto,void* opaque, std::string extname) { };
 void           Module::OnSyncUserMetaData(userrec* user, Module* proto,void* opaque, std::string extname) { };
+void           Module::OnSyncOtherMetaData(Module* proto, void* opaque) { };
 void           Module::OnDecodeMetaData(int target_type, void* target, std::string extname, std::string extdata) { };
 void           Module::ProtoSendMetaData(void* opaque, int target_type, void* target, std::string extname, std::string extdata) { };
 void           Module::OnWallops(userrec* user, std::string text) { };
@@ -310,6 +299,11 @@ void               Module::OnDelKLine(userrec* source, std::string hostmask) { };
 void           Module::OnDelQLine(userrec* source, std::string nickmask) { };
 void           Module::OnDelELine(userrec* source, std::string hostmask) { };
 void           Module::OnCleanup(int target_type, void* item) { };
+void           Module::Implements(char* Implements) { for (int j = 0; j < 255; j++) Implements[j] = 0; };
+void           Module::OnChannelDelete(chanrec* chan) { };
+Priority       Module::Prioritize() { return PRIORITY_DONTCARE; }
+void           Module::OnSetAway(userrec* user) { };
+void           Module::OnCancelAway(userrec* user) { };
 
 /* server is a wrapper class that provides methods to all of the C-style
  * exports in the core
@@ -328,12 +322,63 @@ void Server::AddSocket(InspSocket* sock)
        module_sockets.push_back(sock);
 }
 
+void Server::RemoveSocket(InspSocket* sock)
+{
+        for (std::vector<InspSocket*>::iterator a = module_sockets.begin(); a < module_sockets.end(); a++)
+        {
+                InspSocket* s = (InspSocket*)*a;
+                if (s == sock)
+                {
+                       log(DEBUG,"Forcibly removed socket");
+                        ServerInstance->SE->DelFd(s->GetFd());
+                        s->Close();
+                        module_sockets.erase(a);
+                        delete s;
+                        return;
+                }
+        }
+}
+
+long Server::PriorityAfter(std::string modulename)
+{
+       for (unsigned int j = 0; j < Config->module_names.size(); j++)
+       {
+               if (Config->module_names[j] == modulename)
+               {
+                       return ((j << 8) | PRIORITY_AFTER);
+               }
+       }
+       return PRIORITY_DONTCARE;
+}
+
+long Server::PriorityBefore(std::string modulename)
+{
+       for (unsigned int j = 0; j < Config->module_names.size(); j++)
+       {
+               if (Config->module_names[j] == modulename)
+               {
+                       return ((j << 8) | PRIORITY_BEFORE);
+               }
+       }
+       return PRIORITY_DONTCARE;
+}
+
 void Server::RehashServer()
 {
        WriteOpers("*** Rehashing config file");
        Config->Read(false,NULL);
 }
 
+ServerConfig* Server::GetConfig()
+{
+       return Config;
+}
+
+std::string Server::GetVersion()
+{
+       return ServerInstance->GetVersionString();
+}
+
 void Server::DelSocket(InspSocket* sock)
 {
        for (std::vector<InspSocket*>::iterator a = module_sockets.begin(); a < module_sockets.end(); a++)
@@ -346,6 +391,34 @@ void Server::DelSocket(InspSocket* sock)
        }
 }
 
+long Server::GetChannelCount()
+{
+       return (long)chanlist.size();
+}
+
+/* This is ugly, yes, but hash_map's arent designed to be
+ * addressed in this manner, and this is a bit of a kludge.
+ * Luckily its a specialist function and rarely used by
+ * many modules (in fact, it was specially created to make
+ * m_safelist possible, initially).
+ */
+
+chanrec* Server::GetChannelIndex(long index)
+{
+       int target = 0;
+       for (chan_hash::iterator n = chanlist.begin(); n != chanlist.end(); n++, target++)
+       {
+               if (index == target)
+                       return n->second;
+       }
+       return NULL;
+}
+
+void Server::AddTimer(InspTimer* T)
+{
+       ::AddTimer(T);
+}
+
 void Server::SendOpers(std::string s)
 {
        WriteOpers("%s",s.c_str());
@@ -378,10 +451,10 @@ chanuserlist Server::GetUsers(chanrec* chan)
 {
        chanuserlist userl;
        userl.clear();
-       std::vector<char*> *list = chan->GetUsers();
-       for (std::vector<char*>::iterator i = list->begin(); i != list->end(); i++)
+       std::map<char*,char*> *list = chan->GetUsers();
+       for (std::map<char*,char*>::iterator i = list->begin(); i != list->end(); i++)
        {
-               char* o = *i;
+               char* o = i->second;
                userl.push_back((userrec*)o);
        }
        return userl;
@@ -391,6 +464,18 @@ void Server::ChangeUserNick(userrec* user, std::string nickname)
        force_nickchange(user,nickname.c_str());
 }
 
+void Server::KickUser(userrec* source, userrec* target, chanrec* chan, std::string reason)
+{
+       if (source)
+       {
+               kick_channel(source,target,chan,(char*)reason.c_str());
+       }
+       else
+       {
+               server_kick_channel(target,chan,(char*)reason.c_str(),true);
+       }
+}
+
 void Server::QuitUser(userrec* user, std::string reason)
 {
        kill_link(user,reason.c_str());
@@ -403,12 +488,12 @@ bool Server::IsUlined(std::string server)
 
 void Server::CallCommandHandler(std::string commandname, char** parameters, int pcnt, userrec* user)
 {
-       call_handler(commandname.c_str(),parameters,pcnt,user);
+       ServerInstance->Parser->CallHandler(commandname,parameters,pcnt,user);
 }
 
 bool Server::IsValidModuleCommand(std::string commandname, int pcnt, userrec* user)
 {
-       return is_valid_cmd(commandname.c_str(), pcnt, user);
+       return ServerInstance->Parser->IsValidCommand(commandname, pcnt, user);
 }
 
 void Server::Log(int level, std::string s)
@@ -416,29 +501,33 @@ void Server::Log(int level, std::string s)
        log(level,"%s",s.c_str());
 }
 
-void Server::AddCommand(char* cmd, handlerfunc f, char flags, int minparams, char* source)
+void Server::AddCommand(command_t *f)
 {
-       createcommand(cmd,f,flags,minparams,source);
+       if (!ServerInstance->Parser->CreateCommand(f))
+       {
+               ModuleException err("Command "+std::string(f->command)+" already exists.");
+               throw (err);
+       }
 }
 
 void Server::SendMode(char **parameters, int pcnt, userrec *user)
 {
-       server_mode(parameters,pcnt,user);
+       ServerInstance->ModeGrok->ServerMode(parameters,pcnt,user);
 }
 
 void Server::Send(int Socket, std::string s)
 {
-       Write(Socket,"%s",s.c_str());
+       Write_NoFormat(Socket,s.c_str());
 }
 
 void Server::SendServ(int Socket, std::string s)
 {
-       WriteServ(Socket,"%s",s.c_str());
+       WriteServ_NoFormat(Socket,s.c_str());
 }
 
 void Server::SendFrom(int Socket, userrec* User, std::string s)
 {
-       WriteFrom(Socket,User,"%s",s.c_str());
+       WriteFrom_NoFormat(Socket,User,s.c_str());
 }
 
 void Server::SendTo(userrec* Source, userrec* Dest, std::string s)
@@ -451,24 +540,24 @@ void Server::SendTo(userrec* Source, userrec* Dest, std::string s)
        else
        {
                // otherwise it comes from the user specified
-               WriteTo(Source,Dest,"%s",s.c_str());
+               WriteTo_NoFormat(Source,Dest,s.c_str());
        }
 }
 
 void Server::SendChannelServerNotice(std::string ServName, chanrec* Channel, std::string text)
 {
-       WriteChannelWithServ((char*)ServName.c_str(), Channel, "%s", text.c_str());
+       WriteChannelWithServ_NoFormat((char*)ServName.c_str(), Channel, text.c_str());
 }
 
 void Server::SendChannel(userrec* User, chanrec* Channel, std::string s,bool IncludeSender)
 {
        if (IncludeSender)
        {
-               WriteChannel(Channel,User,"%s",s.c_str());
+               WriteChannel_NoFormat(Channel,User,s.c_str());
        }
        else
        {
-               ChanExceptSender(Channel,User,"%s",s.c_str());
+               ChanExceptSender_NoFormat(Channel,User,0,s.c_str());
        }
 }
 
@@ -481,11 +570,11 @@ void Server::SendCommon(userrec* User, std::string text,bool IncludeSender)
 {
        if (IncludeSender)
        {
-               WriteCommon(User,"%s",text.c_str());
+               WriteCommon_NoFormat(User,text.c_str());
        }
        else
        {
-               WriteCommonExcept(User,"%s",text.c_str());
+               WriteCommonExcept_NoFormat(User,text.c_str());
        }
 }
 
@@ -536,22 +625,22 @@ bool Server::IsOnChannel(userrec* User, chanrec* Chan)
 
 std::string Server::GetServerName()
 {
-       return getservername();
+       return Config->ServerName;
 }
 
 std::string Server::GetNetworkName()
 {
-       return getnetworkname();
+       return Config->Network;
 }
 
 std::string Server::GetServerDescription()
 {
-       return getserverdesc();
+       return Config->ServerDesc;
 }
 
 Admin Server::GetAdmin()
 {
-       return Admin(getadminname(),getadminemail(),getadminnick());
+       return Admin(Config->AdminName,Config->AdminEmail,Config->AdminNick);
 }
 
 
@@ -562,24 +651,28 @@ bool Server::AddExtendedMode(char modechar, int type, bool requires_oper, int pa
        {
                if (type == MT_SERVER)
                {
-                       log(DEBUG,"*** API ERROR *** Modes of type MT_SERVER are reserved for future expansion");
+                       ModuleException e("Modes of type MT_SERVER are reserved for future expansion");
+                       throw(e);
                        return false;
                }
                if (((params_when_on>0) || (params_when_off>0)) && (type == MT_CLIENT))
                {
-                       log(DEBUG,"*** API ERROR *** Parameters on MT_CLIENT modes are not supported");
+                       ModuleException e("Parameters on MT_CLIENT modes are not supported");
+                       throw(e);
                        return false;
                }
                if ((params_when_on>1) || (params_when_off>1))
                {
-                       log(DEBUG,"*** API ERROR *** More than one parameter for an MT_CHANNEL mode is not yet supported");
+                       ModuleException e("More than one parameter for an MT_CHANNEL mode is not yet supported");
+                       throw(e);
                        return false;
                }
                return DoAddExtendedMode(modechar,type,requires_oper,params_when_on,params_when_off);
        }
        else
        {
-               log(DEBUG,"*** API ERROR *** Muppet modechar detected.");
+               ModuleException e("Muppet modechar detected.");
+               throw(e);
        }
        return false;
 }
@@ -601,10 +694,18 @@ int Server::CountUsers(chanrec* c)
 bool Server::UserToPseudo(userrec* user,std::string message)
 {
        unsigned int old_fd = user->fd;
-       user->fd = FD_MAGIC_NUMBER;
-       user->ClearBuffer();
        Write(old_fd,"ERROR :Closing link (%s@%s) [%s]",user->ident,user->host,message.c_str());
-       SE->DelFd(old_fd);
+       user->FlushWriteBuf();
+       user->ClearBuffer();
+       user->fd = FD_MAGIC_NUMBER;
+
+       if (find(local_users.begin(),local_users.end(),user) != local_users.end())
+       {
+               local_users.erase(find(local_users.begin(),local_users.end(),user));
+               log(DEBUG,"Delete local user");
+       }
+
+       ServerInstance->SE->DelFd(old_fd);
         shutdown(old_fd,2);
         close(old_fd);
        return true;
@@ -612,12 +713,25 @@ bool Server::UserToPseudo(userrec* user,std::string message)
 
 bool Server::PseudoToUser(userrec* alive,userrec* zombie,std::string message)
 {
+       log(DEBUG,"PseudoToUser");
        zombie->fd = alive->fd;
+       FOREACH_MOD(I_OnUserQuit,OnUserQuit(alive,message));
        alive->fd = FD_MAGIC_NUMBER;
+       alive->FlushWriteBuf();
        alive->ClearBuffer();
-       Write(zombie->fd,":%s!%s@%s NICK %s",alive->nick,alive->ident,alive->host,zombie->nick);
+       // save these for later
+       std::string oldnick = alive->nick;
+       std::string oldhost = alive->host;
+       std::string oldident = alive->ident;
        kill_link(alive,message.c_str());
+        if (find(local_users.begin(),local_users.end(),alive) != local_users.end())
+        {
+               local_users.erase(find(local_users.begin(),local_users.end(),alive));
+               log(DEBUG,"Delete local user");
+        }
+       // Fix by brain - cant write the user until their fd table entry is updated
        fd_ref_table[zombie->fd] = zombie;
+       Write(zombie->fd,":%s!%s@%s NICK %s",oldnick.c_str(),oldident.c_str(),oldhost.c_str(),zombie->nick);
         for (unsigned int i = 0; i < zombie->chans.size(); i++)
         {
                 if (zombie->chans[i].channel != NULL)
@@ -637,6 +751,9 @@ bool Server::PseudoToUser(userrec* alive,userrec* zombie,std::string message)
                         }
                 }
         }
+       if ((find(local_users.begin(),local_users.end(),zombie) == local_users.end()) && (zombie->fd != FD_MAGIC_NUMBER))
+               local_users.push_back(zombie);
+
        return true;
 }
 
@@ -697,26 +814,26 @@ long Server::CalcDuration(std::string delta)
 
 bool Server::IsValidMask(std::string mask)
 {
-       const char* dest = mask.c_str();
+       char* dest = (char*)mask.c_str();
         if (strchr(dest,'!')==0)
                 return false;
         if (strchr(dest,'@')==0)
                 return false;
-        for (unsigned int i = 0; i < strlen(dest); i++)
-                if (dest[i] < 32)
+        for (char* i = dest; *i; i++)
+                if (*i < 32)
                         return false;
-        for (unsigned int i = 0; i < strlen(dest); i++)
-                if (dest[i] > 126)
+        for (char* i = dest; *i; i++)
+                if (*i > 126)
                         return false;
         unsigned int c = 0;
-        for (unsigned int i = 0; i < strlen(dest); i++)
-                if (dest[i] == '!')
+        for (char* i = dest; *i; i++)
+                if (*i == '!')
                         c++;
         if (c>1)
                 return false;
         c = 0;
-        for (unsigned int i = 0; i < strlen(dest); i++)
-                if (dest[i] == '@')
+        for (char* i = dest; *i; i++)
+                if (*i == '@')
                         c++;
         if (c>1)
                 return false;
@@ -728,7 +845,7 @@ Module* Server::FindModule(std::string name)
 {
        for (int i = 0; i <= MODCOUNT; i++)
        {
-               if (module_names[i] == name)
+               if (Config->module_names[i] == name)
                {
                        return modules[i];
                }
@@ -738,10 +855,10 @@ Module* Server::FindModule(std::string name)
 
 ConfigReader::ConfigReader()
 {
-       include_stack.clear();
+       Config->ClearStack();
        this->cache = new std::stringstream(std::stringstream::in | std::stringstream::out);
        this->errorlog = new std::stringstream(std::stringstream::in | std::stringstream::out);
-       this->readerror = LoadConf(CONFIG_FILE,this->cache,this->errorlog);
+       this->readerror = Config->LoadConf(CONFIG_FILE,this->cache,this->errorlog);
        if (!this->readerror)
                this->error = CONF_FILE_NOT_FOUND;
 }
@@ -758,9 +875,10 @@ ConfigReader::~ConfigReader()
 
 ConfigReader::ConfigReader(std::string filename)
 {
+       Config->ClearStack();
        this->cache = new std::stringstream(std::stringstream::in | std::stringstream::out);
        this->errorlog = new std::stringstream(std::stringstream::in | std::stringstream::out);
-       this->readerror = LoadConf(filename.c_str(),this->cache,this->errorlog);
+       this->readerror = Config->LoadConf(filename.c_str(),this->cache,this->errorlog);
        if (!this->readerror)
                this->error = CONF_FILE_NOT_FOUND;
 };
@@ -772,7 +890,7 @@ std::string ConfigReader::ReadValue(std::string tag, std::string name, int index
        char n[MAXBUF];
        strlcpy(t,tag.c_str(),MAXBUF);
        strlcpy(n,name.c_str(),MAXBUF);
-       int res = ReadConf(cache,t,n,index,val);
+       int res = Config->ReadConf(cache,t,n,index,val);
        if (!res)
        {
                this->error = CONF_VALUE_NOT_FOUND;
@@ -788,7 +906,7 @@ bool ConfigReader::ReadFlag(std::string tag, std::string name, int index)
        char n[MAXBUF];
        strlcpy(t,tag.c_str(),MAXBUF);
        strlcpy(n,name.c_str(),MAXBUF);
-       int res = ReadConf(cache,t,n,index,val);
+       int res = Config->ReadConf(cache,t,n,index,val);
        if (!res)
        {
                this->error = CONF_VALUE_NOT_FOUND;
@@ -805,15 +923,15 @@ long ConfigReader::ReadInteger(std::string tag, std::string name, int index, boo
        char n[MAXBUF];
        strlcpy(t,tag.c_str(),MAXBUF);
        strlcpy(n,name.c_str(),MAXBUF);
-       int res = ReadConf(cache,t,n,index,val);
+       int res = Config->ReadConf(cache,t,n,index,val);
        if (!res)
        {
                this->error = CONF_VALUE_NOT_FOUND;
                return 0;
        }
-       for (unsigned int i = 0; i < strlen(val); i++)
+       for (char* i = val; *i; i++)
        {
-               if (!isdigit(val[i]))
+               if (!isdigit(*i))
                {
                        this->error = CONF_NOT_A_NUMBER;
                        return 0;
@@ -869,12 +987,12 @@ void ConfigReader::DumpErrors(bool bail, userrec* user)
 
 int ConfigReader::Enumerate(std::string tag)
 {
-       return EnumConf(cache,tag.c_str());
+       return Config->EnumConf(cache,tag.c_str());
 }
 
 int ConfigReader::EnumerateValues(std::string tag, int index)
 {
-       return EnumValues(cache, tag.c_str(), index);
+       return Config->EnumValues(cache, tag.c_str(), index);
 }
 
 bool ConfigReader::Verify()