Produce debug output only if SMUX_DEBUG is defined

[stg.git] / projects / stargazer / plugins / configuration / sgconfig / rsconf.cpp

diff --git a/projects/stargazer/plugins/configuration/sgconfig/rsconf.cpp b/projects/stargazer/plugins/configuration/sgconfig/rsconf.cpp
index 8d2ce15377f14c958b7dfa665a87ff82da62c56b..b4cbf544aa82f169049691fbfeef316251a31547 100644 (file)
--- a/projects/stargazer/plugins/configuration/sgconfig/rsconf.cpp
+++ b/projects/stargazer/plugins/configuration/sgconfig/rsconf.cpp
@@ -26,11 +26,14 @@
 *
 *******************************************************************/
 
 *
 *******************************************************************/
 

+#include <unistd.h> // cloase, usleep
+

 #include <cerrno>
 #include <csignal>
 #include <cerrno>
 #include <csignal>

+#include <cstdio> // snprintf

 
 

+#include "stg/blowfish.h"

 #include "configproto.h"
 #include "configproto.h"

-#include "blowfish.h"

 
 #ifndef ENODATA
 // FreeBSD 4.* - suxx
 
 #ifndef ENODATA
 // FreeBSD 4.* - suxx


@@ -126,113 +129,98 @@ close(sock);
 return 0;
 }
 //-----------------------------------------------------------------------------
 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;
+    state = confHdr;

     struct sockaddr_in outerAddr;
     socklen_t outerAddrLen(sizeof(outerAddr));
     struct sockaddr_in outerAddr;
     socklen_t outerAddrLen(sizeof(outerAddr));

-    int outerSocket = accept(cp->listenSocket,
+    int outerSocket = accept(listenSocket,

                              (struct sockaddr*)(&outerAddr),
                              &outerAddrLen);
 
                              (struct sockaddr*)(&outerAddr),
                              &outerAddrLen);
 

-    if (!cp->nonstop)
+    if (!nonstop)

         {
         {

-        continue;
+        break;

         }
 
     if (outerSocket == -1)
         {
         printfd(__FILE__, "accept failed\n");
         }
 
     if (outerSocket == -1)
         {
         printfd(__FILE__, "accept failed\n");

-        usleep(100000);

         continue;
         }
 
         continue;
         }
 

-    cp->adminIP = *(unsigned int*)&(outerAddr.sin_addr);
+    adminIP = *(unsigned int*)&(outerAddr.sin_addr);

 
     printfd(__FILE__, "Connection accepted from %s\n", inet_ntostring(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(outerSocket) < 0)
+        if (RecvHdr(outerSocket) < 0)

             {
             close(outerSocket);
             continue;
             }
             {
             close(outerSocket);
             continue;
             }

-        if (cp->state == confLogin)
+        if (state == confLogin)

             {
             {

-            if (cp->SendHdrAnswer(outerSocket, ans_ok) < 0)
+            if (SendHdrAnswer(outerSocket, ans_ok) < 0)

                 {
                 close(outerSocket);
                 continue;
                 }
                 {
                 close(outerSocket);
                 continue;
                 }

-
-            if (cp->RecvLogin(outerSocket) < 0)
+            if (RecvLogin(outerSocket) < 0)

                 {
                 close(outerSocket);
                 continue;
                 }
                 {
                 close(outerSocket);
                 continue;
                 }

-
-            if (cp->state == confLoginCipher)
+            if (state == confLoginCipher)

                 {
                 {

-                if (cp->SendLoginAnswer(outerSocket, ans_ok) < 0)
+                if (SendLoginAnswer(outerSocket) < 0)

                     {
                     close(outerSocket);
                     continue;
                     }
                     {
                     close(outerSocket);
                     continue;
                     }

-                if (cp->RecvLoginS(outerSocket) < 0)
+                if (RecvLoginS(outerSocket) < 0)

                     {
                     close(outerSocket);
                     continue;
                     }
                     {
                     close(outerSocket);
                     continue;
                     }

-
-                if (cp->state == confData)
+                if (state == confData)

                     {
                     {

-                    if (cp->SendLoginSAnswer(outerSocket, ans_ok) < 0)
+                    if (SendLoginSAnswer(outerSocket, ans_ok) < 0)

                         {
                         close(outerSocket);
                         continue;
                         }
                         {
                         close(outerSocket);
                         continue;
                         }

-                    if (cp->RecvData(outerSocket) < 0)
+                    if (RecvData(outerSocket) < 0)

                         {
                         close(outerSocket);
                         continue;
                         }
                         {
                         close(outerSocket);
                         continue;
                         }

-                    cp->state = confHdr;
+                    state = confHdr;

                     }
                 else
                     {
                     }
                 else
                     {

-                    if (cp->SendLoginSAnswer(outerSocket, ans_err) < 0)
+                    if (SendLoginSAnswer(outerSocket, ans_err) < 0)

                         {
                         close(outerSocket);
                         continue;
                         }
                         {
                         close(outerSocket);
                         continue;
                         }

-                    cp->WriteLogAccessFailed(cp->adminIP);
+                    WriteLogAccessFailed(adminIP);

                     }
                 }
             else
                 {
                     }
                 }
             else
                 {

-                cp->WriteLogAccessFailed(cp->adminIP);
+                WriteLogAccessFailed(adminIP);

                 }
             }
         else
             {
                 }
             }
         else
             {

-            cp->WriteLogAccessFailed(cp->adminIP);
-            if (cp->SendHdrAnswer(outerSocket, ans_err) < 0)
+            WriteLogAccessFailed(adminIP);
+            if (SendHdrAnswer(outerSocket, ans_err) < 0)

                 {
                 close(outerSocket);
                 continue;
                 {
                 close(outerSocket);
                 continue;


@@ -241,28 +229,33 @@ while (cp->nonstop)
         }
     else
         {
         }
     else
         {

-        cp->WriteLogAccessFailed(cp->adminIP);
+        WriteLogAccessFailed(adminIP);

         }
     close(outerSocket);
     }
         }
     close(outerSocket);
     }

-
-return NULL;

 }
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::RecvHdr(int sock)
 {
 char buf[sizeof(STG_HEADER)];
 memset(buf, 0, sizeof(STG_HEADER));
 }
 //-----------------------------------------------------------------------------
 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) - 1; // Without 0-char
