That's enough tweaks for now

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

diff --git a/src/socket.cpp b/src/socket.cpp
index da179d9c8cc12d1905ad9747a22a7cf66efb543f..4c03c933dc98e0a634f9fa79cd4043337f816fbf 100644 (file)
--- a/src/socket.cpp
+++ b/src/socket.cpp
@@ -29,6 +29,7 @@ using namespace std;
 #include <sstream>
 #include <iostream>
 #include <fstream>
+#include <stdexcept>
 #include "socket.h"
 #include "inspircd.h"
 #include "inspircd_io.h"
@@ -58,9 +59,11 @@ InspSocket::InspSocket(int newfd, char* ip)
        socket_ref[this->fd] = this;
 }
 
-InspSocket::InspSocket(std::string host, int port, bool listening, unsigned long maxtime)
+InspSocket::InspSocket(std::string ahost, int aport, bool listening, unsigned long maxtime)
 {
        this->fd = -1;
+       this->host = ahost;
+       this->Buffer = "";
        if (listening) {
                if ((this->fd = OpenTCPSocket()) == ERROR)
                {
@@ -72,7 +75,7 @@ InspSocket::InspSocket(std::string host, int port, bool listening, unsigned long
                }
                else
                {
-                       if (BindSocket(this->fd,this->client,this->server,port,(char*)host.c_str()) == ERROR)
+                       if (BindSocket(this->fd,this->client,this->server,aport,(char*)ahost.c_str()) == ERROR)
                        {
                                this->Close();
                                this->fd = -1;
@@ -93,8 +96,8 @@ InspSocket::InspSocket(std::string host, int port, bool listening, unsigned long
        }
        else
        {
-               this->host = host;
-               this->port = port;
+               this->host = ahost;
+               this->port = aport;
 
                if (!inet_aton(host.c_str(),&addy))
                {
@@ -241,28 +244,41 @@ char* InspSocket::Read()
 // and should be aborted.
 int InspSocket::Write(std::string data)
 {
-       this->Buffer.append(data);
+       try
+       {
+               if ((data != "") && (this->Buffer.length() + data.length() < this->Buffer.max_size()))
+                       this->Buffer.append(data);
+       }
+       catch (std::length_error)
+       {
+               log(DEBUG,"std::length_error exception caught while appending to socket buffer!");
+               return 0;
+       }
        return data.length();
 }
 
 void InspSocket::FlushWriteBuffer()
 {
-       int result = 0;
-       if (this->Buffer.length())
+       if ((this->fd > -1) && (this->state == I_CONNECTED))
        {
-               result = send(this->fd,this->Buffer.c_str(),this->Buffer.length(),0);
-               if (result > 0)
+               int result = 0, v = 0;
+               const char* n = Buffer.c_str();
+               v = Buffer.length();
+               if (v > 0)
                {
-                       if (result == (int)this->Buffer.length())
-                       {
-                               this->Buffer = "";
-                       }
-                       else
+                       result = write(this->fd,n,v);
+                       if (result > 0)
                        {
-                               /* If we wrote some, advance the buffer forwards */
-                               char* n = (char*)this->Buffer.c_str();
-                               n += result;
-                               this->Buffer = n;
+                               if (result == v)
+                               {
+                                       Buffer = "";
+                               }
+                               else
+                               {
+                                       /* If we wrote some, advance the buffer forwards */
+                                       n += result;
+                                       Buffer = n;
+                               }
                        }
                }
        }
@@ -292,8 +308,6 @@ bool InspSocket::Poll()
        int incoming = -1;
        bool n = true;
 
-       log(DEBUG,"InspSocket::Poll()");
-       
        switch (this->state)
        {
                case I_RESOLVING:
@@ -301,7 +315,7 @@ bool InspSocket::Poll()
                        return this->DoResolve();
                break;
                case I_CONNECTING:
-                       log(DEBUG,"State = I_CONNECTED");
+                       log(DEBUG,"State = I_CONNECTING");
                        this->SetState(I_CONNECTED);
                        /* Our socket was in write-state, so delete it and re-add it
                         * in read-state.
@@ -318,9 +332,7 @@ bool InspSocket::Poll()
                        return true;
                break;
                case I_CONNECTED:
-                       log(DEBUG,"State = I_CONNECTED");
                        n = this->OnDataReady();
-                       log(DEBUG,"State return: %d",(int)n);
                        /* Flush any pending, but not till after theyre done with the event
                         * so there are less write calls involved. */
                        this->FlushWriteBuffer();