valkey-io/valkey

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

Geschlossen

#844 geöffnet am 30.07.2024

Auf GitHub ansehenContributor Guide
 (18 Kommentare) (1 Reaktion) (1 zugewiesene Person)C (1.138 Forks)batch import
good first issuehelp wanted

Repository-Metriken

Stars
 (25.779 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 17T 7h) (74 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Überprüfen Sie die Implementierung von HRANDFIELD und ZRANDMEMBER im Quellcode, verfolgen Sie, wie das count Argument verarbeitet wird, und identifizieren Sie, wo der negative Wert eine unbegrenzte Speicherzuweisung verursacht. Fügen Sie eine Prüfung hinzu, um den absoluten Wert von count auf eine sichere Grenze basierend auf der Hash /Set Größe oder einem globalen Maximum zu begrenzen.
Tech Stack
c
Domain
backenddatabase
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65