Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Rust fd://3 socket activation adopts the fd blindly — no validation it is a listening socket

Aperta
#29 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
64/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
go, rust
Ambito
networking, security

Direzione di ricerca

Inizia dal percorso di dispatch di Rust per fd://3 e dal relativo helper di fd-wrapping, quindi confrontalo con la gestione esistente in Go in go/main.go:78. Esegui o aggiungi il test di subprocess proposto, che dup2s un listener reale su fd 3 ed esercita il percorso end-to-end. Il lavoro è completato quando un fd non valido fallisce rapidamente all’avvio con un errore chiaro, mentre un socket AF_UNIX valido in ascolto continua a funzionare.

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

Descrizione

Priority: P3 Type: Enhancement

Summary

With --listen-socket=fd://3, the Rust implementation adopts fd 3 via from_raw_fd without verifying it is actually a listening AF_UNIX socket. Go's net.FileListener (go/main.go:78) rejects non-socket fds; Rust only surfaces the problem later as io::Error from set_nonblocking/accept (mitigated by the accept-error backoff added in #27 — no busy loop, but also no clear early failure).

Neither implementation checks the LISTEN_PID/LISTEN_FDS environment variables from the sd_listen_fds convention, so this is parity-plus hardening.

Affected implementation(s)

  • Rust
  • Go (partially validates via net.FileListener; also skips LISTEN_FDS env check)

Expected behavior

--listen-socket=fd://3 with an fd that is not a listening AF_UNIX socket should fail fast at startup with a clear error, not degrade into accept errors at runtime.

Suggested fix

Before adopting the fd:

  • fstat and check S_IFSOCK
  • getsockopt(SOL_SOCKET, SO_ACCEPTCONN) to confirm it is listening
  • Optionally validate LISTEN_PID == getpid() and LISTEN_FDS >= 1 per the systemd convention (would also make sense in Go for parity)

Also worth a subprocess-based test that dup2s a real listener onto fd 3 and exercises the fd://3 dispatch path end-to-end (currently only the fd-wrapping helper is unit-tested).

Context

Split out from the review follow-ups on PR #27. Related: #25, #28.

Lingua principale
Rust
Stelle
2
Fork
0
Merge medio
9h 35m
PR unite (30g)
6

Guida per i contributori

Apri la guida per i contributori

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 ChainSafe/docker-socket-policy

Tutte le issue di ChainSafe/docker-socket-policy

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.