_ctypes fails to extend the return value per libffi contract
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 77.2k
- Forks
- 35.9k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in Modules/_ctypes/callbacks.c bei der verknüpften Rückgabewertbehandlung und lies den referenzierten libffi-Closure-Vertrag. Reproduziere oder untersuche das c_int-Callback-Verhalten auf s390x und verifiziere anschließend, dass schmale ganzzahlige Rückgabewerte der erforderlichen ABI-Erweiterung folgen, anstatt nur restype->size Bytes zu kopieren. Prüfe den verknüpften CPython PR auf die bereits laufenden Arbeiten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- c, python
- Bereich
- backend
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 35/100