minor fixes and refactoring in log parser
authorVictor Kirhenshtein <victor@netxms.org>
Thu, 5 Jan 2017 17:51:53 +0000 (19:51 +0200)
committerVictor Kirhenshtein <victor@netxms.org>
Thu, 5 Jan 2017 17:51:53 +0000 (19:51 +0200)
include/nxlpapi.h
src/agent/subagents/logwatch/logwatch.cpp
src/libnxlp/parser.cpp

index 4df55f0..11cf280 100644 (file)
@@ -240,6 +240,8 @@ private:
 
    const LogParserRule *findRuleByName(const TCHAR *name) const;
 
+   void setStatus(const TCHAR *status) { nx_strncpy(m_status, status, MAX_PARSER_STATUS_LEN); }
+
 public:
        LogParser();
        LogParser(LogParser *src);
@@ -257,7 +259,6 @@ public:
        void setName(const TCHAR *name);
        const TCHAR *getName() { return m_name; }
 
-       void setStatus(const TCHAR *status) { nx_strncpy(m_status, status, MAX_PARSER_STATUS_LEN); }
        const TCHAR *getStatus() { return m_status; }
 
        void setThread(THREAD th) { m_thread = th; }
index b0805c8..b831495 100644 (file)
@@ -1,6 +1,6 @@
 /*
 ** NetXMS LogWatch subagent
-** Copyright (C) 2008-2014 Victor Kirhenshtein
+** Copyright (C) 2008-2017 Victor Kirhenshtein
 **
 ** This program is free software; you can redistribute it and/or modify
 ** it under the terms of the GNU General Public License as published by
@@ -310,7 +310,7 @@ static BOOL SubagentInit(Config *config)
  */
 static NETXMS_SUBAGENT_PARAM s_parameters[] =
 {
-       { _T("LogWatch.Parser.Status(*)"), H_ParserStats, _T("S"), DCI_DT_INT, _T("Parser {instance} status") },
+       { _T("LogWatch.Parser.Status(*)"), H_ParserStats, _T("S"), DCI_DT_STRING, _T("Parser {instance} status") },
        { _T("LogWatch.Parser.MatchedRecords(*)"), H_ParserStats, _T("M"), DCI_DT_INT, _T("Number of records matched by parser {instance}") },
        { _T("LogWatch.Parser.ProcessedRecords(*)"), H_ParserStats, _T("P"), DCI_DT_INT, _T("Number of records processed by parser {instance}") }
 };
index 26aa23d..1d3933f 100644 (file)
@@ -168,11 +168,11 @@ LogParser::~LogParser()
 
        for(i = 0; i < m_numRules; i++)
                delete m_rules[i];
-       safe_free(m_rules);
-       safe_free(m_name);
-       safe_free(m_fileName);
+       free(m_rules);
+       free(m_name);
+       free(m_fileName);
 #ifdef _WIN32
-   safe_free(m_marker);
+   free(m_marker);
 #endif
 }