From 98745101a103d1aef7a6c79cec8708729a2895fa Mon Sep 17 00:00:00 2001 From: Rainer Gerhards Date: Wed, 18 Sep 2013 10:10:56 +0200 Subject: 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. --- ChangeLog | 4 ++++ plugins/imfile/imfile.c | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 463ff253..ca544fff 100644 --- a/ChangeLog +++ b/ChangeLog @@ -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")) { -- cgit v1.2.3