]> git.stg.codes - stg.git/blobdiff - projects/stargazer/plugins/configuration/sgconfig/rsconf.cpp
Cosmetic changes and possible sgconfig hang's fix
[stg.git] / projects / stargazer / plugins / configuration / sgconfig / rsconf.cpp
index 8ceb7e70cc0126591f866a41ff18f3a7eba77fdf..f494ee404e739aaedc4c293592ebb065bd2a91a0 100644 (file)
 *
 *******************************************************************/
 
-/*#include <unistd.h>
-#include <stdio.h>
-#include <netinet/in.h>
-#include <sys/types.h>
-#include <sys/socket.h>
-#include <arpa/inet.h>
-#include <netinet/tcp.h>
-#include <netinet/in.h>
-#include <string.h>
-#include <fcntl.h>
-#include <string.h>
-#include <stdlib.h>
-#include <pthread.h>*/
+#include <unistd.h> // cloase, usleep
 
 #include <cerrno>
 #include <csignal>
+#include <cstdio> // snprintf
 
+#include "stg/blowfish.h"
 #include "configproto.h"
-#include "blowfish.h"
+
+#ifndef ENODATA
+// FreeBSD 4.* - suxx
+#define ENODATA -1
+#endif
 
 enum CONF_STATE
     {
@@ -109,13 +103,6 @@ if (res == -1)
     errorStr = "Listen admin socket failed";
     return -1;
     }
-outerAddrLen = sizeof(outerAddr);
-
-/*if (0 != fcntl(listenSocket, F_SETFL, O_NONBLOCK))
-    {
-    errorStr = "fcntl error!";
-    return -1;
-    }*/
 
 errorStr = "";
 nonstop = true;
@@ -134,7 +121,7 @@ addr.sin_family = PF_INET;
 addr.sin_port = htons(port);
 addr.sin_addr.s_addr = inet_addr("127.0.0.1");
 
-addrLen = sizeof(outerAddr);
+addrLen = sizeof(addr);
 sock = socket(PF_INET, SOCK_STREAM, 0);
 connect(sock, (sockaddr*)&addr, addrLen);
 close(sock);
@@ -142,132 +129,113 @@ close(sock);
 return 0;
 }
 //-----------------------------------------------------------------------------
