diff options
author | Rainer Gerhards <rgerhards@adiscon.com> | 2010-04-26 11:43:14 +0200 |
---|---|---|
committer | Rainer Gerhards <rgerhards@adiscon.com> | 2010-04-26 11:43:14 +0200 |
commit | 96895d9e471716f4a449589fefa9f80a0564755a (patch) | |
tree | eadbeb9356103888052b64571709db80ce447c1f | |
parent | 404230c4bdd42e1d9ca673ba3cc128c2b1e936e4 (diff) | |
download | rsyslog-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-- | ChangeLog | 8 | ||||
-rw-r--r-- | runtime/parser.c | 2 |
2 files changed, 9 insertions, 1 deletions
@@ -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; |