diff options
author | Rainer Gerhards <rgerhards@adiscon.com> | 2013-09-18 10:10:56 +0200 |
---|---|---|
committer | Rainer Gerhards <rgerhards@adiscon.com> | 2013-09-18 10:10:56 +0200 |
commit | 98745101a103d1aef7a6c79cec8708729a2895fa (patch) | |
tree | 4b7a8c826b6b328e7fb0aaf7996552b7b0bbd9d9 | |
parent | 5bb38abf3dcf5ad30e386e234d23ff0ad7b02f36 (diff) | |
download | rsyslog-98745101a103d1aef7a6c79cec8708729a2895fa.tar.gz rsyslog-98745101a103d1aef7a6c79cec8708729a2895fa.tar.bz2 rsyslog-98745101a103d1aef7a6c79cec8708729a2895fa.zip |
bugfix: imfile "facility" input parameter improperly handled
caused facility not to be set, and severity to be overwritten with
the facility value.
Thanks to forum user dmunny for reporting this bug.
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | plugins/imfile/imfile.c | 2 |
2 files changed, 5 insertions, 1 deletions
@@ -3,6 +3,10 @@ Version 7.4.5 [v7.4-stable] 2013-09-?? - bugfix: doc: imuxsock legacy param $SystemLogSocketParseTrusted was misspelled Thanks to David Lang for alerting us +- bugfix: imfile "facility" input parameter improperly handled + caused facility not to be set, and severity to be overwritten with + the facility value. + Thanks to forum user dmunny for reporting this bug. - bugfix: small memory leak in imfile when $ResetConfigVariables was used Thanks to Grégory Nuyttens for reporting this bug and providig a fix - bugfix: segfault on startup if TLS was used but no CA cert set diff --git a/plugins/imfile/imfile.c b/plugins/imfile/imfile.c index 66ed8bc4..9c824c18 100644 --- a/plugins/imfile/imfile.c +++ b/plugins/imfile/imfile.c @@ -473,7 +473,7 @@ CODESTARTnewInpInst } else if(!strcmp(inppblk.descr[i].name, "severity")) { inst->iSeverity = pvals[i].val.d.n; } else if(!strcmp(inppblk.descr[i].name, "facility")) { - inst->iSeverity = pvals[i].val.d.n; + inst->iFacility = pvals[i].val.d.n; } else if(!strcmp(inppblk.descr[i].name, "readmode")) { inst->readMode = pvals[i].val.d.n; } else if(!strcmp(inppblk.descr[i].name, "maxlinesatonce")) { |