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

The read_buffer pointer must be set to null in situations where the buffer has been freed (or consumed).

Browse Source
This commit is contained in:
counterpoint
2015-10-13 15:10:55 +02:00
parent bb53eb0f6d
commit cc42707dc0
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
server/modules/protocol/mysql_client.c
View File

@ -1141,6 +1141,7 @@ int gw_read_client_event(
/* Temporarily suppressed: SESSION_ROUTE_QUERY(session, read_buffer); */ /* Temporarily suppressed: SESSION_ROUTE_QUERY(session, read_buffer); */
/* Replaced with freeing the read buffer. */ /* Replaced with freeing the read buffer. */
gwbuf_free(read_buffer); gwbuf_free(read_buffer);
read_buffer = NULL;
/** /**
* Close router session which causes closing of backends. * Close router session which causes closing of backends.
*/ */
@ -1163,12 +1164,14 @@ int gw_read_client_event(
{ {
/** add incomplete mysql packet to read queue */ /** add incomplete mysql packet to read queue */
dcb->dcb_readqueue = gwbuf_append(dcb->dcb_readqueue, read_buffer); dcb->dcb_readqueue = gwbuf_append(dcb->dcb_readqueue, read_buffer);
read_buffer = NULL;
} }
} }
else else
{ {
/** Feed whole packet to router */ /** Feed whole packet to router */
rc = SESSION_ROUTE_QUERY(session, read_buffer); rc = SESSION_ROUTE_QUERY(session, read_buffer);
read_buffer = NULL;
} }
/** Routing succeed */ /** Routing succeed */