]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/extra/m_mssql.cpp
Merge pull request #109 from Justasic/insp20
[user/henk/code/inspircd.git] / src / modules / extra / m_mssql.cpp
index f1f155e7992a0c59f3aefb52ea28de5331fc813e..eca646fcfb8d2e1620fadf8198a7576cf51868d8 100644 (file)
@@ -1,16 +1,26 @@
-/*              +------------------------------------+
- *              | Inspire Internet Relay Chat Daemon |
- *              +------------------------------------+
+/*
+ * InspIRCd -- Internet Relay Chat Daemon
  *
- *     InspIRCd: (C) 2002-2009 InspIRCd Development Team
- * See: http://wiki.inspircd.org/Credits
+ *   Copyright (C) 2008-2009 Dennis Friis <peavey@inspircd.org>
+ *   Copyright (C) 2009 Daniel De Graaf <danieldg@inspircd.org>
+ *   Copyright (C) 2008-2009 Craig Edwards <craigedwards@brainbox.cc>
+ *   Copyright (C) 2008 Robin Burchell <robin+git@viroteck.net>
+ *   Copyright (C) 2008 Pippijn van Steenhoven <pip88nl@gmail.com>
  *
- * This program is free but copyrighted software; see
- *                       the file COPYING for details.
+ * This file is part of InspIRCd.  InspIRCd is free software: you can
+ * redistribute it and/or modify it under the terms of the GNU General Public
+ * License as published by the Free Software Foundation, version 2.
  *
- * ---------------------------------------------------
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+ * FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+ * details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
+
 #include "inspircd.h"
 #include <tds.h>
 #include <tdsconvert.h>
@@ -35,7 +45,7 @@ typedef std::deque<MsSQLResult*> ResultQueue;
 unsigned long count(const char * const str, char a)
 {
        unsigned long n = 0;
-       for (const char *p = reinterpret_cast<const char *>(str); *p; ++p)
+       for (const char *p = str; *p; ++p)
        {
                if (*p == '?')
                        ++n;
@@ -50,13 +60,9 @@ Mutex* LoggingMutex;
 class QueryThread : public SocketThread
 {
   private:
-       ModuleMsSQL* Parent;
-       InspIRCd* ServerInstance;
+       ModuleMsSQL* const Parent;
   public:
-       QueryThread(InspIRCd* si, ModuleMsSQL* mod)
-       : SocketThread(si), Parent(mod), ServerInstance(si)
-       {
-       }
+       QueryThread(ModuleMsSQL* mod) : Parent(mod) { }
        ~QueryThread() { }
        virtual void Run();
        virtual void OnNotify();
@@ -232,7 +238,6 @@ class SQLConn : public classbase
 {
  private:
        ResultQueue results;
-       InspIRCd* ServerInstance;
        Module* mod;
        SQLhost host;
        TDSLOGIN* login;
@@ -242,8 +247,8 @@ class SQLConn : public classbase
  public:
        QueryQueue queue;
 
-       SQLConn(InspIRCd* SI, Module* m, const SQLhost& hi)
-       : ServerInstance(SI), mod(m), host(hi), login(NULL), sock(NULL), context(NULL)
+       SQLConn(Module* m, const SQLhost& hi)
+       : mod(m), host(hi), login(NULL), sock(NULL), context(NULL)
        {
                if (OpenDB())
                {
@@ -280,7 +285,7 @@ class SQLConn : public classbase
                CloseDB();
        }
 
-       SQLerror Query(SQLrequest &req)
+       SQLerror Query(SQLrequestreq)
        {
                if (!sock)
                        return SQLerror(SQL_BAD_CONN, "Socket was NULL, check if SQL server is running.");
@@ -297,17 +302,17 @@ class SQLConn : public classbase
                /* The length of the longest parameter */
                maxparamlen = 0;
 
-               for(ParamL::iterator i = req.query.p.begin(); i != req.query.p.end(); i++)
+               for(ParamL::iterator i = req->query.p.begin(); i != req->query.p.end(); i++)
                {
                        if (i->size() > maxparamlen)
                                maxparamlen = i->size();
                }
 
                /* How many params are there in the query? */
-               paramcount = count(req.query.q.c_str(), '?');
+               paramcount = count(req->query.q.c_str(), '?');
 
                /* This stores copy of params to be inserted with using numbered params 1;3B*/
-               ParamL paramscopy(req.query.p);
+               ParamL paramscopy(req->query.p);
 
                /* To avoid a lot of allocations, allocate enough memory for the biggest the escaped query could possibly be.
                 * sizeofquery + (maxtotalparamlength*2) + 1
@@ -315,12 +320,12 @@ class SQLConn : public classbase
                 * The +1 is for null-terminating the string
                 */
 
-               query = new char[req.query.q.length() + (maxparamlen*paramcount*2) + 1];
+               query = new char[req->query.q.length() + (maxparamlen*paramcount*2) + 1];
                queryend = query;
 
