daandemeyer / rpms / systemd

Forked from rpms/systemd 2 years ago
Clone
65878a
From b22a215807207c1a6332a22505840478ac633afd Mon Sep 17 00:00:00 2001
65878a
From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl>
65878a
Date: Sun, 29 Dec 2013 23:39:28 -0500
65878a
Subject: [PATCH] systemd-python: fix setting of exception codes
65878a
65878a
The return value of 0 would be treated as failure by mistake,
65878a
resulting in " SystemError: error return without exception set".
65878a
The way that set_error() is used is changed to be the same
65878a
everywhere.
65878a
---
65878a
 src/python-systemd/_daemon.c | 14 +++++-----
65878a
 src/python-systemd/_reader.c | 63 ++++++++++++++++++++------------------------
65878a
 2 files changed, 35 insertions(+), 42 deletions(-)
65878a
65878a
diff --git a/src/python-systemd/_daemon.c b/src/python-systemd/_daemon.c
65878a
index f0ab16f..c6db69f 100644
65878a
--- a/src/python-systemd/_daemon.c
65878a
+++ b/src/python-systemd/_daemon.c
65878a
@@ -88,7 +88,7 @@ static PyObject* notify(PyObject *self, PyObject *args, PyObject *keywds) {
65878a
 #endif
65878a
 
65878a
         r = sd_notify(unset, msg);
65878a
-        if (set_error(r, NULL, NULL))
65878a
+        if (set_error(r, NULL, NULL) < 0)
65878a
                 return NULL;
65878a
 
65878a
         return PyBool_FromLong(r);
65878a
@@ -123,7 +123,7 @@ static PyObject* listen_fds(PyObject *self, PyObject *args, PyObject *keywds) {
65878a
 #endif
65878a
 
65878a
         r = sd_listen_fds(unset);
65878a
-        if (set_error(r, NULL, NULL))
65878a
+        if (set_error(r, NULL, NULL) < 0)
65878a
                 return NULL;
65878a
 
65878a
         return long_FromLong(r);
65878a
@@ -151,7 +151,7 @@ static PyObject* is_fifo(PyObject *self, PyObject *args) {
65878a
 #endif
65878a
 
65878a
         r = sd_is_fifo(fd, path);
65878a
-        if (set_error(r, path, NULL))
65878a
+        if (set_error(r, path, NULL) < 0)
65878a
                 return NULL;
65878a
 
65878a
         return PyBool_FromLong(r);
65878a
@@ -179,7 +179,7 @@ static PyObject* is_mq(PyObject *self, PyObject *args) {
65878a
 #endif
65878a
 
65878a
         r = sd_is_mq(fd, path);
65878a
-        if (set_error(r, path, NULL))
65878a
+        if (set_error(r, path, NULL) < 0)
65878a
                 return NULL;
65878a
 
65878a
         return PyBool_FromLong(r);
65878a
@@ -203,7 +203,7 @@ static PyObject* is_socket(PyObject *self, PyObject *args) {
65878a
                 return NULL;
65878a
 
65878a
         r = sd_is_socket(fd, family, type, listening);
65878a
-        if (set_error(r, NULL, NULL))
65878a
+        if (set_error(r, NULL, NULL) < 0)
65878a
                 return NULL;
65878a
 
65878a
         return PyBool_FromLong(r);
65878a
@@ -230,7 +230,7 @@ static PyObject* is_socket_inet(PyObject *self, PyObject *args) {
65878a
         }
65878a
 
65878a
         r = sd_is_socket_inet(fd, family, type, listening, (uint16_t) port);
65878a
-        if (set_error(r, NULL, NULL))
65878a
+        if (set_error(r, NULL, NULL) < 0)
65878a
                 return NULL;
65878a
 
65878a
         return PyBool_FromLong(r);
65878a
@@ -265,7 +265,7 @@ static PyObject* is_socket_unix(PyObject *self, PyObject *args) {
65878a
 #endif
65878a
 
65878a
         r = sd_is_socket_unix(fd, type, listening, path, length);
65878a
-        if (set_error(r, path, NULL))
65878a
+        if (set_error(r, path, NULL) < 0)
65878a
                 return NULL;
65878a
 
65878a
         return PyBool_FromLong(r);
65878a
diff --git a/src/python-systemd/_reader.c b/src/python-systemd/_reader.c
65878a
index bc5db19..0c88926 100644
65878a
--- a/src/python-systemd/_reader.c
65878a
+++ b/src/python-systemd/_reader.c
65878a
@@ -206,8 +206,7 @@ PyDoc_STRVAR(Reader_reliable_fd__doc__,
65878a
 static PyObject* Reader_reliable_fd(Reader *self, PyObject *args)
65878a
 {
65878a
     int r = sd_journal_reliable_fd(self->j);
65878a
-    set_error(r, NULL, NULL);
65878a
-    if (r < 0)
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
     return PyBool_FromLong(r);
65878a
 }
65878a
@@ -221,8 +220,7 @@ PyDoc_STRVAR(Reader_get_events__doc__,
65878a
 static PyObject* Reader_get_events(Reader *self, PyObject *args)
65878a
 {
65878a
     int r = sd_journal_get_events(self->j);
65878a
-    set_error(r, NULL, NULL);
65878a
-    if (r < 0)
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
     return long_FromLong(r);
65878a
 }
65878a
@@ -242,8 +240,7 @@ static PyObject* Reader_get_timeout(Reader *self, PyObject *args)
65878a
     uint64_t t;
65878a
 
65878a
     r = sd_journal_get_timeout(self->j, &t);
65878a
-    set_error(r, NULL, NULL);
65878a
-    if (r < 0)
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
 
65878a
     if (t == (uint64_t) -1)
65878a
@@ -265,8 +262,7 @@ static PyObject* Reader_get_timeout_ms(Reader *self, PyObject *args)
65878a
     uint64_t t;
65878a
 
65878a
     r = sd_journal_get_timeout(self->j, &t);
65878a
-    set_error(r, NULL, NULL);
65878a
-    if (r < 0)
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
 
65878a
     return absolute_timeout(t);
65878a
@@ -304,7 +300,7 @@ static PyObject* Reader_get_usage(Reader *self, PyObject *args)
65878a
     uint64_t bytes;
65878a
 
65878a
     r = sd_journal_get_usage(self->j, &bytes);
65878a
-    if (set_error(r, NULL, NULL))
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
 
65878a
     assert_cc(sizeof(unsigned long long) == sizeof(bytes));
65878a
@@ -366,8 +362,7 @@ static PyObject* Reader_next(Reader *self, PyObject *args)
65878a
         assert_not_reached("should not be here");
65878a
     Py_END_ALLOW_THREADS
65878a
 
65878a
-    set_error(r, NULL, NULL);
65878a
-    if (r < 0)
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
     return PyBool_FromLong(r);
65878a
 }
65878a
@@ -445,7 +440,8 @@ static PyObject* Reader_get(Reader *self, PyObject *args)
65878a
     if (r == -ENOENT) {
65878a
         PyErr_SetString(PyExc_KeyError, field);
65878a
         return NULL;
65878a
-    } else if (set_error(r, NULL, "field name is not valid"))
65878a
+    }
65878a
+    if (set_error(r, NULL, "field name is not valid") < 0)
65878a
         return NULL;
65878a
 
65878a
     r = extract(msg, msg_len, NULL, &value);
65878a
@@ -530,7 +526,7 @@ static PyObject* Reader_get_realtime(Reader *self, PyObject *args)
65878a
     assert(!args);
65878a
 
65878a
     r = sd_journal_get_realtime_usec(self->j, &timestamp);
65878a
-    if (set_error(r, NULL, NULL))
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
 
65878a
     assert_cc(sizeof(unsigned long long) == sizeof(timestamp));
65878a
@@ -555,7 +551,7 @@ static PyObject* Reader_get_monotonic(Reader *self, PyObject *args)
65878a
     assert(!args);
65878a
 
65878a
     r = sd_journal_get_monotonic_usec(self->j, &timestamp, &id;;
65878a
-    if (set_error(r, NULL, NULL))
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
 
65878a
     assert_cc(sizeof(unsigned long long) == sizeof(timestamp));
65878a
@@ -598,8 +594,7 @@ static PyObject* Reader_add_match(Reader *self, PyObject *args, PyObject *keywds
65878a
         return NULL;
65878a
 
65878a
     r = sd_journal_add_match(self->j, match, match_len);
65878a
-    set_error(r, NULL, "Invalid match");
65878a
-    if (r < 0)
65878a
+    if (set_error(r, NULL, "Invalid match") < 0)
65878a
             return NULL;
65878a
 
65878a
     Py_RETURN_NONE;
65878a
@@ -616,8 +611,7 @@ static PyObject* Reader_add_disjunction(Reader *self, PyObject *args)
65878a
 {
65878a
     int r;
65878a
     r = sd_journal_add_disjunction(self->j);
65878a
-    set_error(r, NULL, NULL);
65878a
-    if (r < 0)
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
     Py_RETURN_NONE;
65878a
 }
65878a
@@ -633,8 +627,7 @@ static PyObject* Reader_add_conjunction(Reader *self, PyObject *args)
65878a
 {
65878a
     int r;
65878a
     r = sd_journal_add_conjunction(self->j);
65878a
-    set_error(r, NULL, NULL);
65878a
-    if (r < 0)
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
     Py_RETURN_NONE;
65878a
 }
65878a
@@ -661,7 +654,7 @@ static PyObject* Reader_seek_head(Reader *self, PyObject *args)
65878a
     Py_BEGIN_ALLOW_THREADS
65878a
     r = sd_journal_seek_head(self->j);
65878a
     Py_END_ALLOW_THREADS
65878a
-    if (set_error(r, NULL, NULL))
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
     Py_RETURN_NONE;
65878a
 }
65878a
@@ -678,7 +671,7 @@ static PyObject* Reader_seek_tail(Reader *self, PyObject *args)
65878a
     Py_BEGIN_ALLOW_THREADS
65878a
     r = sd_journal_seek_tail(self->j);
65878a
     Py_END_ALLOW_THREADS
65878a
-    if (set_error(r, NULL, NULL))
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
     Py_RETURN_NONE;
65878a
 }
65878a
@@ -699,7 +692,7 @@ static PyObject* Reader_seek_realtime(Reader *self, PyObject *args)
65878a
     Py_BEGIN_ALLOW_THREADS
65878a
     r = sd_journal_seek_realtime_usec(self->j, timestamp);
65878a
     Py_END_ALLOW_THREADS
65878a
-    if (set_error(r, NULL, NULL))
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
     Py_RETURN_NONE;
65878a
 }
65878a
@@ -723,20 +716,20 @@ static PyObject* Reader_seek_monotonic(Reader *self, PyObject *args)
65878a
 
65878a
     if (bootid) {
65878a
         r = sd_id128_from_string(bootid, &id;;
65878a
-        if (set_error(r, NULL, "Invalid bootid"))
65878a
+        if (set_error(r, NULL, "Invalid bootid") < 0)
65878a
             return NULL;
65878a
     } else {
65878a
         Py_BEGIN_ALLOW_THREADS
65878a
         r = sd_id128_get_boot(&id;;
65878a
         Py_END_ALLOW_THREADS
65878a
-        if (set_error(r, NULL, NULL))
65878a
+        if (set_error(r, NULL, NULL) < 0)
65878a
             return NULL;
65878a
     }
65878a
 
65878a
     Py_BEGIN_ALLOW_THREADS
65878a
     r = sd_journal_seek_monotonic_usec(self->j, id, timestamp);
65878a
     Py_END_ALLOW_THREADS
65878a
-    if (set_error(r, NULL, NULL))
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
 
65878a
     Py_RETURN_NONE;
65878a
@@ -809,7 +802,7 @@ static PyObject* Reader_seek_cursor(Reader *self, PyObject *args)
65878a
     Py_BEGIN_ALLOW_THREADS
65878a
     r = sd_journal_seek_cursor(self->j, cursor);
65878a
     Py_END_ALLOW_THREADS
65878a
-    if (set_error(r, NULL, "Invalid cursor"))
65878a
+    if (set_error(r, NULL, "Invalid cursor") < 0)
65878a
         return NULL;
65878a
     Py_RETURN_NONE;
65878a
 }
65878a
@@ -828,7 +821,7 @@ static PyObject* Reader_get_cursor(Reader *self, PyObject *args)
65878a
     assert(!args);
65878a
 
65878a
     r = sd_journal_get_cursor(self->j, &cursor);
65878a
-    if (set_error(r, NULL, NULL))
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
 
65878a
     return unicode_FromString(cursor);
65878a
@@ -851,8 +844,7 @@ static PyObject* Reader_test_cursor(Reader *self, PyObject *args)
65878a
         return NULL;
65878a
 
65878a
     r = sd_journal_test_cursor(self->j, cursor);
65878a
-    set_error(r, NULL, NULL);
65878a
-    if (r < 0)
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
 
65878a
     return PyBool_FromLong(r);
65878a
@@ -876,7 +868,7 @@ static PyObject* Reader_query_unique(Reader *self, PyObject *args)
65878a
     Py_BEGIN_ALLOW_THREADS
65878a
     r = sd_journal_query_unique(self->j, query);
65878a
     Py_END_ALLOW_THREADS
65878a
-    if (set_error(r, NULL, "Invalid field name"))
65878a
+    if (set_error(r, NULL, "Invalid field name") < 0)
65878a
         return NULL;
65878a
 
65878a
     value_set = PySet_New(0);
65878a
@@ -930,7 +922,8 @@ static PyObject* Reader_get_catalog(Reader *self, PyObject *args)
65878a
         else
65878a
             set_error(r, NULL, NULL);
65878a
         return NULL;
65878a
-    } else if (set_error(r, NULL, NULL))
65878a
+    }
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
 
65878a
     return unicode_FromString(msg);
65878a
@@ -955,13 +948,13 @@ static PyObject* get_catalog(PyObject *self, PyObject *args)
65878a
         return NULL;
65878a
 
65878a
     r = sd_id128_from_string(id_, &id;;
65878a
-    if (set_error(r, NULL, "Invalid id128"))
65878a
+    if (set_error(r, NULL, "Invalid id128") < 0)
65878a
         return NULL;
65878a
 
65878a
     Py_BEGIN_ALLOW_THREADS
65878a
     r = sd_journal_get_catalog_for_message_id(id, &msg;;
65878a
     Py_END_ALLOW_THREADS
65878a
-    if (set_error(r, NULL, NULL))
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
 
65878a
     return unicode_FromString(msg);
65878a
@@ -979,7 +972,7 @@ static PyObject* Reader_get_data_threshold(Reader *self, void *closure)
65878a
     int r;
65878a
 
65878a
     r = sd_journal_get_data_threshold(self->j, &cvalue);
65878a
-    if (set_error(r, NULL, NULL))
65878a
+    if (set_error(r, NULL, NULL) < 0)
65878a
         return NULL;
65878a
 
65878a
     return long_FromSize_t(cvalue);