jamulussoftware / jamulussoftware/jamulus

Rewrite toInt() to fail on invalid input

Offen
#3,897 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
C
Sterne
1.1k
Forks
248
Ø Merge
2 T. 3 Std.
Gemergte PRs (30 T.)
9

Beschreibung

`toInt()` has a variant that sets a success indicator. That's should always be used when validating external input (even stored in a settings ini file, really).

_Originally posted by @pljones in https://github.com/jamulussoftware/jamulus/issues/3772#issuecomment-5257629528_

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Durchsuche die C-Codebasis nach toInt() und der Variante, die einen Erfolgsindikator setzt. Verfolge Aufrufer, die externe Eingaben validieren, einschließlich aus einer ini-Datei geladener Einstellungen, und prüfe, wie ungültige Werte behandelt werden. Abgeschlossen ist die Aufgabe, wenn die relevanten Validierungspfade ungültige Eingaben konsistent erkennen, statt sie stillschweigend zu akzeptieren.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
c
Bereich
security
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Aktiv
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.