summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--ChangeLog3
-rw-r--r--ommysql.c4
2 files changed, 7 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index 8c4d3ede..6e0c7a03 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -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
diff --git a/ommysql.c b/ommysql.c
index d786780b..ee8caab7 100644
--- a/ommysql.c
+++ b/ommysql.c
@@ -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;