-               for(unsigned long i = 0; i < req.query.q.length(); i++)
+               for(unsigned long i = 0; i < req->query.q.length(); i++)
                {
-                       if(req.query.q[i] == '?')
+                       if(req->query.q[i] == '?')
                        {
                                /* We found a place to substitute..what fun.
                                 * use mssql calls to escape and write the
@@ -343,11 +348,11 @@ class SQLConn : public classbase
                                /* Let's check if it's a numbered param. And also calculate it's number.
                                 */
 
-                               while ((i < req.query.q.length() - 1) && (req.query.q[i+1] >= '0') && (req.query.q[i+1] <= '9'))
+                               while ((i < req->query.q.length() - 1) && (req->query.q[i+1] >= '0') && (req->query.q[i+1] <= '9'))
                                {
                                        numbered = true;
                                        ++i;
-                                       paramnum = paramnum * 10 + req.query.q[i] - '0';
+                                       paramnum = paramnum * 10 + req->query.q[i] - '0';
                                }
 
                                if (paramnum > paramscopy.size() - 1)
@@ -383,13 +388,13 @@ class SQLConn : public classbase
                                        }
                                        delete[] escaped;
                                }
-                               else if (req.query.p.size())
+                               else if (req->query.p.size())
                                {
                                        /* Custom escaping for this one. converting ' to '' should make SQL Server happy. Ugly but fast :]
                                         */
-                                       char* escaped = new char[(req.query.p.front().length() * 2) + 1];
+                                       char* escaped = new char[(req->query.p.front().length() * 2) + 1];
                                        char* escend = escaped;
-                                       for (std::string::iterator p = req.query.p.front().begin(); p < req.query.p.front().end(); p++)
+                                       for (std::string::iterator p = req->query.p.front().begin(); p < req->query.p.front().end(); p++)
                                        {
                                                if (*p == '\'')
                                                {
@@ -408,31 +413,31 @@ class SQLConn : public classbase
                                                queryend++;
                                        }
                                        delete[] escaped;
-                                       req.query.p.pop_front();
+                                       req->query.p.pop_front();
                                }
                                else
                                        break;
                        }
                        else
                        {
-                               *queryend = req.query.q[i];
+                               *queryend = req->query.q[i];
                                queryend++;
                        }
                }
                *queryend = 0;
-               req.query.q = query;
+               req->query.q = query;
 
-               MsSQLResult* res = new MsSQLResult((Module*)mod, req.GetSource(), req.id);
+               MsSQLResult* res = new MsSQLResult((Module*)mod, req->source, req->id);
                res->dbid = host.id;
-               res->query = req.query.q;
+               res->query = req->query.q;
 
