python / python/cpython

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

Ouverte
#149,578 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

stdlib type-bug
Langage dominant
Python
Étoiles
77.2k
Forks
35.9k
Métriques de merge des PR
Métriques de PR en attente

Description

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

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par le reproducer et le chemin d’ouverture de tarfile.TarFile en mode append, en vous concentrant sur les archives contenant uniquement des en-têtes PAX globaux. Ajoutez une couverture de régression pour les archives PAX vides créées par Python et par tar, et vérifiez que leur ouverture en mode "a" réussit sans ReadError.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
tooling
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.