diff options
| author | Alan Antonuk <alan.antonuk@gmail.com> | 2021-04-29 04:22:23 +0000 |
|---|---|---|
| committer | Alan Antonuk <alan.antonuk@gmail.com> | 2021-04-28 22:45:23 -0700 |
| commit | e07273089bb199437aaeb7b4afb3944e70166365 (patch) | |
| tree | 43480c120f13b04ae475b8e460ab7b366c785fd1 /librabbitmq | |
| parent | 81926ffdb8eadeba6dfebf994a23b81db3a32b00 (diff) | |
| download | rabbitmq-c-e07273089bb199437aaeb7b4afb3944e70166365.tar.gz | |
lib: fix clang-tidy finding in simple_rpc_inner
While this wasn't causing any errors, it did have the potential to cause
a bug if the error codes ever were positive.
Signed-off-by: GitHub <noreply@github.com>
Diffstat (limited to 'librabbitmq')
| -rw-r--r-- | librabbitmq/amqp_socket.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/librabbitmq/amqp_socket.c b/librabbitmq/amqp_socket.c index fdff1d2..cde1b8f 100644 --- a/librabbitmq/amqp_socket.c +++ b/librabbitmq/amqp_socket.c @@ -1056,7 +1056,7 @@ static amqp_rpc_reply_t simple_rpc_inner( retry: status = wait_frame_inner(state, &frame, deadline); - if (status < 0) { + if (status != AMQP_STATUS_OK) { if (status == AMQP_STATUS_TIMEOUT) { amqp_socket_close(state->socket, AMQP_SC_FORCE); } |