Fix route_by_statement return value

The return value of route_by_statement was not initialized and not set if a COM_CHANGE_USER was processed.
mariadb-corporation · Jun 12, 2018 · e99d982 · e99d982
1 parent 24870e2
commit e99d982
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/server/modules/protocol/MySQL/mariadbclient/mysql_client.cc b/server/modules/protocol/MySQL/mariadbclient/mysql_client.cc
@@ -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)
 {
-    int rc;
+    int rc = 1;
     GWBUF* packetbuf;
     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
                 proto->stored_query = packetbuf;
                 packetbuf = NULL;
+                rc = 1;
             }
             else if (proto->changing_user)
             {