envoyproxy/envoy

[redis_proxy] potential out of memory/ with untrusted buffer when onData()

Open

#12.689 aperta il 17 ago 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/redishelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

Since documentation of redis_proxy shows it is only accepts trusted data now, I think this is not a security issue, so I post it here.

In the decoder of redis_proxy filter, it reads an integer from the buffer and constructs a vector with size equal to it(pending_integer_.integer_). https://github.com/envoyproxy/envoy/blob/master/source/extensions/filters/network/common/redis/codec_impl.cc#L442

Untrusted buffer containing a very large integer here(as the size of the vector) may lead to out of memory or uncaught length_error exception.

We may consider adding a constraint on the vector length(for example, not more than buffer.size()?) to avoid this bug.

The issue and test case can be found here: https://oss-fuzz.com/testcase-detail/5663259240431616

Guida contributor

Direzione di ricerca
Esamina il decoder in source/extensions/filters/network/common/redis/codec impl.cc intorno alla riga 442. Aggiungi un controllo dei limiti sulla dimensione intera prima di costruire il vettore per prevenire OOM. Assicurati che la dimensione del buffer non venga superata.
Tech stack
ccpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Redis protocol
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75