mirror of
https://github.com/postgres/postgres.git
synced 2026-07-08 09:10:57 -04:00
PL/Python and its hstore and jsonb transforms build SQL values from Python containers by calling Python C API functions that can return NULL, and in several places the result was used without first checking it. On the sequence side, PySequence_GetItem() is used when converting a returned sequence into a SQL array or composite value, when reading the argument list passed to plpy.execute() or plpy.cursor(), and when reading the list of type names given to plpy.prepare(). On the mapping side, the hstore and jsonb transforms call PyMapping_Size() and PyMapping_Items() and then index the result with PyList_GetItem() and PyTuple_GetItem(). All of these return NULL (or -1), with a Python exception set, for a broken object: for example one whose __getitem__() or items() raises, or which reports a length that disagrees with what it actually yields. The unchecked result was then dereferenced, crashing the backend. Fix this by checking the result of each call and reporting a regular error if it failed, so that the underlying Python exception is surfaced instead of taking down the session. Author: Richard Guo <guofenglinux@gmail.com> Reviewed-by: Ayush Tiwari <ayushtiwari.slg01@gmail.com> Discussion: https://postgr.es/m/CAMbWs49BKM9wP6m8bCXEpHwQKp7usvOGV6Jf=J7FYr_BCpxLqg@mail.gmail.com Backpatch-through: 14
540 lines
12 KiB
C
540 lines
12 KiB
C
#include "postgres.h"
|
|
|
|
#include "miscadmin.h"
|
|
#include "plpy_elog.h"
|
|
#include "plpy_typeio.h"
|
|
#include "plpy_util.h"
|
|
#include "utils/fmgrprotos.h"
|
|
#include "utils/jsonb.h"
|
|
#include "utils/numeric.h"
|
|
|
|
PG_MODULE_MAGIC_EXT(
|
|
.name = "jsonb_plpython",
|
|
.version = PG_VERSION
|
|
);
|
|
|
|
/* for PLyObject_AsString in plpy_typeio.c */
|
|
typedef char *(*PLyObject_AsString_t) (PyObject *plrv);
|
|
static PLyObject_AsString_t PLyObject_AsString_p;
|
|
|
|
typedef void (*PLy_elog_impl_t) (int elevel, const char *fmt, ...);
|
|
static PLy_elog_impl_t PLy_elog_impl_p;
|
|
|
|
/*
|
|
* decimal_constructor is a function from python library and used
|
|
* for transforming strings into python decimal type
|
|
*/
|
|
static PyObject *decimal_constructor;
|
|
|
|
static PyObject *PLyObject_FromJsonbContainer(JsonbContainer *jsonb);
|
|
static void PLyObject_ToJsonbValue(PyObject *obj,
|
|
JsonbInState *jsonb_state, bool is_elem);
|
|
|
|
typedef PyObject *(*PLyUnicode_FromStringAndSize_t)
|
|
(const char *s, Py_ssize_t size);
|
|
static PLyUnicode_FromStringAndSize_t PLyUnicode_FromStringAndSize_p;
|
|
|
|
/* Static asserts verify that typedefs above match original declarations */
|
|
StaticAssertVariableIsOfType(&PLyObject_AsString, PLyObject_AsString_t);
|
|
StaticAssertVariableIsOfType(&PLyUnicode_FromStringAndSize, PLyUnicode_FromStringAndSize_t);
|
|
StaticAssertVariableIsOfType(&PLy_elog_impl, PLy_elog_impl_t);
|
|
|
|
|
|
/*
|
|
* Module initialize function: fetch function pointers for cross-module calls.
|
|
*/
|
|
void
|
|
_PG_init(void)
|
|
{
|
|
PLyObject_AsString_p = (PLyObject_AsString_t)
|
|
load_external_function("$libdir/" PLPYTHON_LIBNAME, "PLyObject_AsString",
|
|
true, NULL);
|
|
PLyUnicode_FromStringAndSize_p = (PLyUnicode_FromStringAndSize_t)
|
|
load_external_function("$libdir/" PLPYTHON_LIBNAME, "PLyUnicode_FromStringAndSize",
|
|
true, NULL);
|
|
PLy_elog_impl_p = (PLy_elog_impl_t)
|
|
load_external_function("$libdir/" PLPYTHON_LIBNAME, "PLy_elog_impl",
|
|
true, NULL);
|
|
}
|
|
|
|
/* These defines must be after the _PG_init */
|
|
#define PLyObject_AsString (PLyObject_AsString_p)
|
|
#define PLyUnicode_FromStringAndSize (PLyUnicode_FromStringAndSize_p)
|
|
#undef PLy_elog
|
|
#define PLy_elog (PLy_elog_impl_p)
|
|
|
|
/*
|
|
* PLyUnicode_FromJsonbValue
|
|
*
|
|
* Transform string JsonbValue to Python string.
|
|
*/
|
|
static PyObject *
|
|
PLyUnicode_FromJsonbValue(JsonbValue *jbv)
|
|
{
|
|
Assert(jbv->type == jbvString);
|
|
|
|
return PLyUnicode_FromStringAndSize(jbv->val.string.val, jbv->val.string.len);
|
|
}
|
|
|
|
/*
|
|
* PLyUnicode_ToJsonbValue
|
|
*
|
|
* Transform Python string to JsonbValue.
|
|
*/
|
|
static void
|
|
PLyUnicode_ToJsonbValue(PyObject *obj, JsonbValue *jbvElem)
|
|
{
|
|
jbvElem->type = jbvString;
|
|
jbvElem->val.string.val = PLyObject_AsString(obj);
|
|
jbvElem->val.string.len = strlen(jbvElem->val.string.val);
|
|
}
|
|
|
|
/*
|
|
* PLyObject_FromJsonbValue
|
|
*
|
|
* Transform JsonbValue to PyObject.
|
|
*/
|
|
static PyObject *
|
|
PLyObject_FromJsonbValue(JsonbValue *jsonbValue)
|
|
{
|
|
switch (jsonbValue->type)
|
|
{
|
|
case jbvNull:
|
|
Py_RETURN_NONE;
|
|
|
|
case jbvBinary:
|
|
return PLyObject_FromJsonbContainer(jsonbValue->val.binary.data);
|
|
|
|
case jbvNumeric:
|
|
{
|
|
Datum num;
|
|
char *str;
|
|
|
|
num = NumericGetDatum(jsonbValue->val.numeric);
|
|
str = DatumGetCString(DirectFunctionCall1(numeric_out, num));
|
|
|
|
return PyObject_CallFunction(decimal_constructor, "s", str);
|
|
}
|
|
|
|
case jbvString:
|
|
return PLyUnicode_FromJsonbValue(jsonbValue);
|
|
|
|
case jbvBool:
|
|
if (jsonbValue->val.boolean)
|
|
Py_RETURN_TRUE;
|
|
else
|
|
Py_RETURN_FALSE;
|
|
|
|
default:
|
|
elog(ERROR, "unexpected jsonb value type: %d", jsonbValue->type);
|
|
return NULL;
|
|
}
|
|
}
|
|
|
|
/*
|
|
* PLyObject_FromJsonbContainer
|
|
*
|
|
* Transform JsonbContainer to PyObject.
|
|
*/
|
|
static PyObject *
|
|
PLyObject_FromJsonbContainer(JsonbContainer *jsonb)
|
|
{
|
|
JsonbIteratorToken r;
|
|
JsonbValue v;
|
|
JsonbIterator *it;
|
|
PyObject *result;
|
|
|
|
/* this can recurse via PLyObject_FromJsonbValue() */
|
|
check_stack_depth();
|
|
|
|
it = JsonbIteratorInit(jsonb);
|
|
r = JsonbIteratorNext(&it, &v, true);
|
|
|
|
switch (r)
|
|
{
|
|
case WJB_BEGIN_ARRAY:
|
|
if (v.val.array.rawScalar)
|
|
{
|
|
JsonbValue tmp;
|
|
|
|
if ((r = JsonbIteratorNext(&it, &v, true)) != WJB_ELEM ||
|
|
(r = JsonbIteratorNext(&it, &tmp, true)) != WJB_END_ARRAY ||
|
|
(r = JsonbIteratorNext(&it, &tmp, true)) != WJB_DONE)
|
|
elog(ERROR, "unexpected jsonb token: %d", r);
|
|
|
|
result = PLyObject_FromJsonbValue(&v);
|
|
}
|
|
else
|
|
{
|
|
PyObject *volatile elem = NULL;
|
|
|
|
result = PyList_New(0);
|
|
if (!result)
|
|
return NULL;
|
|
|
|
PG_TRY();
|
|
{
|
|
while ((r = JsonbIteratorNext(&it, &v, true)) != WJB_DONE)
|
|
{
|
|
if (r != WJB_ELEM)
|
|
continue;
|
|
|
|
elem = PLyObject_FromJsonbValue(&v);
|
|
|
|
PyList_Append(result, elem);
|
|
Py_XDECREF(elem);
|
|
elem = NULL;
|
|
}
|
|
}
|
|
PG_CATCH();
|
|
{
|
|
Py_XDECREF(elem);
|
|
Py_XDECREF(result);
|
|
PG_RE_THROW();
|
|
}
|
|
PG_END_TRY();
|
|
}
|
|
break;
|
|
|
|
case WJB_BEGIN_OBJECT:
|
|
{
|
|
PyObject *volatile result_v = PyDict_New();
|
|
PyObject *volatile key = NULL;
|
|
PyObject *volatile val = NULL;
|
|
|
|
if (!result_v)
|
|
return NULL;
|
|
|
|
PG_TRY();
|
|
{
|
|
while ((r = JsonbIteratorNext(&it, &v, true)) != WJB_DONE)
|
|
{
|
|
if (r != WJB_KEY)
|
|
continue;
|
|
|
|
key = PLyUnicode_FromJsonbValue(&v);
|
|
if (!key)
|
|
{
|
|
Py_XDECREF(result_v);
|
|
result_v = NULL;
|
|
break;
|
|
}
|
|
|
|
if ((r = JsonbIteratorNext(&it, &v, true)) != WJB_VALUE)
|
|
elog(ERROR, "unexpected jsonb token: %d", r);
|
|
|
|
val = PLyObject_FromJsonbValue(&v);
|
|
if (!val)
|
|
{
|
|
Py_XDECREF(key);
|
|
key = NULL;
|
|
Py_XDECREF(result_v);
|
|
result_v = NULL;
|
|
break;
|
|
}
|
|
|
|
PyDict_SetItem(result_v, key, val);
|
|
|
|
Py_XDECREF(key);
|
|
key = NULL;
|
|
Py_XDECREF(val);
|
|
val = NULL;
|
|
}
|
|
}
|
|
PG_CATCH();
|
|
{
|
|
Py_XDECREF(result_v);
|
|
Py_XDECREF(key);
|
|
Py_XDECREF(val);
|
|
PG_RE_THROW();
|
|
}
|
|
PG_END_TRY();
|
|
|
|
result = result_v;
|
|
}
|
|
break;
|
|
|
|
default:
|
|
elog(ERROR, "unexpected jsonb token: %d", r);
|
|
return NULL;
|
|
}
|
|
|
|
return result;
|
|
}
|
|
|
|
/*
|
|
* PLyMapping_ToJsonbValue
|
|
*
|
|
* Transform Python dict to JsonbValue.
|
|
*/
|
|
static void
|
|
PLyMapping_ToJsonbValue(PyObject *obj, JsonbInState *jsonb_state)
|
|
{
|
|
Py_ssize_t pcount;
|
|
PyObject *volatile items;
|
|
|
|
pcount = PyMapping_Size(obj);
|
|
if (pcount < 0)
|
|
PLy_elog(ERROR, "could not get size of Python mapping");
|
|
|
|
items = PyMapping_Items(obj);
|
|
if (items == NULL)
|
|
PLy_elog(ERROR, "could not get items from Python mapping");
|
|
|
|
PG_TRY();
|
|
{
|
|
Py_ssize_t i;
|
|
|
|
pushJsonbValue(jsonb_state, WJB_BEGIN_OBJECT, NULL);
|
|
|
|
for (i = 0; i < pcount; i++)
|
|
{
|
|
JsonbValue jbvKey;
|
|
PyObject *item = PyList_GetItem(items, i);
|
|
PyObject *key;
|
|
PyObject *value;
|
|
|
|
/* The mapping's items() must yield key/value pairs */
|
|
if (item == NULL || !PyTuple_Check(item) || PyTuple_Size(item) < 2)
|
|
PLy_elog(ERROR, "items() of a Python mapping must return key/value pairs");
|
|
|
|
key = PyTuple_GetItem(item, 0);
|
|
value = PyTuple_GetItem(item, 1);
|
|
|
|
/* Python dictionary can have None as key */
|
|
if (key == Py_None)
|
|
{
|
|
jbvKey.type = jbvString;
|
|
jbvKey.val.string.len = 0;
|
|
jbvKey.val.string.val = "";
|
|
}
|
|
else
|
|
{
|
|
/* All others types of keys we serialize to string */
|
|
PLyUnicode_ToJsonbValue(key, &jbvKey);
|
|
}
|
|
|
|
pushJsonbValue(jsonb_state, WJB_KEY, &jbvKey);
|
|
PLyObject_ToJsonbValue(value, jsonb_state, false);
|
|
}
|
|
|
|
pushJsonbValue(jsonb_state, WJB_END_OBJECT, NULL);
|
|
}
|
|
PG_FINALLY();
|
|
{
|
|
Py_DECREF(items);
|
|
}
|
|
PG_END_TRY();
|
|
}
|
|
|
|
/*
|
|
* PLySequence_ToJsonbValue
|
|
*
|
|
* Transform python list to JsonbValue. Expects transformed PyObject and
|
|
* a state required for jsonb construction.
|
|
*/
|
|
static void
|
|
PLySequence_ToJsonbValue(PyObject *obj, JsonbInState *jsonb_state)
|
|
{
|
|
Py_ssize_t i;
|
|
Py_ssize_t pcount;
|
|
PyObject *volatile value = NULL;
|
|
|
|
pcount = PySequence_Size(obj);
|
|
|
|
pushJsonbValue(jsonb_state, WJB_BEGIN_ARRAY, NULL);
|
|
|
|
PG_TRY();
|
|
{
|
|
for (i = 0; i < pcount; i++)
|
|
{
|
|
value = PySequence_GetItem(obj, i);
|
|
|
|
/* PySequence_GetItem() can return NULL, with an exception set */
|
|
if (value == NULL)
|
|
PLy_elog(ERROR, "could not get element %d from sequence", (int) i);
|
|
|
|
PLyObject_ToJsonbValue(value, jsonb_state, true);
|
|
Py_XDECREF(value);
|
|
value = NULL;
|
|
}
|
|
}
|
|
PG_CATCH();
|
|
{
|
|
Py_XDECREF(value);
|
|
PG_RE_THROW();
|
|
}
|
|
PG_END_TRY();
|
|
|
|
pushJsonbValue(jsonb_state, WJB_END_ARRAY, NULL);
|
|
}
|
|
|
|
/*
|
|
* PLyNumber_ToJsonbValue(PyObject *obj)
|
|
*
|
|
* Transform python number to JsonbValue.
|
|
*/
|
|
static JsonbValue *
|
|
PLyNumber_ToJsonbValue(PyObject *obj, JsonbValue *jbvNum)
|
|
{
|
|
Numeric num;
|
|
char *str = PLyObject_AsString(obj);
|
|
|
|
PG_TRY();
|
|
{
|
|
Datum numd;
|
|
|
|
numd = DirectFunctionCall3(numeric_in,
|
|
CStringGetDatum(str),
|
|
ObjectIdGetDatum(InvalidOid),
|
|
Int32GetDatum(-1));
|
|
num = DatumGetNumeric(numd);
|
|
}
|
|
PG_CATCH();
|
|
{
|
|
ereport(ERROR,
|
|
(errcode(ERRCODE_DATATYPE_MISMATCH),
|
|
errmsg("could not convert value \"%s\" to jsonb", str)));
|
|
}
|
|
PG_END_TRY();
|
|
|
|
pfree(str);
|
|
|
|
/*
|
|
* jsonb doesn't allow NaN or infinity (per JSON specification), so we
|
|
* have to reject those here explicitly.
|
|
*/
|
|
if (numeric_is_nan(num))
|
|
ereport(ERROR,
|
|
(errcode(ERRCODE_NUMERIC_VALUE_OUT_OF_RANGE),
|
|
errmsg("cannot convert NaN to jsonb")));
|
|
if (numeric_is_inf(num))
|
|
ereport(ERROR,
|
|
(errcode(ERRCODE_NUMERIC_VALUE_OUT_OF_RANGE),
|
|
errmsg("cannot convert infinity to jsonb")));
|
|
|
|
jbvNum->type = jbvNumeric;
|
|
jbvNum->val.numeric = num;
|
|
|
|
return jbvNum;
|
|
}
|
|
|
|
/*
|
|
* PLyObject_ToJsonbValue(PyObject *obj)
|
|
*
|
|
* Transform python object to JsonbValue.
|
|
*/
|
|
static void
|
|
PLyObject_ToJsonbValue(PyObject *obj, JsonbInState *jsonb_state, bool is_elem)
|
|
{
|
|
JsonbValue *out;
|
|
|
|
/* this can recurse via PLyMapping_ToJsonbValue() */
|
|
check_stack_depth();
|
|
|
|
if (!PyUnicode_Check(obj))
|
|
{
|
|
if (PySequence_Check(obj))
|
|
{
|
|
PLySequence_ToJsonbValue(obj, jsonb_state);
|
|
return;
|
|
}
|
|
else if (PyMapping_Check(obj))
|
|
{
|
|
PLyMapping_ToJsonbValue(obj, jsonb_state);
|
|
return;
|
|
}
|
|
}
|
|
|
|
out = palloc_object(JsonbValue);
|
|
|
|
if (obj == Py_None)
|
|
out->type = jbvNull;
|
|
else if (PyUnicode_Check(obj))
|
|
PLyUnicode_ToJsonbValue(obj, out);
|
|
|
|
/*
|
|
* PyNumber_Check() returns true for booleans, so boolean check should
|
|
* come first.
|
|
*/
|
|
else if (PyBool_Check(obj))
|
|
{
|
|
out->type = jbvBool;
|
|
out->val.boolean = (obj == Py_True);
|
|
}
|
|
else if (PyNumber_Check(obj))
|
|
out = PLyNumber_ToJsonbValue(obj, out);
|
|
else
|
|
ereport(ERROR,
|
|
(errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
|
|
errmsg("Python type \"%s\" cannot be transformed to jsonb",
|
|
PLyObject_AsString((PyObject *) obj->ob_type))));
|
|
|
|
if (jsonb_state->parseState)
|
|
{
|
|
/* We're in an array or object, so push value as element or field. */
|
|
pushJsonbValue(jsonb_state, is_elem ? WJB_ELEM : WJB_VALUE, out);
|
|
}
|
|
else
|
|
{
|
|
/*
|
|
* We are at top level, so it's a raw scalar. If we just shove the
|
|
* scalar value into jsonb_state->result, JsonbValueToJsonb will take
|
|
* care of wrapping it into a dummy array.
|
|
*/
|
|
jsonb_state->result = out;
|
|
}
|
|
}
|
|
|
|
/*
|
|
* plpython_to_jsonb
|
|
*
|
|
* Transform python object to Jsonb datum
|
|
*/
|
|
PG_FUNCTION_INFO_V1(plpython_to_jsonb);
|
|
Datum
|
|
plpython_to_jsonb(PG_FUNCTION_ARGS)
|
|
{
|
|
PyObject *obj = (PyObject *) PG_GETARG_POINTER(0);
|
|
JsonbInState jsonb_state = {0};
|
|
|
|
PLyObject_ToJsonbValue(obj, &jsonb_state, true);
|
|
PG_RETURN_POINTER(JsonbValueToJsonb(jsonb_state.result));
|
|
}
|
|
|
|
/*
|
|
* jsonb_to_plpython
|
|
*
|
|
* Transform Jsonb datum to PyObject and return it as internal.
|
|
*/
|
|
PG_FUNCTION_INFO_V1(jsonb_to_plpython);
|
|
Datum
|
|
jsonb_to_plpython(PG_FUNCTION_ARGS)
|
|
{
|
|
PyObject *result;
|
|
Jsonb *in = PG_GETARG_JSONB_P(0);
|
|
|
|
/*
|
|
* Initialize pointer to Decimal constructor. First we try "cdecimal", C
|
|
* version of decimal library. In case of failure we use slower "decimal"
|
|
* module.
|
|
*/
|
|
if (!decimal_constructor)
|
|
{
|
|
PyObject *decimal_module = PyImport_ImportModule("cdecimal");
|
|
|
|
if (!decimal_module)
|
|
{
|
|
PyErr_Clear();
|
|
decimal_module = PyImport_ImportModule("decimal");
|
|
}
|
|
Assert(decimal_module);
|
|
decimal_constructor = PyObject_GetAttrString(decimal_module, "Decimal");
|
|
}
|
|
|
|
result = PLyObject_FromJsonbContainer(&in->root);
|
|
if (!result)
|
|
PLy_elog(ERROR, "transformation from jsonb to Python failed");
|
|
|
|
return PointerGetDatum(result);
|
|
}
|