fixed clang detected issues
authorVictor Kirhenshtein <victor@netxms.org>
Sun, 23 Apr 2017 07:53:46 +0000 (10:53 +0300)
committerVictor Kirhenshtein <victor@netxms.org>
Sun, 23 Apr 2017 07:53:56 +0000 (10:53 +0300)
src/agent/subagents/linux/cpu.cpp
src/libnxlp/file.cpp
src/server/core/console.cpp
src/server/core/session.cpp
src/server/libnxsrv/snmpproxy.cpp

index 55809ca..9b41f5e 100644 (file)
@@ -103,11 +103,11 @@ static void CpuUsageCollector()
                }
                else if (buffer[0] == 'i' && buffer[1] == 'n' && buffer[2] == 't' && buffer[3] == 'r')
                {
-         sscanf(buffer, "intr %llu", &m_cpuInterrupts);
+         ret = sscanf(buffer, "intr %llu", &m_cpuInterrupts);
                }
                else if (buffer[0] == 'c' && buffer[1] == 't' && buffer[2] == 'x' && buffer[3] == 't')
       {
-         sscanf(buffer, "ctxt %llu", &m_cpuContextSwitches);
+         ret = sscanf(buffer, "ctxt %llu", &m_cpuContextSwitches);
       }
                else
                   continue;
index 7d3e989..013bc41 100644 (file)
@@ -566,7 +566,7 @@ bool LogParser::monitorFile(CONDITION stopCondition, bool readFromCurrPos)
                         int readSize = min(pos, 4);
                         lseek(fh, -readSize, SEEK_CUR);
                         int bytes = _read(fh, buffer, readSize);
-                        if (!memcmp(buffer, "\x00\x00\x00\x00", readSize))
+                        if ((bytes == readSize) && !memcmp(buffer, "\x00\x00\x00\x00", readSize))
                         {
                            LogParserTrace(6, _T("LogParser: detected reset of preallocated file \"%s\""), fname);
                            lseek(fh, 0, SEEK_SET);
index bae50e3..737560e 100644 (file)
@@ -1019,7 +1019,7 @@ int ProcessConsoleCommand(const TCHAR *pszCmdLine, CONSOLE_CTX pCtx)
       pArg = ExtractWord(pArg, szBuffer);
       dwNode1 = _tcstoul(szBuffer, NULL, 0);
 
-      pArg = ExtractWord(pArg, szBuffer);
+      ExtractWord(pArg, szBuffer);
       dwNode2 = _tcstoul(szBuffer, NULL, 0);
 
       if ((dwNode1 != 0) && (dwNode2 != 0))
index 533a54f..4789f3b 100644 (file)
@@ -2612,22 +2612,32 @@ void ClientSession::getConfigurationVariables(UINT32 dwRqId)
             msg.setField(dwId++, DBGetField(hResult, i, 4, szBuffer, MAX_CONFIG_VALUE));
          }
          DBFreeResult(hResult);
-      }
-      hResult = DBSelect(hdb, _T("SELECT var_name,var_value,var_description FROM config_values"));
-      if (hResult != NULL)
-      {
-         dwNumRecords = DBGetNumRows(hResult);
-         msg.setField(VID_NUM_VALUES, dwNumRecords);
-         for(i = 0; i < dwNumRecords; i++)
+
+         hResult = DBSelect(hdb, _T("SELECT var_name,var_value,var_description FROM config_values"));
+         if (hResult != NULL)
          {
-            msg.setField(dwId++, DBGetField(hResult, i, 0, szBuffer, MAX_DB_STRING));
-            msg.setField(dwId++, DBGetField(hResult, i, 1, szBuffer, MAX_CONFIG_VALUE));
-            msg.setField(dwId++, DBGetField(hResult, i, 2, szBuffer, MAX_DB_STRING));
+            dwNumRecords = DBGetNumRows(hResult);
+            msg.setField(VID_NUM_VALUES, dwNumRecords);
+            for(i = 0; i < dwNumRecords; i++)
+            {
+               msg.setField(dwId++, DBGetField(hResult, i, 0, szBuffer, MAX_DB_STRING));
+               msg.setField(dwId++, DBGetField(hResult, i, 1, szBuffer, MAX_CONFIG_VALUE));
+               msg.setField(dwId++, DBGetField(hResult, i, 2, szBuffer, MAX_DB_STRING));
+            }
+            DBFreeResult(hResult);
+
+            msg.setField(VID_RCC, RCC_SUCCESS);
+         }
+         else
+         {
+            msg.setField(VID_RCC, RCC_DB_FAILURE);
          }
-         DBFreeResult(hResult);
+      }
+      else
+      {
+         msg.setField(VID_RCC, RCC_DB_FAILURE);
       }
       DBConnectionPoolReleaseConnection(hdb);
-      msg.setField(VID_RCC, RCC_SUCCESS);
    }
    else
    {
index 75d108c..2d4acc5 100644 (file)
@@ -50,10 +50,10 @@ SNMP_ProxyTransport::~SNMP_ProxyTransport()
  */
 int SNMP_ProxyTransport::sendMessage(SNMP_PDU *pdu)
 {
-   BYTE *pBuffer;
    int nRet = -1;
        NXCPMessage msg(m_agentConnection->getProtocolVersion());
 
+   BYTE *pBuffer;
    size_t size = pdu->encode(&pBuffer, m_securityContext);
    if (size != 0)
    {
@@ -61,7 +61,7 @@ int SNMP_ProxyTransport::sendMessage(SNMP_PDU *pdu)
                msg.setField(VID_IP_ADDRESS, m_ipAddr);
                msg.setField(VID_PORT, m_port);
                msg.setField(VID_PDU_SIZE, (UINT32)size);
-               msg.setField(VID_PDU, pBuffer, (UINT32)size);
+               msg.setField(VID_PDU, pBuffer, size);
       free(pBuffer);
 
       if (m_waitForResponse)