postgrespro.git / summary / log / commit / refs

commit    dad8bed04ab98ada84ecd58ace6f59839aa161c4
Author:   Alexander Korotkov <akorotkov@postgresql.org>
Date:     Fri Jun 15 12:01:46 2018 +0000

    Fix memory leak in PLySequence_ToJsonbValue()
    
    PyObject returned from PySequence_GetItem() is not released.  Similar code in PLyMapping_ToJsonbValue() is correct, because according to Python documentation
    PyList_GetItem() and PyTuple_GetItem() return a borrowed reference while
    PySequence_GetItem() returns new reference.  contrib/jsonb_plpython is new
    in PostgreSQL 11, no backpatch is needed.
    
    Author: Nikita Glukhov
    Discussion: https://postgr.es/m/6001af16-b242-2527-bc7e-84b8a959163b%40postgrespro.ru


 contrib/jsonb_plpython/jsonb_plpython.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/contrib/jsonb_plpython/jsonb_plpython.c b/contrib/jsonb_plpython/jsonb_plpython.c
index f752d6c3cd..d6d6eeb9c1 100644
--- a/contrib/jsonb_plpython/jsonb_plpython.c
+++ b/contrib/jsonb_plpython/jsonb_plpython.c
@@ -308,6 +308,8 @@ PLySequence_ToJsonbValue(PyObject *obj, JsonbParseState **jsonb_state)
 		PyObject   *value = PySequence_GetItem(obj, i);
 
 		(void) PLyObject_ToJsonbValue(value, jsonb_state, true);
+
+		Py_XDECREF(value);
 	}
 
 	return pushJsonbValue(jsonb_state, WJB_END_ARRAY, NULL);


[parent: 969274d81301]