summaryrefslogtreecommitdiffstats
path: root/Modules/_sqlite/statement.c
diff options
context:
space:
mode:
authorGuido van Rossum <guido@python.org>2007-08-29 03:34:29 (GMT)
committerGuido van Rossum <guido@python.org>2007-08-29 03:34:29 (GMT)
commitfa9a121952db01168e8d131549680d8bb8e9ddb7 (patch)
treed5b34d56a98e011f5ad8593ba404882c3d2c62e1 /Modules/_sqlite/statement.c
parent625cbf28eeeed3f2d8afffa6c442f10d58ae6607 (diff)
downloadcpython-fa9a121952db01168e8d131549680d8bb8e9ddb7.zip
cpython-fa9a121952db01168e8d131549680d8bb8e9ddb7.tar.gz
cpython-fa9a121952db01168e8d131549680d8bb8e9ddb7.tar.bz2
"Fix" a few places that were using PyObject_AsCharBuffer() to convert a string
(PyUnicode these days) to a char* + length. The fix consists of calling PyUnicode_AsString() and strlen(). This is not ideal, but AsCharBuffer() is definitely not the API to use.
Diffstat (limited to 'Modules/_sqlite/statement.c')
-rw-r--r--Modules/_sqlite/statement.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/Modules/_sqlite/statement.c b/Modules/_sqlite/statement.c
index b80b955..a5801d6 100644
--- a/Modules/_sqlite/statement.c
+++ b/Modules/_sqlite/statement.c
@@ -50,10 +50,12 @@ int pysqlite_statement_create(pysqlite_Statement* self, pysqlite_Connection* con
self->st = NULL;
self->in_use = 0;
- if (PyObject_AsCharBuffer(sql, &sql_cstr, &sql_cstr_len) < 0) {
+ sql_cstr = PyUnicode_AsString(sql);
+ if (sql_cstr == NULL) {
rc = PYSQLITE_SQL_WRONG_TYPE;
return rc;
}
+ sql_cstr_len = strlen(sql_cstr); /* XXX */
self->in_weakreflist = NULL;
Py_INCREF(sql);
@@ -214,10 +216,12 @@ int pysqlite_statement_recompile(pysqlite_Statement* self, PyObject* params)
Py_ssize_t sql_len;
sqlite3_stmt* new_st;
- if (PyObject_AsCharBuffer(self->sql, &sql_cstr, &sql_len) < 0) {
+ sql_cstr = PyUnicode_AsString(self->sql);
+ if (sql_cstr == NULL) {
rc = PYSQLITE_SQL_WRONG_TYPE;
return rc;
}
+ sql_len = strlen(sql_cstr); /* XXXX */
rc = sqlite3_prepare(self->db,
sql_cstr,