mkctr panics when a path passed to --files does not exist

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

Nessuno ha ancora preso questa issue.

  • #36 di @RaphaelFakhri — chiusa senza merge

Valutazione

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

Direzione di ricerca

Inizia dal punto di ingresso layerFromFiles ed esamina la relativa callback filepath.WalkDir, incluso il modo in cui vengono gestiti gli errori di writeDir. Riproduci il problema con l'esempio fornito mkctr --files=./missing:/app/missing. Il lavoro è completato quando il comando segnala un errore che indica il percorso mancante, termina con uno stato diverso da zero e non va in panic.

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

Descrizione

Running mkctr with a --files source that does not exist crashes with a nil pointer dereference instead of reporting an error.

For example, mkctr --files=./missing:/app/missing ... reaches layerFromFiles, which calls filepath.WalkDir on the source. When the source cannot be read, WalkDir calls the callback once with a nil fs.DirEntry and a non-nil error. The callback ignores the error and calls d.IsDir(), which panics:

panic: runtime error: invalid memory address or nil pointer dereference

The expected behavior is an error that names the missing path, and a non-zero exit status.

The callback also drops the error returned by writeDir for the parent directory, so a failing tar write is not reported at that point.

Lingua principale
Go
Stelle
101
Fork
10
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

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 tailscale/mkctr

Tutte le issue di tailscale/mkctr

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.