python / python/cpython

plistlib inherits expat's newline-normalizing behavior, which can corrupt data in XML plists

Aperta
#139,423 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:

This is just a special case of #84561 but it came up in the specific circumstance of XML plists in MOPUp, here: https://github.com/glyph/MOPUp/pull/448/files#r2280646662

I also discovered that it is possible to get the unmodified data, by changing plistlib._PlistParser.handle_data to look like this:

    def handle_data(self, data):
        ctx = self.parser.GetInputContext()
        if data == '\n' and ctx[0:1] != b'\n':
            data = ctx[0:1].decode("utf-8")
        self.data.append(data)

so we are not totally bound by the whims of Expat here; these types of bug could be fixed.

CPython versions tested on:

3.12, 3.13, 3.11

Operating systems tested on:

macOS

Linked PRs
  • gh-148305

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 plistlib._PlistParser.handle_data e il comportamento del contesto di input descritto nel report. Verifica come vengono attualmente normalizzati i dati plist XML e assicurati che i dati non modificati vengano preservati; controlla il PR collegato gh-148305 prima di iniziare, perché questo lavoro potrebbe essere già in corso.

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

Valutazione

Stack tecnologico
python
Ambito
backend
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
30/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.