Move other #9731 check to start of its functions

At arma's suggestion.  Looks like I missed this one.
This commit is contained in:
Nick Mathewson 2013-10-31 14:10:23 -04:00
parent 96f92f2062
commit 8f793c38fb

View File

@ -2711,6 +2711,9 @@ connection_consider_empty_write_buckets(connection_t *conn)
{ {
const char *reason; const char *reason;
if (!connection_is_rate_limited(conn))
return; /* Always okay. */
if (global_write_bucket <= 0) { if (global_write_bucket <= 0) {
reason = "global write bucket exhausted. Pausing."; reason = "global write bucket exhausted. Pausing.";
} else if (connection_counts_as_relayed_traffic(conn, approx_time()) && } else if (connection_counts_as_relayed_traffic(conn, approx_time()) &&
@ -2723,9 +2726,6 @@ connection_consider_empty_write_buckets(connection_t *conn)
} else } else
return; /* all good, no need to stop it */ return; /* all good, no need to stop it */
if (!connection_is_rate_limited(conn))
return; /* Always okay. */
LOG_FN_CONN(conn, (LOG_DEBUG, LD_NET, "%s", reason)); LOG_FN_CONN(conn, (LOG_DEBUG, LD_NET, "%s", reason));
conn->write_blocked_on_bw = 1; conn->write_blocked_on_bw = 1;
connection_stop_writing(conn); connection_stop_writing(conn);