jamulussoftware / jamulussoftware/jamulus
Rewrite toInt() to fail on invalid input
- 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
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