uutils / uutils/diffutils

cmp::parse_params exits the process on --help instead of returning

Offen
#220 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Rust
Sterne
276
Forks
39
Ø Merge
3 Std. 27 Min.
Gemergte PRs (30 T.)
3

Beschreibung

When --help is passed to cmp::parse_params, it calls process::exit(0) which kills the whole process. Found this while working on the fuzz targets, the fuzzer kept crashing because of it.
Workaround is to filter --help before calling parse_params, but ideally the function should just return the help text instead of exiting.

Not sure if this is worth a bigger refactor or just leaving the workaround in place, open to suggestions.
Example CI failure: https://github.com/uutils/diffutils/actions/runs/24710862259

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne beim Einstiegspunkt cmp::parse_params und untersuche, wie der --help-Pfad behandelt wird. Überprüfe anschließend den fuzz-target-Aufruf, der den Prozessabbruch aufgedeckt hat. Reproduziere den CI-Fehler oder führe das relevante fuzz target aus. Als erledigt gilt die Aufgabe, wenn Aufrufer --help behandeln können, ohne dass der Prozess beendet wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
rust
Bereich
cli, testing
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
65/100

Neue Issues direkt in Ihr Postfach

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