Various fixes of issues reported by static analyzers.

[stg.git] / projects / sgconf / common_sg.cpp

diff --git a/projects/sgconf/common_sg.cpp b/projects/sgconf/common_sg.cpp
index bf2ac6225afe53bc149f8f5f7771f4966cd2843f..86214a3dd916c81b6333733f90f97f7dc1546215 100644 (file)
--- a/projects/sgconf/common_sg.cpp
+++ b/projects/sgconf/common_sg.cpp
@@ -213,13 +213,6 @@ return usr;
 void ConvertKOI8(const string & src, string * dst, int encType)
 {
 iconv_t cd;
-char * ob = new char[src.size() * 2 + 1];
-char * ib = new char[src.size() + 1];
-
-strcpy(ib, src.c_str());
-
-char * outbuf = ob;
-char * inbuf = ib;
 
 setlocale(LC_ALL, "");
 
@@ -239,11 +232,6 @@ else
 
 size_t nconv = 1;
 
-size_t insize = strlen(ib);
-size_t outsize = insize * 2 + 1;
-
-insize = src.size();
-
 cd = iconv_open(charsetT, charsetF);
 if (cd == (iconv_t) -1)
     {
@@ -259,7 +247,20 @@ if (cd == (iconv_t) -1)
     exit(ICONV_ERR_CODE);
     }
 
-#if defined(FREE_BSD) || defined(FREE_BSD5)
+char * ob = new char[src.size() * 2 + 1];
+char * ib = new char[src.size() + 1];
+
+strcpy(ib, src.c_str());
+
+char * outbuf = ob;
+char * inbuf = ib;
+
+size_t insize = strlen(ib);
+size_t outsize = insize * 2 + 1;
+
+insize = src.size();
+
+#if defined(CONST_ICONV)
 nconv = iconv(cd, (const char **)&inbuf, &insize, &outbuf, &outsize);
 #else
 nconv = iconv(cd, &inbuf, &insize, &outbuf, &outsize);
@@ -335,13 +336,13 @@ if (ud->login == "")
     return;
     }
 
-if (!req->cash.res_empty())
+if (!req->cash.empty())
     cout << "cash=" << ud->cash << endl;
 
-if (!req->credit.res_empty())
+if (!req->credit.empty())
     cout << "credit=" << ud->credit << endl;
 
-if (!req->creditExpire.res_empty())
+if (!req->creditExpire.empty())
     {
     char buf[32];
     struct tm brokenTime;
@@ -361,32 +362,32 @@ if (!req->creditExpire.res_empty())
     cout << "creditExpire=" << buf << endl;
     }
 
-if (!req->down.res_empty())
+if (!req->down.empty())
     cout << "down=" << ud->down << endl;
 
-if (!req->passive.res_empty())
+if (!req->passive.empty())
     cout << "passive=" << ud->passive << endl;
 
-if (!req->disableDetailStat.res_empty())
+if (!req->disableDetailStat.empty())
     cout << "disableDetailStat=" << ud->disableDetailStat << endl;
 
-if (!req->alwaysOnline.res_empty())
+if (!req->alwaysOnline.empty())
     cout << "alwaysOnline=" << ud->alwaysOnline << endl;
 
-if (!req->prepaidTraff.res_empty())
+if (!req->prepaidTraff.empty())
     cout << "prepaidTraff=" << ud->prepaidTraff << endl;
 
 for (int i = 0; i < DIR_NUM; i++)
     {
-    if (!req->u[i].res_empty())
+    if (!req->u[i].empty())
         cout << "u" << i << "=" << ud->stat.mu[i] << endl;
-    if (!req->d[i].res_empty())
+    if (!req->d[i].empty())
         cout << "d" << i << "=" << ud->stat.md[i] << endl;
     }
 
 for (int i = 0; i < USERDATA_NUM; i++)
     {
-    if (!req->ud[i].res_empty())
+    if (!req->ud[i].empty())
         {
         string str;
         ConvertFromKOI8(ud->userData[i], &str);
@@ -408,7 +409,7 @@ StringReqParams strReqParams[] =
 };
 for (unsigned i = 0; i < sizeof(strReqParams) / sizeof(StringReqParams); i++)
     {
-    if (!strReqParams[i].reqParam.res_empty())
+    if (!strReqParams[i].reqParam.empty())
         {
         string str;
         ConvertFromKOI8(*strReqParams[i].value, &str);
@@ -425,8 +426,6 @@ abcbd = (AuthByCbData *)d;
 
 bool * result = abcbd->result;
 
-REQUEST * req = (REQUEST *)abcbd->data;
-
 for (std::vector<std::string>::const_iterator it = list.begin(); it != list.end(); ++it)
     cout << *it << "\n";
 cout << endl;