-// æÕÎËÃÉÑ ÏÂÝÅÎÉÑ Ó ËÏÎÆÉÇÕÒÁÔÏÒÏÍ
-void * CONFIGPROTO::Run(void * a)
+void CONFIGPROTO::Run()
 {
-/*
- * Function Name:ReciveSendConf
- * Parameters: void * a ÕËÁÚÁÔÅÌØ ÎÁ ÜËÚÅÍÐÌÑÒ ËÌÁÓÓÁ CONFIGPROTO
- * Description: üÔÁ ÆÕÎËÃÉÑ ÏÂÅÓÐÅÞÉ×ÁÅÔ ÓÅÔÅ×ÏÅ ×ÚÁÉÍÏÄÅÊÓÔ×ÉÅ
- *  Ó íÅÎÅÄÖÅÒÏÍ ëÌÉÅÎÔÏ×. ÷ ÅÅ ÚÁÄÁÞÉ ×ÈÏÄÉÔ: ÐÒÉÅÍ ÚÁÐÒÏÓÏ× ÐÏ TCP/IP
- *  ÉÈ ÒÁÓÛÉÆÒÏ×ËÁ, ÐÅÒÅÄÁÞÁ ÐÒÉÎÑÔÙÈ ÄÁÎÎÙÈ ÁÎÁÌÉÚÁÔÏÒÕ É ÏÔÐÒÁ×ËÁ ÏÔ×ÅÔÁ ÎÁÚÁÄ.
- * Returns: ×ÏÚ×ÒÁÝÁÅÔ NULL
- */
-
-CONFIGPROTO * cp = (CONFIGPROTO*)a;
-cp->state = confHdr;
+state = confHdr;
 
-while (cp->nonstop)
+while (nonstop)
     {
-    cp->state = confHdr;
-    cp->outerSocket = accept(cp->listenSocket,
-                             (struct sockaddr*)(&cp->outerAddr),
-                             &cp->outerAddrLen);
+    state = confHdr;
+    struct sockaddr_in outerAddr;
+    socklen_t outerAddrLen(sizeof(outerAddr));
+    int outerSocket = accept(listenSocket,
+                             (struct sockaddr*)(&outerAddr),
+                             &outerAddrLen);
 
-    if (!cp->nonstop)
+    if (!nonstop)
         {
-        continue;
+        break;
         }
 
-    if (cp->outerSocket == -1)
+    if (outerSocket == -1)
         {
         printfd(__FILE__, "accept failed\n");
-        usleep(100000);
         continue;
         }
 
-    cp->adminIP = *(unsigned int*)&(cp->outerAddr.sin_addr);
+    adminIP = *(unsigned int*)&(outerAddr.sin_addr);
 
-    /* TODO
-    if (!cp->hostAllow->HostAllowed(cp->adminIP))
-        {
-        close(outerSocket);
-        continue;
-        }*/
-
-    printfd(__FILE__, "Connection accepted from %s\n", inet_ntostring(cp->outerAddr.sin_addr.s_addr).c_str());
+    printfd(__FILE__, "Connection accepted from %s\n", inet_ntostring(outerAddr.sin_addr.s_addr).c_str());
 
-    if (cp->state == confHdr)
+    if (state == confHdr)
         {
-        if (cp->RecvHdr(cp->outerSocket) < 0)
+        if (RecvHdr(outerSocket) < 0)
             {
-            close(cp->outerSocket);
+            close(outerSocket);
             continue;
             }
-        if (cp->state == confLogin)
+        if (state == confLogin)
             {
-            if (cp->SendHdrAnswer(cp->outerSocket, ans_ok) < 0)
+            if (SendHdrAnswer(outerSocket, ans_ok) < 0)
                 {
-                close(cp->outerSocket);
+                close(outerSocket);
                 continue;
                 }
 
-            if (cp->RecvLogin(cp->outerSocket) < 0)
+            if (RecvLogin(outerSocket) < 0)
                 {
-                close(cp->outerSocket);
+                close(outerSocket);
                 continue;
                 }
 
-            if (cp->state == confLoginCipher)
+            if (state == confLoginCipher)
                 {
-                if (cp->SendLoginAnswer(cp->outerSocket, ans_ok) < 0)
+                if (SendLoginAnswer(outerSocket) < 0)
                     {
-                    close(cp->outerSocket);
+                    close(outerSocket);
                     continue;
                     }
-                if (cp->RecvLoginS(cp->outerSocket) < 0)
+                if (RecvLoginS(outerSocket) < 0)
                     {
-                    close(cp->outerSocket);
+                    close(outerSocket);
                     continue;
                     }
 
-                if (cp->state == confData)
+                if (state == confData)
                     {
-                    if (cp->SendLoginSAnswer(cp->outerSocket, ans_ok) < 0)
+                    if (SendLoginSAnswer(outerSocket, ans_ok) < 0)
                         {
-                        close(cp->outerSocket);
+                        close(outerSocket);
                         continue;
                         }
-                    if (cp->RecvData(cp->outerSocket) < 0)
+                    if (RecvData(outerSocket) < 0)
                         {
-                        close(cp->outerSocket);
+                        close(outerSocket);
                         continue;
                         }
-                    cp->state = confHdr;
+                    state = confHdr;
                     }
                 else
                     {
-                    if (cp->SendLoginSAnswer(cp->outerSocket, ans_err) < 0)
+                    if (SendLoginSAnswer(outerSocket, ans_err) < 0)
                         {
-                        close(cp->outerSocket);
+                        close(outerSocket);
                         continue;
                         }
-                    cp->WriteLogAccessFailed(cp->adminIP);
+                    WriteLogAccessFailed(adminIP);
                     }
                 }
             else
                 {
-                cp->WriteLogAccessFailed(cp->adminIP);
+                WriteLogAccessFailed(adminIP);
                 }
             }
         else
             {
-            cp->WriteLogAccessFailed(cp->adminIP);
-            if (cp->SendHdrAnswer(cp->outerSocket, ans_err) < 0)
+            WriteLogAccessFailed(adminIP);
+            if (SendHdrAnswer(outerSocket, ans_err) < 0)
                 {
-                close(cp->outerSocket);
+                close(outerSocket);
                 continue;
                 }
             }
         }
     else
         {
-        cp->WriteLogAccessFailed(cp->adminIP);
+        WriteLogAccessFailed(adminIP);
         }
-    close(cp->outerSocket);
+    close(outerSocket);
     }
-
-return NULL;
 }
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::RecvHdr(int sock)
@@ -275,8 +243,8 @@ int CONFIGPROTO::RecvHdr(int sock)
 char buf[sizeof(STG_HEADER)];
 memset(buf, 0, sizeof(STG_HEADER));
 int ret;