+size_t pos = 0;
+while (pos < stgHdrLen)

     {
     {

-    ret = recv(sock, &buf[i], 1, 0);
-    if (ret <= 0)
+    if (!WaitPackets(sock))
+        {
+        state = confHdr;
+        SendError("Bad request");
+        return -1;
+        }
+    int ret = recv(sock, &buf[pos], stgHdrLen - pos, 0);
+    if (ret < 0)

         {
         state = confHdr;
         return -1;
         }
         {
         state = confHdr;
         return -1;
         }

+    pos += ret;

     }
 
 if (0 == strncmp(buf, STG_HEADER, strlen(STG_HEADER)))
     }
 
 if (0 == strncmp(buf, STG_HEADER, strlen(STG_HEADER)))


@@ -281,12 +274,9 @@ return -1;
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::SendHdrAnswer(int sock, int err)
 {
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::SendHdrAnswer(int sock, int err)
 {

-int ret;
-

 if (err)
     {
 if (err)
     {

-    ret = send(sock, ERR_HEADER, sizeof(ERR_HEADER)-1, 0);
-    if (ret < 0)
+    if (send(sock, ERR_HEADER, sizeof(ERR_HEADER) - 1, 0) < 0)

         {
         WriteServLog("send ERR_HEADER error in SendHdrAnswer.");
         return -1;
         {
         WriteServLog("send ERR_HEADER error in SendHdrAnswer.");
         return -1;


@@ -294,8 +284,7 @@ if (err)
     }
 else
     {
     }
 else
     {

-    ret = send(sock, OK_HEADER, sizeof(OK_HEADER)-1, 0);
-    if (ret < 0)
+    if (send(sock, OK_HEADER, sizeof(OK_HEADER) - 1, 0) < 0)

         {
         WriteServLog("send OK_HEADER error in SendHdrAnswer.");
         return -1;
         {
         WriteServLog("send OK_HEADER error in SendHdrAnswer.");
         return -1;


@@ -307,48 +296,46 @@ return 0;
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::RecvLogin(int sock)
 {
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::RecvLogin(int sock)
 {

-char login[ADM_LOGIN_LEN+1];
-int ret;
+char login[ADM_LOGIN_LEN + 1];

 
 memset(login, 0, ADM_LOGIN_LEN + 1);
 
 
 memset(login, 0, ADM_LOGIN_LEN + 1);
 

-ret = recv(sock, login, ADM_LOGIN_LEN, 0);
+size_t pos = 0;
+while (pos < ADM_LOGIN_LEN) {
+    if (!WaitPackets(sock))
+        {
+        state = confHdr;
+        return ENODATA;
+        }

 
 

-if (ret < 0)
-    {
-    // Error in network
-    close(sock);
-    state = confHdr;
-    return ENODATA;
-    }
+    int ret = recv(sock, &login[pos], ADM_LOGIN_LEN - pos, 0);

 
 

-if (ret < ADM_LOGIN_LEN)
-    {
-    // Error in protocol
-    close(sock);
-    state = confHdr;
-    return ENODATA;
-    }
+    if (ret < 0)
+        {
+        // Error in network
+        state = confHdr;
+        return ENODATA;
+        }
+
+    pos += ret;
+}

 
 

-if (admins->FindAdmin(login, &currAdmin))
+if (admins->Find(login, &currAdmin))

     {
     // Admin not found
     {
     // Admin not found

-    close(sock);

     state = confHdr;
     return ENODATA;
     }
     state = confHdr;
     return ENODATA;
     }

-currAdmin.SetAdminIP(adminIP);
+
+currAdmin->SetIP(adminIP);

 adminLogin = login;
 state = confLoginCipher;
 return 0;
 }
 //-----------------------------------------------------------------------------
 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);
-if (ret < 0)
+if (send(sock, OK_LOGIN, sizeof(OK_LOGIN) - 1, 0) < 0)

     {
     WriteServLog("Send OK_LOGIN error in SendLoginAnswer.");
     return -1;
     {
     WriteServLog("Send OK_LOGIN error in SendLoginAnswer.");
     return -1;


@@ -359,40 +346,43 @@ return 0;
 int CONFIGPROTO::RecvLoginS(int sock)
 {
 char loginS[ADM_LOGIN_LEN + 1];
 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);
 
 memset(loginS, 0, ADM_LOGIN_LEN + 1);
 

-int total = 0;
-
-while (total < ADM_LOGIN_LEN)
+size_t pos = 0;
+while (pos < ADM_LOGIN_LEN)

     {
     {

-    ret = recv(sock, &loginS[total], ADM_LOGIN_LEN - total, 0);
+    if (!WaitPackets(sock))
+        {
+        state = confHdr;
+        return ENODATA;
+        }
+
+    int ret = recv(sock, &loginS[pos], ADM_LOGIN_LEN - pos, 0);

 
     if (ret < 0)
         {
         // Network error
         printfd(__FILE__, "recv error: '%s'\n", strerror(errno));
 
     if (ret < 0)
         {
         // Network error
         printfd(__FILE__, "recv error: '%s'\n", strerror(errno));

-        close(sock);

         state = confHdr;
         return ENODATA;
         }
 
         state = confHdr;
         return ENODATA;
         }
 

-    total += ret;
+    pos += ret;

     }
 
     }
 

-if (currAdmin.GetLogin() == "")
+if (currAdmin->GetLogin().empty())

     {
     state = confHdr;
     return ENODATA;
     }
 
     {
     state = confHdr;
     return ENODATA;
     }
 

-EnDecodeInit(currAdmin.GetPassword().c_str(), ADM_PASSWD_LEN, &ctx);
+BLOWFISH_CTX ctx;
+EnDecodeInit(currAdmin->GetPassword().c_str(), ADM_PASSWD_LEN, &ctx);

 
 

-for (int i = 0; i < ADM_LOGIN_LEN/8; i++)
+char login[ADM_LOGIN_LEN + 1];
+for (size_t 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())
     }
 
 if (currAdmin == admins->GetNoAdmin())


@@ -402,7 +392,7 @@ if (currAdmin == admins->GetNoAdmin())
     return 0;
     }
 
     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;
     {
     state = confHdr;
     return ENODATA;


@@ -414,12 +404,9 @@ return 0;
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::SendLoginSAnswer(int sock, int err)
 {
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::SendLoginSAnswer(int sock, int err)
 {

-int ret;
-

 if (err)
     {
 if (err)
     {

-    ret = send(sock, ERR_LOGINS, sizeof(ERR_LOGINS)-1, 0);
-    if (ret < 0)
+    if (send(sock, ERR_LOGINS, sizeof(ERR_LOGINS) - 1, 0) < 0)

         {
         WriteServLog("send ERR_LOGIN error in SendLoginAnswer.");
         return -1;
         {
         WriteServLog("send ERR_LOGIN error in SendLoginAnswer.");
         return -1;


@@ -427,8 +414,7 @@ if (err)
     }
 else
     {
     }
 else
     {

-    ret = send(sock, OK_LOGINS, sizeof(OK_LOGINS)-1, 0);
-    if (ret < 0)
+    if (send(sock, OK_LOGINS, sizeof(OK_LOGINS) - 1, 0) < 0)

         {
         WriteServLog("send OK_LOGINS error in SendLoginSAnswer.");
         return -1;
         {
         WriteServLog("send OK_LOGINS error in SendLoginSAnswer.");
         return -1;


@@ -439,49 +425,50 @@ return 0;
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::RecvData(int sock)
 {
 //-----------------------------------------------------------------------------
 int CONFIGPROTO::RecvData(int sock)
 {

-int ret;
-char bufferS[8];
-char buffer[9];
-
-buffer[8] = 0;
-

 requestList.clear();
 BLOWFISH_CTX ctx;
 
 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)
     {
 
 while (1)
     {

-    int total = 0;

     bool done = false;
     bool done = false;

-    while (total < 8)
+    char bufferS[8];
+    size_t pos = 0;
+    while (pos < sizeof(bufferS))

         {
         {

-        ret = recv(sock, &bufferS[total], 8 - total, 0);
+        if (!WaitPackets(sock))
+            {
+            done = true;
+            break;
+            }
+
+        int ret = recv(sock, &bufferS[pos], sizeof(bufferS) - pos, 0);

         if (ret < 0)
             {
             // Network error
         if (ret < 0)
             {
             // Network error

-            close(sock);
-            return 0;
+            printfd(__FILE__, "recv error: '%s'\n", strerror(errno));
+            return -1;

             }
 
             }
 

-        if (ret < 8)
+        if (ret == 0)

             {
             {

-            if (memchr(buffer, 0, ret) != NULL)
-                {
-                done = true;
-                break;
-                }
+            done = true;
+            break;

             }
 
             }
 

-        total += ret;
+        pos += ret;

         }
 
         }
 

+    char buffer[8];
+    buffer[7] = 0;
+

     DecodeString(buffer, bufferS, &ctx);
     DecodeString(buffer, bufferS, &ctx);

-    requestList.push_back(std::string(buffer, total));
+    requestList.push_back(std::string(buffer, pos));

 
 

-    if (done || memchr(buffer, 0, total) != NULL)
+    if (done || memchr(buffer, 0, pos) != NULL)

         {
         {

-        // ëÏÎÅà ÐÏÓÙÌËÉ
+        // End of data

         if (ParseCommand())
             {
             SendError("Bad command");
         if (ParseCommand())
             {
             SendError("Bad command");


@@ -503,22 +490,21 @@ char buff[8];
 char buffS[8];
 int n = 0;
 int k = 0;
 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])
         {
 
 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++;
 
         n++;
         k++;
 

-        if (n%8 == 0)
+        if (n % 8 == 0)

             {
             EncodeString(buffS, buff, &ctx);
             {
             EncodeString(buffS, buff, &ctx);

-            ret = send(sock, buffS, 8, 0);
+            int ret = send(sock, buffS, 8, 0);

             if (ret < 0)
                 {
                 return -1;
             if (ret < 0)
                 {
                 return -1;


@@ -526,14 +512,14 @@ while (li != answerList.end())
             }
         }
     k = 0;// new node
             }
         }
     k = 0;// new node

-    li++;
+    ++li;

     }
 
 if (answerList.empty()) {
     return 0;
 }
 
     }
 
 if (answerList.empty()) {
     return 0;
 }
 

-buff[n%8] = 0;
+buff[n % 8] = 0;

 EncodeString(buffS, buff, &ctx);
 
 answerList.clear();
 EncodeString(buffS, buff, &ctx);
 
 answerList.clear();


@@ -545,7 +531,7 @@ void CONFIGPROTO::SendError(const char * text)
 {
 char s[255];
 answerList.clear();
 {
 char s[255];
 answerList.clear();

-sprintf(s, "<Error value=\"%s\"/>", text);
+snprintf(s, 255, "<Error value=\"%s\"/>", text);

 answerList.push_back(s);
 }
 //-----------------------------------------------------------------------------
 answerList.push_back(s);
 }
 //-----------------------------------------------------------------------------