Revert the dcb_close changes in handleError.

This commit is contained in:
Martin Brampton
2015-09-17 13:35:18 +01:00
parent 1ad8e27c91
commit dc3b0b067b

View File

@ -4879,8 +4879,6 @@ static void handleError (
CHK_BACKEND_REF(bref); CHK_BACKEND_REF(bref);
bref_clear_state(bref, BREF_IN_USE); bref_clear_state(bref, BREF_IN_USE);
bref_set_state(bref, BREF_CLOSED); bref_set_state(bref, BREF_CLOSED);
if (bref_was_in_use)
dcb_close(backend_dcb);
} }
else else
{ {
@ -4938,6 +4936,7 @@ static void handleError (
break; break;
} }
} }
dcb_close(backend_dcb);
} }
@ -4965,14 +4964,6 @@ static void handle_error_reply_client(
CHK_BACKEND_REF(bref); CHK_BACKEND_REF(bref);
bref_clear_state(bref, BREF_IN_USE); bref_clear_state(bref, BREF_IN_USE);
bref_set_state(bref, BREF_CLOSED); bref_set_state(bref, BREF_CLOSED);
if (bref_was_in_use)
{
dcb_close(backend_dcb);
}
}
else
{
dcb_close(backend_dcb);
} }
if (sesstate == SESSION_STATE_ROUTER_READY) if (sesstate == SESSION_STATE_ROUTER_READY)
@ -5023,7 +5014,6 @@ static bool handle_error_new_connection(
if ((bref = get_bref_from_dcb(myrses, backend_dcb)) == NULL) if ((bref = get_bref_from_dcb(myrses, backend_dcb)) == NULL)
{ {
succp = true; succp = true;
dcb_close(backend_dcb);
goto return_succp; goto return_succp;
} }
CHK_BACKEND_REF(bref); CHK_BACKEND_REF(bref);
@ -5063,10 +5053,6 @@ static bool handle_error_new_connection(
DCB_REASON_NOT_RESPONDING, DCB_REASON_NOT_RESPONDING,
&router_handle_state_switch, &router_handle_state_switch,
(void *)bref); (void *)bref);
if (bref_was_in_use)
{
dcb_close(backend_dcb);
}
router_nservers = router_get_servercount(inst); router_nservers = router_get_servercount(inst);
max_nslaves = rses_get_max_slavecount(myrses, router_nservers); max_nslaves = rses_get_max_slavecount(myrses, router_nservers);
max_slave_rlag = rses_get_max_replication_lag(myrses); max_slave_rlag = rses_get_max_replication_lag(myrses);