-int stgHdrLen = strlen(STG_HEADER);
-for (int i = 0; i < stgHdrLen; i++)
+size_t stgHdrLen = sizeof(STG_HEADER);
+for (size_t i = 0; i < stgHdrLen; i++)
     {
     ret = recv(sock, &buf[i], 1, 0);
     if (ret <= 0)
@@ -306,7 +274,7 @@ int ret;
 
 if (err)
     {
-    ret = send(sock, ERR_HEADER, sizeof(ERR_HEADER)-1, 0);
+    ret = send(sock, ERR_HEADER, sizeof(ERR_HEADER) - 1, 0);
     if (ret < 0)
         {
         WriteServLog("send ERR_HEADER error in SendHdrAnswer.");
@@ -315,7 +283,7 @@ if (err)
     }
 else
     {
-    ret = send(sock, OK_HEADER, sizeof(OK_HEADER)-1, 0);
+    ret = send(sock, OK_HEADER, sizeof(OK_HEADER) - 1, 0);
     if (ret < 0)
         {
         WriteServLog("send OK_HEADER error in SendHdrAnswer.");
@@ -328,31 +296,16 @@ return 0;
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::RecvLogin(int sock)
 {
-char login[ADM_LOGIN_LEN+1];
+char login[ADM_LOGIN_LEN + 1];
 int ret;
 
 memset(login, 0, ADM_LOGIN_LEN + 1);
 
-//printfd(__FILE__, "RecvLogin\n");
-
-/*for (int i = 0; i < ADM_LOGIN_LEN; i++)
-    {
-    ret = recv(sock, &login[i], 1, 0);
-
-    if (ret <= 0)
-        {
-        close(sock);
-        state = confHdr;
-        return ENODATA;
-        }
-    }*/
-
 ret = recv(sock, login, ADM_LOGIN_LEN, 0);
 
 if (ret < 0)
     {
     // Error in network
-    close(sock);
     state = confHdr;
     return ENODATA;
     }
@@ -360,7 +313,6 @@ if (ret < 0)
 if (ret < ADM_LOGIN_LEN)
     {
     // Error in protocol
-    close(sock);
     state = confHdr;
     return ENODATA;
     }
@@ -368,21 +320,20 @@ if (ret < ADM_LOGIN_LEN)
 if (admins->FindAdmin(login, &currAdmin))
     {
     // Admin not found
-    close(sock);
     state = confHdr;
     return ENODATA;
     }
-currAdmin.SetAdminIP(adminIP);
+currAdmin->SetIP(adminIP);
 adminLogin = login;
 state = confLoginCipher;
 return 0;
 }
 //-----------------------------------------------------------------------------
-int CONFIGPROTO::SendLoginAnswer(int sock, int)
+int CONFIGPROTO::SendLoginAnswer(int sock)
 {
 int ret;
 
-ret = send(sock, OK_LOGIN, sizeof(OK_LOGIN)-1, 0);
+ret = send(sock, OK_LOGIN, sizeof(OK_LOGIN) - 1, 0);
 if (ret < 0)
     {
     WriteServLog("Send OK_LOGIN error in SendLoginAnswer.");
@@ -395,36 +346,19 @@ int CONFIGPROTO::RecvLoginS(int sock)
 {
 char loginS[ADM_LOGIN_LEN + 1];
 char login[ADM_LOGIN_LEN + 1];
-int ret;
 BLOWFISH_CTX ctx;
 memset(loginS, 0, ADM_LOGIN_LEN + 1);
 
-//printfd(__FILE__, "RecvLoginS\n");
-
-/*for (int i = 0; i < ADM_LOGIN_LEN; i++)
-    {
-    ret = recv(sock, &loginS[i], 1, 0);
-
-    if (ret <= 0)
-        {
-        //printfd(__FILE__, "RecvLoginS close\n");
-        close(sock);
-        state = confHdr;
-        return ENODATA;
-        }
-    }*/
-
 int total = 0;
 
 while (total < ADM_LOGIN_LEN)
     {
-    ret = recv(sock, &loginS[total], ADM_LOGIN_LEN - total, 0);
+    int ret = recv(sock, &loginS[total], ADM_LOGIN_LEN - total, 0);
 
     if (ret < 0)
         {
         // Network error
         printfd(__FILE__, "recv error: '%s'\n", strerror(errno));
-        close(sock);
         state = confHdr;
         return ENODATA;
         }
@@ -432,27 +366,17 @@ while (total < ADM_LOGIN_LEN)
     total += ret;
     }
 
-// TODO: implement select on socket
-/*if (total < ADM_LOGIN_LEN)
-    {
-    // Protocol error
-    printfd(__FILE__, "Protocol error. Need %d bytes of cryptologin. Got %d bytes.\n", ADM_LOGIN_LEN, ret);
-    close(sock);
-    state = confHdr;
-    return ENODATA;
-    }*/
-
-if (currAdmin.GetLogin() == "")
+if (currAdmin->GetLogin().empty())
     {
     state = confHdr;
     return ENODATA;
     }
 
-EnDecodeInit(currAdmin.GetPassword().c_str(), ADM_PASSWD_LEN, &ctx);
+EnDecodeInit(currAdmin->GetPassword().c_str(), ADM_PASSWD_LEN, &ctx);
 
-for (int i = 0; i < ADM_LOGIN_LEN/8; i++)
+for (int i = 0; i < ADM_LOGIN_LEN / 8; i++)
     {
-    DecodeString(login + i*8, loginS + i*8, &ctx);
+    DecodeString(login + i * 8, loginS + i * 8, &ctx);
     }
 
 if (currAdmin == admins->GetNoAdmin())
@@ -462,7 +386,7 @@ if (currAdmin == admins->GetNoAdmin())
     return 0;
     }
 
-if (strncmp(currAdmin.GetLogin().c_str(), login, ADM_LOGIN_LEN) != 0)
+if (strncmp(currAdmin->GetLogin().c_str(), login, ADM_LOGIN_LEN) != 0)
     {
     state = confHdr;
     return ENODATA;
@@ -474,11 +398,9 @@ return 0;
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::SendLoginSAnswer(int sock, int err)
 {
-int ret;
-
 if (err)
     {
-    ret = send(sock, ERR_LOGINS, sizeof(ERR_LOGINS)-1, 0);
+    int ret = send(sock, ERR_LOGINS, sizeof(ERR_LOGINS) - 1, 0);
     if (ret < 0)
         {
         WriteServLog("send ERR_LOGIN error in SendLoginAnswer.");
@@ -487,7 +409,7 @@ if (err)
     }
 else
     {
-    ret = send(sock, OK_LOGINS, sizeof(OK_LOGINS)-1, 0);
+    int ret = send(sock, OK_LOGINS, sizeof(OK_LOGINS) - 1, 0);
     if (ret < 0)
         {
         WriteServLog("send OK_LOGINS error in SendLoginSAnswer.");
@@ -499,9 +421,6 @@ return 0;
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::RecvData(int sock)
 {
-//printfd(__FILE__, "RecvData\n");
-//int n = 0;
-int ret;
 char bufferS[8];
 char buffer[9];
 
@@ -510,36 +429,25 @@ buffer[8] = 0;
 requestList.clear();
 BLOWFISH_CTX ctx;
 
-EnDecodeInit(currAdmin.GetPassword().c_str(), ADM_PASSWD_LEN, &ctx);
+EnDecodeInit(currAdmin->GetPassword().c_str(), ADM_PASSWD_LEN, &ctx);
 
 while (1)
     {
-    /*ret = recv(sock, &bufferS[n++], 1, 0);
-    if (ret <= 0)
-        {
-        //printfd(__FILE__, "RecvData close\n");
-        close(sock);
-        return 0;
-        }*/
     int total = 0;
     bool done = false;
     while (total < 8)
         {
-        ret = recv(sock, &bufferS[total], 8 - total, 0);
+        int ret = recv(sock, &bufferS[total], 8 - total, 0);
         if (ret < 0)
             {
             // Network error
-            close(sock);
-            return 0;
+            return -1;
             }
 
-        if (ret < 8)
+        if (ret == 0)
             {
-            if (memchr(buffer, 0, ret) != NULL)
-                {
-                done = true;
-                break;
-                }
+            done = true;
+            break;
             }
 
         total += ret;
@@ -557,25 +465,6 @@ while (1)
             }
         return SendDataAnswer(sock);
         }
-
-    /*if (n == 8)
-        {
-        n = 0;
-        DecodeString(buffer, bufferS, &ctx);
-        requestList.push_back(std::string(buffer, 8));
-        for (int j = 0; j < 8; j++)
-            {
-            if (buffer[j] == 0)
-                {
-                // ëÏÎÅàÐÏÓÙÌËÉ
-                if (ParseCommand())
-                    {
-                    SendError("Bad command");
-                    }
-                return SendDataAnswer(sock);
-                }
-            }
-        }*/
     }
 return 0;
 }
@@ -591,22 +480,21 @@ char buff[8];
 char buffS[8];
 int n = 0;
 int k = 0;
-int ret = 0;
 
-EnDecodeInit(currAdmin.GetPassword().c_str(), ADM_PASSWD_LEN, &ctx);
+EnDecodeInit(currAdmin->GetPassword().c_str(), ADM_PASSWD_LEN, &ctx);
 
 while (li != answerList.end())
     {
     while ((*li).c_str()[k])
         {
-        buff[n%8] = (*li).c_str()[k];
+        buff[n % 8] = (*li).c_str()[k];
         n++;
         k++;
 
-        if (n%8 == 0)
+        if (n % 8 == 0)
             {
             EncodeString(buffS, buff, &ctx);
-            ret = send(sock, buffS, 8, 0);
+            int ret = send(sock, buffS, 8, 0);
             if (ret < 0)
                 {
                 return -1;
@@ -614,14 +502,14 @@ while (li != answerList.end())
             }
         }
     k = 0;// new node
-    li++;
+    ++li;
     }
 
 if (answerList.empty()) {
     return 0;
 }
 
-buff[n%8] = 0;
+buff[n % 8] = 0;
 EncodeString(buffS, buff, &ctx);
 
 answerList.clear();
@@ -633,7 +521,7 @@ void CONFIGPROTO::SendError(const char * text)
 {
 char s[255];
 answerList.clear();
-sprintf(s, "<Error value=\"%s\"/>", text);
+snprintf(s, 255, "<Error value=\"%s\"/>", text);
 answerList.push_back(s);
 }
 //-----------------------------------------------------------------------------
@@ -642,6 +530,3 @@ void CONFIGPROTO::WriteLogAccessFailed(uint32_t ip)
 WriteServLog("Admin's connect failed. IP %s", inet_ntostring(ip).c_str());
 }
 //-----------------------------------------------------------------------------
-
-
-