hcq/MaxScale
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix binlogfilter matching

Browse Source 
The matching always checked the default database when it should only check
it if there are no tables in the statement.
This commit is contained in:
Markus Mäkelä
2019-04-01 11:25:09 +03:00
parent 06c01439fb
commit 738ae9178b
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
server/modules/filter/binlogfilter/binlogfiltersession.cc
View File

@ -405,10 +405,11 @@ static bool should_skip_query(const BinlogConfig& config, const std::string& sql
qc_free_table_names(names, n); qc_free_table_names(names, n);
} }
// Also check for the default database in case the query has no tables in it // Also check for the default database in case the query has no tables in it. The dot at the end is
if (!rval && should_skip(config, db)) // required to distinct database names from table names.
if (n == 0)
{ {
rval = true; rval = should_skip(config, db + '.');
} }
gwbuf_free(buf); gwbuf_free(buf);