diff options
| author | Michael Klishin <michael@rabbitmq.com> | 2014-08-01 11:41:49 +0400 |
|---|---|---|
| committer | Michael Klishin <michael@rabbitmq.com> | 2014-08-01 11:41:49 +0400 |
| commit | 2e4ab5a49c1d374b488e1841e6c7d752e8181c5d (patch) | |
| tree | 5fa186e1d43eb546a028e831ec989218d13424d3 | |
| parent | b88e3ceca1bce4f7592bbf6d1d92f018c6283165 (diff) | |
| parent | 26474538d10799ab6f8c25be1218477c4b15a114 (diff) | |
| download | rabbitmq-server-2e4ab5a49c1d374b488e1841e6c7d752e8181c5d.tar.gz | |
merge heads
| -rw-r--r-- | src/rabbit_reader.erl | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/rabbit_reader.erl b/src/rabbit_reader.erl index ddaf205e..fd5dc208 100644 --- a/src/rabbit_reader.erl +++ b/src/rabbit_reader.erl @@ -596,10 +596,13 @@ handle_exception(State = #v1{connection = #connection{protocol = Protocol}, State1 = close_connection(terminate_channels(State)), ok = send_on_channel0(State1#v1.sock, CloseMethod, Protocol), State1; -handle_exception(State, Channel, Reason) -> +handle_exception(State = #v1{connection_state = tuning, + connection = #connection{name = S}}, Channel, Reason) -> %% We don't trust the client at this point - force them to wait %% for a bit so they can't DOS us with repeated failed logins etc. timer:sleep(?SILENT_CLOSE_DELAY * 1000), + log(error, "AMQP connection ~p (~s) handshake error:~n~p~n", + [self(), S, Reason]), throw({handshake_error, State#v1.connection_state, Channel, Reason}). %% we've "lost sync" with the client and hence must not accept any @@ -971,7 +974,8 @@ validate_negotiated_integer_value(Field, Min, ClientValue) -> ServerValue = get_env(Field), if ClientValue /= 0 andalso ClientValue < Min -> fail_negotiation(Field, min, ServerValue, ClientValue); - ServerValue /= 0 andalso ClientValue > ServerValue -> + ServerValue /= 0 andalso (ClientValue =:= 0 orelse + ClientValue > ServerValue) -> fail_negotiation(Field, max, ServerValue, ClientValue); true -> ok |