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

Fix route_by_statement return value

Browse Source 
The return value of route_by_statement was not initialized and not set if
a COM_CHANGE_USER was processed.
This commit is contained in:
Markus Mäkelä
2018-06-12 11:46:31 +03:00
parent 24870e278c
commit e99d9826ad
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
server/modules/protocol/MySQL/mariadbclient/mysql_client.cc
View File

@ -1575,7 +1575,7 @@ static bool reauthenticate_client(MXS_SESSION* session, GWBUF* packetbuf)
*/ */
static int route_by_statement(MXS_SESSION* session, uint64_t capabilities, GWBUF** p_readbuf) static int route_by_statement(MXS_SESSION* session, uint64_t capabilities, GWBUF** p_readbuf)
{ {
int rc; int rc = 1;
GWBUF* packetbuf; GWBUF* packetbuf;
do do
{ {
@ -1682,6 +1682,7 @@ static int route_by_statement(MXS_SESSION* session, uint64_t capabilities, GWBUF
// Store the original COM_CHANGE_USER for later // Store the original COM_CHANGE_USER for later
proto->stored_query = packetbuf; proto->stored_query = packetbuf;
packetbuf = NULL; packetbuf = NULL;
rc = 1;
} }
else if (proto->changing_user) else if (proto->changing_user)
{ {