From f0a473c55a4ecb6ab85af7d6518261ab98e60d12 Mon Sep 17 00:00:00 2001 From: Douglas Bagnall Date: Tue, 3 Dec 2019 11:17:26 +1300 Subject: [PATCH] pygpo: use correct method flags The METH_KEYWORDS argument must always be combined with METH_VARARGS. In Python up to 3.7 this was checked at runtime, and as we had no callers to get_unix_path() in Python we never noticed. In Python 3.8 it is checked at import time, and everyone notices even if they aren't directly using GPOs. Found and reported by Val Kulkov. BUG: https://bugzilla.samba.org/show_bug.cgi?id=14209 Signed-off-by: Douglas Bagnall Reviewed-by: Andrew Bartlett (cherry picked from commit 12cccf3447333dfd4f5e437cd57ca5ec68724fdd) --- libgpo/pygpo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libgpo/pygpo.c b/libgpo/pygpo.c index 4db8cad7ca4..b8dfcd5572f 100644 --- a/libgpo/pygpo.c +++ b/libgpo/pygpo.c @@ -117,7 +117,7 @@ out: } static PyMethodDef GPO_methods[] = { - {"get_unix_path", (PyCFunction)py_gpo_get_unix_path, METH_KEYWORDS, + {"get_unix_path", (PyCFunction)py_gpo_get_unix_path, METH_VARARGS | METH_KEYWORDS, NULL }, {NULL} }; -- 2.20.1