-               char* msquery = strdup(req.query.q.data());
+               char* msquery = strdup(req->query.q.data());
                LoggingMutex->Lock();
                ServerInstance->Logs->Log("m_mssql",DEBUG,"doing Query: %s",msquery);
                LoggingMutex->Unlock();
                if (tds_submit_query(sock, msquery) != TDS_SUCCEED)
                {
-                       std::string error("failed to execute: "+std::string(req.query.q.data()));
+                       std::string error("failed to execute: "+std::string(req->query.q.data()));
                        delete[] query;
                        delete res;
                        free(msquery);
@@ -511,7 +516,7 @@ class SQLConn : public classbase
        {
                SQLConn* sc = (SQLConn*)pContext->parent;
                LoggingMutex->Lock();
-               sc->ServerInstance->Logs->Log("m_mssql", DEBUG, "Message for DB with id: %s -> %s", sc->host.id.c_str(), pMessage->message);
+               ServerInstance->Logs->Log("m_mssql", DEBUG, "Message for DB with id: %s -> %s", sc->host.id.c_str(), pMessage->message);
                LoggingMutex->Unlock();
                return 0;
        }
@@ -520,7 +525,7 @@ class SQLConn : public classbase
        {
                SQLConn* sc = (SQLConn*)pContext->parent;
                LoggingMutex->Lock();
-               sc->ServerInstance->Logs->Log("m_mssql", DEFAULT, "Error for DB with id: %s -> %s", sc->host.id.c_str(), pMessage->message);
+               ServerInstance->Logs->Log("m_mssql", DEFAULT, "Error for DB with id: %s -> %s", sc->host.id.c_str(), pMessage->message);
                LoggingMutex->Unlock();
                return 0;
        }
@@ -600,7 +605,7 @@ class SQLConn : public classbase
                {
                        MsSQLResult* res = results[0];
                        ResultsMutex->Lock();
-                       if (res->GetDest())
+                       if (res->dest)
                        {
                                res->Send();
                        }
@@ -629,8 +634,8 @@ class SQLConn : public classbase
 
        void DoLeadingQuery()
        {
-               SQLrequest& req = queue.front();
-               req.error = Query(req);
+               SQLrequest* req = queue.front();
+               req->error = Query(req);
        }
 
 };
@@ -641,41 +646,35 @@ class ModuleMsSQL : public Module
  private:
        unsigned long currid;
        QueryThread* queryDispatcher;
+       ServiceProvider sqlserv;
 
  public:
-       ModuleMsSQL(InspIRCd* Me)
-       : Module(Me), currid(0)
+       ModuleMsSQL()
+       : currid(0), sqlserv(this, "SQL/mssql", SERVICE_DATA)
        {
                LoggingMutex = new Mutex();
                ResultsMutex = new Mutex();
+               queryDispatcher = new QueryThread(this);
+       }
 
-               ServerInstance->Modules->UseInterface("SQLutils");
-
-               if (!ServerInstance->Modules->PublishFeature("SQL", this))
-               {
-                       throw ModuleException("m_mssql: Unable to publish feature 'SQL'");
-               }
-
+       void init()
+       {
                ReadConf();
 
-               queryDispatcher = new QueryThread(ServerInstance, this);
                ServerInstance->Threads->Start(queryDispatcher);
 
-               ServerInstance->Modules->PublishInterface("SQL", this);
-               Implementation eventlist[] = { I_OnRequest, I_OnRehash };
-               ServerInstance->Modules->Attach(eventlist, this, 2);
+               Implementation eventlist[] = { I_OnRehash };
+               ServerInstance->Modules->Attach(eventlist, this, 1);
+               ServerInstance->Modules->AddService(sqlserv);
        }
 
        virtual ~ModuleMsSQL()
        {
+               queryDispatcher->join();
                delete queryDispatcher;
                ClearQueue();
                ClearAllConnections();
 
-               ServerInstance->Modules->UnpublishInterface("SQL", this);
-               ServerInstance->Modules->UnpublishFeature("SQL");
-               ServerInstance->Modules->DoneWithInterface("SQLutils");
-
                delete LoggingMutex;
                delete ResultsMutex;
        }
@@ -708,7 +707,7 @@ class ModuleMsSQL : public Module
 
        bool HostInConf(const SQLhost &h)
        {
-               ConfigReader conf(ServerInstance);
+               ConfigReader conf;
                for(int i = 0; i < conf.Enumerate("database"); i++)
                {
                        SQLhost host;
@@ -728,7 +727,7 @@ class ModuleMsSQL : public Module
        {
                ClearOldConnections();
 
-               ConfigReader conf(ServerInstance);
+               ConfigReader conf;
                for(int i = 0; i < conf.Enumerate("database"); i++)
                {
                        SQLhost host;
@@ -759,7 +758,7 @@ class ModuleMsSQL : public Module
 
                SQLConn* newconn;
 
-               newconn = new SQLConn(ServerInstance, this, hi);
+               newconn = new SQLConn(this, hi);
 
                connections.insert(std::make_pair(hi.id, newconn));
        }
@@ -781,49 +780,39 @@ class ModuleMsSQL : public Module
 
        void ClearAllConnections()
        {
-               ConnMap::iterator i;
-               while ((i = connections.begin()) != connections.end())
-               {
-                       connections.erase(i);
+               for(ConnMap::iterator i = connections.begin(); i != connections.end(); ++i)
                        delete i->second;
-               }
+               connections.clear();
        }
 
-       virtual void OnRehash(User* user, const std::string &parameter)
+       virtual void OnRehash(User* user)
        {
                queryDispatcher->LockQueue();
                ReadConf();
                queryDispatcher->UnlockQueueWakeup();
        }
 
-       virtual const char* OnRequest(Request* request)
+       void OnRequest(Request& request)
        {
-               if(strcmp(SQLREQID, request->GetId()) == 0)
+               if(strcmp(SQLREQID, request.id) == 0)
                {
-                       SQLrequest* req = (SQLrequest*)request;
+                       SQLrequest* req = (SQLrequest*)&request;
 
                        queryDispatcher->LockQueue();
 
                        ConnMap::iterator iter;
 
-                       const char* returnval = NULL;
-
                        if((iter = connections.find(req->dbid)) != connections.end())
                        {
                                req->id = NewID();
-                               iter->second->queue.push(*req);
-                               returnval= SQLSUCCESS;
+                               iter->second->queue.push(new SQLrequest(*req));
                        }
                        else
                        {
                                req->error.Id(SQL_BAD_DBID);
                        }
-
                        queryDispatcher->UnlockQueueWakeup();
-
-                       return returnval;
                }
-               return NULL;
        }
 
        unsigned long NewID()
@@ -836,14 +825,14 @@ class ModuleMsSQL : public Module
 
        virtual Version GetVersion()
        {
-               return Version("$Id$", VF_VENDOR | VF_SERVICEPROVIDER, API_VERSION);
+               return Version("MsSQL provider", VF_VENDOR);
        }
 
 };
 
 void QueryThread::OnNotify()
 {
-       mod->SendQueue();
+       Parent->SendQueue();
 }
 
 void QueryThread::Run()