diff options
author | Rainer Gerhards <rgerhards@adiscon.com> | 2007-07-30 10:22:44 +0000 |
---|---|---|
committer | Rainer Gerhards <rgerhards@adiscon.com> | 2007-07-30 10:22:44 +0000 |
commit | 207572e21775bb0755a3b71dd45267614c9e04a1 (patch) | |
tree | ddcec04c48411ce83094e46bf37ca5f5120fba87 | |
parent | 6808a7e2f187760187edbd6aefe3513e2e5bb862 (diff) | |
download | rsyslog-207572e21775bb0755a3b71dd45267614c9e04a1.tar.gz rsyslog-207572e21775bb0755a3b71dd45267614c9e04a1.tar.xz rsyslog-207572e21775bb0755a3b71dd45267614c9e04a1.zip |
fixed a bug that caused ommysql to always complain about missing templates
-rw-r--r-- | ChangeLog | 3 | ||||
-rw-r--r-- | ommysql.c | 4 |
2 files changed, 7 insertions, 0 deletions
@@ -1,5 +1,8 @@ --------------------------------------------------------------------------- Version 1.17.6 (rgerhards), 2007-07-3? +- fixed bug in OMSRcreate() - always returned SR_RET_OK +- fixed a bug that caused ommysql to always complain about missing + templates --------------------------------------------------------------------------- Version 1.17.5 (rgerhards), 2007-07-30 - continued to work on modularization @@ -376,6 +376,10 @@ CODE_STD_STRING_REQUESTparseSelectorAct(1) * We specify that the SQL option must be present in the template. * This is for your own protection (prevent sql injection). */ + if(*p != ';') + --p; /* TODO: the whole parsing of the MySQL module needs to be re-thought - but this here + * is clean enough for the time being -- rgerhards, 2007-07-30 + */ if((iRet = cflineParseTemplateName(&p, *ppOMSR, 0, OMSR_RQD_TPL_OPT_SQL, (uchar*) " StdSQLFmt")) != RS_RET_OK) return iRet; |