Impact
The issue only occurs when the CLIENT SETINFO
command times out during connection establishment. The following circumstances can cause such a timeout:
- The client is configured to transmit its identity. This can be disabled via the
DisableIndentity
flag.
- There are network connectivity issues
- The client was configured with aggressive timeouts
The impact differs by use case:
- Sticky connections: Rather than using a connection from the pool on-demand, the caller can stick with a connection. Then you receive persistent out-of-order responses for the lifetime of the connection.
- Pipelines: All commands in the pipeline receive incorrect responses.
- Default connection pool usage without pipelining: When used with the default ConnPool once a connection is returned after use with ConnPool#Put the read buffer will be checked and the connection will be marked as bad due to the unread data. This means that at most one out-of-order response before the connection is discarded.
Patches
We prepared a fix in redis/go-redis#3295 and plan to release patch versions soon.
Workarounds
You can prevent the vulnerability by setting the flag DisableIndentity
(BTW: We also need to fix the spelling.) to true
when constructing the client instance.
Credit
Akhass Wasti
Ramin Ghorashi
Anton Amlinger
Syed Rahman
Mahesh Venkateswaran
Sergey Zavoloka
Aditya Adarwal
Abdulla Anam
Abd-Alhameed
Alex Vanlint
Gaurav Choudhary
Vedanta Jha
Yll Kelani
Ryan Picard
References
Impact
The issue only occurs when the
CLIENT SETINFO
command times out during connection establishment. The following circumstances can cause such a timeout:DisableIndentity
flag.The impact differs by use case:
Patches
We prepared a fix in redis/go-redis#3295 and plan to release patch versions soon.
Workarounds
You can prevent the vulnerability by setting the flag
DisableIndentity
(BTW: We also need to fix the spelling.) totrue
when constructing the client instance.Credit
Akhass Wasti
Ramin Ghorashi
Anton Amlinger
Syed Rahman
Mahesh Venkateswaran
Sergey Zavoloka
Aditya Adarwal
Abdulla Anam
Abd-Alhameed
Alex Vanlint
Gaurav Choudhary
Vedanta Jha
Yll Kelani
Ryan Picard
References