]> git.ipfire.org Git - thirdparty/collectd.git/commitdiff
Fix the return-value check of PyType_Ready 3402/head
authorJia Zhouyang <jiazhouyang@nudt.edu.cn>
Mon, 9 Mar 2020 15:50:21 +0000 (11:50 -0400)
committerGitHub <noreply@github.com>
Mon, 9 Mar 2020 15:50:21 +0000 (11:50 -0400)
Compare the return value of PyType_Ready with -1 directly.

src/python.c

index 01314213b1cbc7f7a43985867fd4e53923531a10..119ffe21742ab20d8909c45c23c000e7ae3a67e9 100644 (file)
@@ -1261,31 +1261,31 @@ static int cpy_init_python(void) {
   Py_Initialize();
   python_sigint_handler = PyOS_setsig(SIGINT, cur_sig);
 
-  if (PyType_Ready(&ConfigType) < 0) {
+  if (PyType_Ready(&ConfigType) == -1) {
     cpy_log_exception("python initialization: ConfigType");
     return 1;
   }
-  if (PyType_Ready(&PluginDataType) < 0) {
+  if (PyType_Ready(&PluginDataType) == -1) {
     cpy_log_exception("python initialization: PluginDataType");
     return 1;
   }
   ValuesType.tp_base = &PluginDataType;
-  if (PyType_Ready(&ValuesType) < 0) {
+  if (PyType_Ready(&ValuesType) == -1) {
     cpy_log_exception("python initialization: ValuesType");
     return 1;
   }
   NotificationType.tp_base = &PluginDataType;
-  if (PyType_Ready(&NotificationType) < 0) {
+  if (PyType_Ready(&NotificationType) == -1) {
     cpy_log_exception("python initialization: NotificationType");
     return 1;
   }
   SignedType.tp_base = &PyLong_Type;
-  if (PyType_Ready(&SignedType) < 0) {
+  if (PyType_Ready(&SignedType) == -1) {
     cpy_log_exception("python initialization: SignedType");
     return 1;
   }
   UnsignedType.tp_base = &PyLong_Type;
-  if (PyType_Ready(&UnsignedType) < 0) {
+  if (PyType_Ready(&UnsignedType) == -1) {
     cpy_log_exception("python initialization: UnsignedType");
     return 1;
   }