tollgate-wrt daemonizes on unknown flags — probes like --version steal port 2121

Aperta Adatta ai principianti
  • Qualcosa non va?
#813 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
65/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
go
Ambito
cli

Direzione di ricerca

Il payload indica il binario /usr/bin/tollgate-wrt, il flag --version e la porta 2121, ma nessun file sorgente. Per prima cosa riproduci il problema sul ramo main attuale eseguendo il binario con un flag sconosciuto e verificando se avvia un daemon e si collega alla porta 2121. Poi trova il punto in cui gli argomenti da riga di comando vengono analizzati prima dell'avvio del daemon. Il lavoro è completo quando un flag sconosciuto stampa l'uso e termina con stato 2 senza avviare il daemon, e --version viene gestito esplicitamente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Migrated from Amperstrand/tollgate-module-basic-go#85 as part of the fork retirement — OpenTollGate/tollgate-module-basic-go is the single source of truth; the fork issue carries the original discussion and any lab evidence. Retirement plan: #758.]

Operational hazard hit twice during lab validation (tasks 9b/10, evidence .omo/evidence/task-9b-fixed-builds.md + task-10-battery.md): running /usr/bin/tollgate-wrt --version (or any unknown flag) does not error out — the binary STARTS AS A DAEMON, and if the service instance is between restarts it can bind :2121 first, leaving two daemons with procd's instance portless. Symptom: mysterious test-run pollution; recovery requires killing the stray by pid. Fix: reject unknown flags in flag parsing (exit 2 with usage) instead of falling through to daemon startup. Suggested-by: two independent workers burned ~30min each on this during security validation.

Migrated 2026-10-09. Applicability to current upstream main (4614ac2, post-rc1) was not re-verified during migration — confirming whether the finding still reproduces upstream is this issue's first task.

Lingua principale
Go
Stelle
12
Fork
14
Merge medio
3g 22h
PR unite (30g)
262

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di OpenTollGate/tollgate-module-basic-go

Tutte le issue di OpenTollGate/tollgate-module-basic-go

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.