summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRainer Gerhards <rgerhards@adiscon.com>2010-04-26 11:43:14 +0200
committerRainer Gerhards <rgerhards@adiscon.com>2010-04-26 11:43:14 +0200
commit96895d9e471716f4a449589fefa9f80a0564755a (patch)
treeeadbeb9356103888052b64571709db80ce447c1f
parent404230c4bdd42e1d9ca673ba3cc128c2b1e936e4 (diff)
downloadrsyslog-96895d9e471716f4a449589fefa9f80a0564755a.tar.gz
rsyslog-96895d9e471716f4a449589fefa9f80a0564755a.tar.xz
rsyslog-96895d9e471716f4a449589fefa9f80a0564755a.zip
bugfix(minor): status variable was uninitialized
However, this would have caused harm only if NO parser modules at all were loaded, which would lead to a defunctional configuration at all. And, even more important, this is impossible as two parser modules are built-in and thus can not be "not loaded", so we always have a minimum of two.
-rw-r--r--ChangeLog8
-rw-r--r--runtime/parser.c2
2 files changed, 9 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index e691716c..98ca6bbd 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+---------------------------------------------------------------------------
+Version 5.4.2 [v5-stable] (rgerhards), 2010-03-??
+- bugfix(minor): status variable was uninitialized
+ However, this would have caused harm only if NO parser modules at
+ all were loaded, which would lead to a defunctional configuration
+ at all. And, even more important, this is impossible as two parser
+ modules are built-in and thus can not be "not loaded", so we always
+ have a minimum of two.
- bugfix: testbench failed when not executed in UTC+1 timezone
accidently, the time zone information was kept inside some
to-be-checked-for responses
diff --git a/runtime/parser.c b/runtime/parser.c
index ca20a11e..cacbe065 100644
--- a/runtime/parser.c
+++ b/runtime/parser.c
@@ -448,7 +448,7 @@ ParsePRI(msg_t *pMsg)
static rsRetVal
ParseMsg(msg_t *pMsg)
{
- rsRetVal localRet;
+ rsRetVal localRet = RS_RET_ERR;
parserList_t *pParserList;
parser_t *pParser;
bool bIsSanitized;