BUG/MINOR: contrib/spoa-server: Do not free reference to NULL
authorGilchrist Dadaglo <dadaglo@amazon.com>
Mon, 24 Aug 2020 19:21:33 +0000 (19:21 +0000)
committerChristopher Faulet <cfaulet@haproxy.com>
Mon, 7 Sep 2020 07:31:25 +0000 (09:31 +0200)
As per https://docs.python.org/3/c-api/refcounting.html, Py_DECREF
should not be called on NULL objects.

This patch must be backported as far as 2.0.

(cherry picked from commit c7d303a0a8962f955e9dd271ee9775dd8eb88cfe)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit dcda9a0cf1d8c381208c426cbf500c7e5f9a3468)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>

contrib/spoa_server/ps_python.c

index a8e751f..dd4e63d 100644 (file)
@@ -449,7 +449,6 @@ static int ps_python_exec_message(struct worker *w, void *ref, int nargs, struct
                ent = PyDict_New();
                if (ent == NULL) {
                        Py_DECREF(kw_args);
-                       Py_DECREF(ent);
                        PyErr_Print();
                        return 0;
                }