bpo-43853: Handle sqlite3_value_text() errors#25422
Conversation
|
@pablogsal, see top comment for a description of the changes. Relevant SQLite docs: https://sqlite.org/c3ref/value_blob.html
We call the API's in the following order:
|
Sorry, something went wrong.
|
Thanks @erlend-aasland for the PR, and @pablogsal for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10. |
Sorry, something went wrong.
|
Thanks for reviewing, Pablo! |
Sorry, something went wrong.
(cherry picked from commit 006fd86) Co-authored-by: Erlend Egeberg Aasland <erlend.aasland@innova.no>
|
Thanks @erlend-aasland for the PR, and @pablogsal for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9. |
Sorry, something went wrong.
|
Sorry, @erlend-aasland and @pablogsal, I could not cleanly backport this to |
Sorry, something went wrong.
|
I'll fix the backport. |
Sorry, something went wrong.
(cherry picked from commit 006fd86) Co-authored-by: Erlend Egeberg Aasland <erlend.aasland@innova.no>
MemoryErrorifsqlite3_value_text()setsSQLITE_NOMEMsqlite3_value_bytes()to find string length; must be called aftersqlite3_value_text()PyUnicode_FromStringwithPyUnicode_FromStringAndSize()and let errors propagatehttps://bugs.python.org/issue43853