valkey-io/valkey

[BUG] A large negative number(e.g. -LONG_MAX) in HRANDFIELD/ZRANDMEMBER can easily cause Valkey to crash.

Aperta

#844 aperta il 30 lug 2024

Vedi su GitHubGuida contributor
 (18 commenti) (1 reazione) (1 assegnatario)C (1138 fork)batch import
good first issuehelp wanted

Metriche repository

Star
 (25.779 stelle)
Metriche merge PR
 (Merge medio 17g 7h) (74 PR mergiate in 30 g)

Descrizione

Describe the bug

When using the HRANDFIELD or ZRANDMEMBER commands, if an extremely negative value(e.g. -LONG_MAX) is provided, the memory will continue to rise and eventually cause Redis to crash.

To reproduce

HRANDFIELD hash -9223372036854775807

Expected behavior

IMHO, it is more reasonable to set a limit on count according to the memory limit or the actual size of the hash table to prevent Redis from crashing. This can prevent crashes caused by careless and incorrect use. If a user really needs to sample a lot of elements that allow duplicates, he can achieve the same purpose by calling the command multiple times.

Guida contributor

Direzione di ricerca
Ispeziona l'implementazione di HRANDFIELD e ZRANDMEMBER nel codice sorgente, traccia come viene elaborato l'argomento count e identifica dove il valore negativo causa un'allocazione di memoria illimitata. Aggiungi un controllo per limitare il valore assoluto di count a un limite sicuro basato sulla dimensione dell'hash/set o su un massimo globale.
Tech stack
c
Dominio
backenddatabase
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
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65