gcc: make a couple of gcc warnings go away

diff --git a/device.c b/device.c
index 1b0b50fd4e796752826d79a278c4d322c7430919..6de7d518c718cfc30496f7a5fb45f9824a25ad2e 100644 (file)
--- a/device.c
+++ b/device.c
@@ -186,8 +186,10 @@ static int device_process_new_device(Manager *m, struct udev_device *dev, bool u
 
         if (names) {
                 FOREACH_WORD(w, l, names, state) {
-                        if (!(e = strndup(w, l)))
+                        if (!(e = strndup(w, l))) {
+                                r = -ENOMEM;
                                 goto fail;
+                        }
 
                         r = unit_add_name(u, e);
                         free(e);
@@ -199,8 +201,10 @@ static int device_process_new_device(Manager *m, struct udev_device *dev, bool u
 
         if (wants) {
                 FOREACH_WORD(w, l, wants, state) {
-                        if (!(e = strndup(w, l)))
+                        if (!(e = strndup(w, l))) {
+                                r = -ENOMEM;
                                 goto fail;
+                        }
 
                         r = unit_add_dependency_by_name(u, UNIT_WANTS, e);
                         free(e);

diff --git a/job.c b/job.c
index be42eaff62f5ed9fcbad31131833bf1aa0b3cc04..e2354af3fc1a2305c5ebcdeb41ab56c0c1682675 100644 (file)
--- a/job.c
+++ b/job.c
@@ -410,7 +410,7 @@ int job_run_and_invalidate(Job *j) {
 int job_finish_and_invalidate(Job *j, bool success) {
         Unit *u;
         Unit *other;
-        UnitType t;
+        JobType t;
         Iterator i;
 
         assert(j);

diff --git a/service.c b/service.c
index 28416894676591c940b2475ec862db211068380d..149a791b05f62a8ca76b3a7c732a90f88c8c75ef 100644 (file)
--- a/service.c
+++ b/service.c
@@ -1436,7 +1436,7 @@ static void service_sigchld_event(Unit *u, pid_t pid, int code, int status) {
                  * don't care about failing commands. */
 
                 if (s->control_command->command_next &&
-                    (success || (s->state == SERVICE_EXEC_STOP || s->state == SERVICE_EXEC_STOP_POST)))
+                    (success || (s->state == SERVICE_STOP || s->state == SERVICE_STOP_POST)))
 
                         /* There is another command to *
                          * execute, so let's do that. */

diff --git a/socket.c b/socket.c
index 79933347d0e02d296e8858d2e421360bf89f4f3e..afa001f20f3c0b306f9847c6f124f77cde983613 100644 (file)
--- a/socket.c
+++ b/socket.c
@@ -701,7 +701,7 @@ static void socket_sigchld_event(Unit *u, pid_t pid, int code, int status) {
         log_debug("%s control process exited, code=%s status=%i", unit_id(u), sigchld_code_to_string(code), status);
 
         if (s->control_command->command_next &&
-            (success || (s->state == SOCKET_EXEC_STOP_PRE || s->state == SOCKET_EXEC_STOP_POST))) {
+            (success || (s->state == SOCKET_STOP_PRE || s->state == SOCKET_STOP_POST))) {
                 log_debug("%s running next command for the state %s", unit_id(u), state_string_table[s->state]);
                 socket_run_next(s, success);
         } else {

diff --git a/util.c b/util.c
index d2b73125c093d4752205a0d428b46591ee5bdea6..9df6af953f0e13d9ce881f442e88a6894ad799c9 100644 (file)
--- a/util.c
+++ b/util.c
@@ -903,8 +903,8 @@ char *xescape(const char *s, const char *bad) {
 
         for (f = s, t = r; *f; f++) {
 
-                if (*f < ' ' || *f >= 127 ||
-                    *f == '\\' || strchr(bad, *f)) {
+                if ((*f < ' ') || (*f >= 127) ||
+                    (*f == '\\') || strchr(bad, *f)) {
                         *(t++) = '\\';
                         *(t++) = 'x';
                         *(t++) = hexchar(*f >> 4);