pypy/module/cpyext: va_end should be called before return

Author: Mingshen Sun

pypy/module/cpyext/src/getargs.c

@@ -1804,6 +1804,7 @@ PyArg_UnpackTuple(PyObject *args, const char *name, Py_ssize_t min, Py_ssize_t m
     if (!PyTuple_Check(args)) {
         PyErr_SetString(PyExc_SystemError,
             "PyArg_UnpackTuple() argument list is not a tuple");
+        va_end(vargs);
         return 0;
     }
     l = PyTuple_GET_SIZE(args);

pypy/module/cpyext/src/varargwrapper.c

@@ -16,8 +16,10 @@ PyTuple_Pack(Py_ssize_t n, ...)
     for (i = 0; i < n; i++) {
         o = va_arg(vargs, PyObject *);
         Py_INCREF(o);
-        if (PyTuple_SetItem(result, i, o) < 0)
+        if (PyTuple_SetItem(result, i, o) < 0) {
+            va_end(vargs);
             return NULL;
+        }
     }
     va_end(vargs);
     return result;