_ctypes fails to extend the return value per libffi contract
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 77.2k
- Fork
- 35.9k
- Metriche di merge delle PR
- Metriche PR in attesa
Descrizione
Bug report
Bug description:
The current implementation is relying on restype->size to write the data into a buffer, however, per libffi contract
integral types narrower than the system register size are widened
Which is not happening here. This has caused a failure on s390x when using libclang. cindex.py calls into libclang through ctypes, which builds the callback closures using libffi. With a c_int return, ctypes writes only the low 32 bits and skips the extension that both the ABI and libffi's closure contract require.
We have also opened a PR to fix this temporarily under llvm: https://github.com/llvm/llvm-project/pull/221024
CPython versions tested on:
3.11
Operating systems tested on:
Linux
Linked PRs
- gh-157045
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia in Modules/_ctypes/callbacks.c, nella gestione collegata del valore restituito, e leggi il contratto delle closure di libffi a cui si fa riferimento. Riproduci o esamina il comportamento della callback c_int su s390x, quindi verifica che i valori restituiti integrali stretti seguano l’estensione ABI richiesta invece di limitarsi a copiare restype->size byte. Controlla il PR di CPython collegato per il lavoro già in corso.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- c, python
- Ambito
- backend
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 35/100