python / python/cpython

tarfile: Degenerate TARs with global PAX headers are not handled correctly

Aperta
#149,578 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

stdlib type-bug
Lingua principale
Python
Stelle
77.2k
Fork
35.9k
Metriche di merge delle PR
Metriche PR in attesa

Descrizione

Bug report

Bug description:

It appears that tarfile.TarFile fails opening PAX TAR files if the only thing they contain is the global headers, no matter if created with the module or with the tar tool. The exception raised is tarfile.ReadError: end of file header. The tar executable handles those files perfectly.

Reproducer

import subprocess
import traceback
import tarfile
import sys
import io

#
# Non-empty Python-created files work
#
filename = "non_empty_python.tar"
with tarfile.TarFile(filename,
                     mode="w",
                     format=tarfile.PAX_FORMAT,
                     pax_headers=dict(abc="def")
) as t:
    t.addfile(tarfile.TarInfo("ghi.txt"), io.BytesIO(b"ghi"))
subprocess.check_call(["tar", "tf", filename],
                      stdout=subprocess.DEVNULL)
with tarfile.TarFile(filename, mode="a"):
    pass

#
# Non-empty tar-created files work
#
filename = "non_empty_tar.tar"
subprocess.check_call(["/bin/bash", "-c", f"""
    tar -c -H pax --pax-option='foo=bar' -f {filename} \
        -P --transform 's/.*/ghi.txt/' <(echo ghi)
"""])
subprocess.check_call(["tar", "tf", filename],
                      stdout=subprocess.DEVNULL)
with tarfile.TarFile(filename, mode="a"):
    pass

#
# Empty Python-created files break
#
filename = "empty_python.tar"
with tarfile.TarFile(filename,
                     mode="w",
                     format=tarfile.PAX_FORMAT,
                     pax_headers=dict(abc="def")
) as t:
    pass
subprocess.check_call(["tar", "tf", filename],
                      stdout=subprocess.DEVNULL)
try:
    with tarfile.TarFile(filename, mode="a"):
        pass
except Exception:
    print(f"Opening empty Python-created PAX TAR {filename!r} failed:",
          file=sys.stderr)
    print(traceback.format_exc(), file=sys.stderr)

#
# Empty tar-created files break
#
filename = "empty_tar.tar"
subprocess.check_call(["/bin/bash", "-c", f"""
    tar -c -H pax --pax-option='foo=bar' -f {filename} \
        --files-from=/dev/null
"""])
subprocess.check_call(["tar", "tf", filename],
                      stdout=subprocess.DEVNULL)
try:
    with tarfile.TarFile(filename, mode="a"):
        pass
except Exception:
    print(f"Opening empty tar-created PAX TAR {filename!r} failed:",
          file=sys.stderr)
    print(traceback.format_exc(), file=sys.stderr)
CPython versions tested on:

3.15

Operating systems tested on:

Linux

Linked PRs
  • gh-149647
  • gh-156792

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.

Direzione di ricerca

Inizia con il reproducer e il percorso di apertura di tarfile.TarFile in modalità append, concentrandoti sugli archivi che contengono solo header PAX globali. Aggiungi una copertura di regressione sia per gli archivi PAX vuoti creati da Python sia per quelli creati da tar, e verifica che l'apertura in modalità "a" abbia successo senza un ReadError.

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

Valutazione

Stack tecnologico
python
Ambito
tooling
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
25/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.