diff options
| author | Hans Nilsson <[email protected]> | 2015-04-29 14:58:06 +0200 |
|---|---|---|
| committer | Hans Nilsson <[email protected]> | 2015-04-30 16:29:51 +0200 |
| commit | ec5dcc8a6b9f86aa8dd02c70b630460cdc0833df (patch) | |
| tree | 1abdbf0fa3ca68bd376673a589be7fce2d566e92 /lib/ssh/src/ssh_connection_handler.erl | |
| parent | de6ef63b143557558436b5c0213030a27d78da0d (diff) | |
| download | otp-ec5dcc8a6b9f86aa8dd02c70b630460cdc0833df.tar.gz otp-ec5dcc8a6b9f86aa8dd02c70b630460cdc0833df.tar.bz2 otp-ec5dcc8a6b9f86aa8dd02c70b630460cdc0833df.zip | |
ssh: Fix bug causing rekeying once per minute after 1st Gbyte Thanks Simon
The port stats are not accumulated so that once rekey_limit bytes
(by default, 1GB) have been transmitted the connection will be
rekeyed every minute, not after the next 1GB.
Diffstat (limited to 'lib/ssh/src/ssh_connection_handler.erl')
| -rw-r--r-- | lib/ssh/src/ssh_connection_handler.erl | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/lib/ssh/src/ssh_connection_handler.erl b/lib/ssh/src/ssh_connection_handler.erl index e1f2e059e8..4dea284071 100644 --- a/lib/ssh/src/ssh_connection_handler.erl +++ b/lib/ssh/src/ssh_connection_handler.erl @@ -70,6 +70,7 @@ undecoded_packet_length, % integer() key_exchange_init_msg, % #ssh_msg_kexinit{} renegotiate = false, % boolean() + last_size_rekey = 0, connection_queue, address, port, @@ -635,7 +636,8 @@ handle_event(renegotiate, StateName, State) -> %% Rekey due to sent data limit reached? handle_event(data_size, connected, #state{ssh_params = Ssh0} = State) -> - {ok, [{send_oct,Sent}]} = inet:getstat(State#state.socket, [send_oct]), + {ok, [{send_oct,Sent0}]} = inet:getstat(State#state.socket, [send_oct]), + Sent = Sent0 - State#state.last_size_rekey, MaxSent = proplists:get_value(rekey_limit, State#state.opts, 1024000000), timer:apply_after(?REKEY_DATA_TIMOUT, gen_fsm, send_all_state_event, [self(), data_size]), case Sent >= MaxSent of @@ -645,7 +647,8 @@ handle_event(data_size, connected, #state{ssh_params = Ssh0} = State) -> {next_state, kexinit, next_packet(State#state{ssh_params = Ssh, key_exchange_init_msg = KeyInitMsg, - renegotiate = true})}; + renegotiate = true, + last_size_rekey = Sent0})}; _ -> {next_state, connected, next_packet(State)} end; |