summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--job.c11
-rw-r--r--main.c8
-rw-r--r--manager.c10
-rw-r--r--manager.h2
-rw-r--r--name.c33
-rw-r--r--name.h2
-rw-r--r--socket-util.c10
-rw-r--r--socket-util.h2
-rw-r--r--socket.c144
-rw-r--r--test1/postfix.socket4
-rw-r--r--test1/syslog.socket4
11 files changed, 181 insertions, 49 deletions
diff --git a/job.c b/job.c
index 0ae1a765fb..7accf7267b 100644
--- a/job.c
+++ b/job.c
@@ -362,6 +362,8 @@ int job_run_and_invalidate(Job *j) {
if (j->state != JOB_WAITING)
return 0;
+ j->state = JOB_RUNNING;
+
switch (j->type) {
case JOB_START:
@@ -422,11 +424,12 @@ int job_run_and_invalidate(Job *j) {
;
}
- if (r >= 0)
- j->state = JOB_RUNNING;
- else if (r == -EALREADY)
+ if (r == -EALREADY)
r = job_finish_and_invalidate(j, true);
- else if (r != -EAGAIN)
+ else if (r == -EAGAIN) {
+ j->state = JOB_WAITING;
+ return -EAGAIN;
+ } else if (r < 0)
r = job_finish_and_invalidate(j, false);
return r;
diff --git a/main.c b/main.c
index b22a639396..f073c8f28d 100644
--- a/main.c
+++ b/main.c
@@ -42,13 +42,7 @@ int main(int argc, char *argv[]) {
printf("- By jobs:\n");
manager_dump_jobs(m, stdout, "\t");
- if ((r = manager_add_job(m, JOB_STOP, syslog, JOB_REPLACE, false, &job)) < 0) {
- log_error("Failed to start default milestone: %s", strerror(-r));
- goto finish;
- }
-
- printf("- By jobs:\n");
- manager_dump_jobs(m, stdout, "\t");
+ manager_run_jobs(m);
retval = 0;
diff --git a/manager.c b/manager.c
index 27b740e58a..7941d89a75 100644
--- a/manager.c
+++ b/manager.c
@@ -882,3 +882,13 @@ void manager_clear_jobs(Manager *m) {
while ((j = hashmap_first(m->jobs)))
job_free(j);
}
+
+void manager_run_jobs(Manager *m) {
+ Job *j;
+ void *state;
+ int r;
+
+ HASHMAP_FOREACH(j, m->jobs, state) {
+ r = job_run_and_invalidate(j);
+ }
+}
diff --git a/manager.h b/manager.h
index e669f9f903..68f0dd94f8 100644
--- a/manager.h
+++ b/manager.h
@@ -54,4 +54,6 @@ void manager_transaction_unlink_job(Manager *m, Job *j);
void manager_clear_jobs(Manager *m);
+void manager_run_jobs(Manager *m);
+
#endif
diff --git a/name.c b/name.c
index 33d7f347b5..b6e62e9bee 100644
--- a/name.c
+++ b/name.c
@@ -617,7 +617,7 @@ static void retroactively_stop_dependencies(Name *n) {
manager_add_job(n->meta.manager, JOB_STOP, other, JOB_REPLACE, true, NULL);
}
-int name_notify(Name *n, NameActiveState os, NameActiveState ns) {
+void name_notify(Name *n, NameActiveState os, NameActiveState ns) {
assert(n);
assert(os < _NAME_ACTIVE_STATE_MAX);
assert(ns < _NAME_ACTIVE_STATE_MAX);
@@ -625,7 +625,7 @@ int name_notify(Name *n, NameActiveState os, NameActiveState ns) {
assert(!(os == NAME_INACTIVE && ns == NAME_DEACTIVATING));
if (os == ns)
- return 0;
+ return;
if (n->meta.job) {
@@ -649,10 +649,11 @@ int name_notify(Name *n, NameActiveState os, NameActiveState ns) {
case JOB_START:
case JOB_VERIFY_ACTIVE:
- if (NAME_IS_ACTIVE_OR_RELOADING(ns))
- return job_finish_and_invalidate(n->meta.job, true);
- else if (ns == NAME_ACTIVATING)
- return 0;
+ if (NAME_IS_ACTIVE_OR_RELOADING(ns)) {
+ job_finish_and_invalidate(n->meta.job, true);
+ return;
+ } else if (ns == NAME_ACTIVATING)
+ return;
else
job_finish_and_invalidate(n->meta.job, false);
@@ -661,10 +662,11 @@ int name_notify(Name *n, NameActiveState os, NameActiveState ns) {
case JOB_RELOAD:
case JOB_RELOAD_OR_START:
- if (ns == NAME_ACTIVE)
- return job_finish_and_invalidate(n->meta.job, true);
- else if (ns == NAME_ACTIVATING || ns == NAME_ACTIVE_RELOADING)
- return 0;
+ if (ns == NAME_ACTIVE) {
+ job_finish_and_invalidate(n->meta.job, true);
+ return;
+ } else if (ns == NAME_ACTIVATING || ns == NAME_ACTIVE_RELOADING)
+ return;
else
job_finish_and_invalidate(n->meta.job, false);
@@ -674,10 +676,11 @@ int name_notify(Name *n, NameActiveState os, NameActiveState ns) {
case JOB_RESTART:
case JOB_TRY_RESTART:
- if (ns == NAME_INACTIVE)
- return job_finish_and_invalidate(n->meta.job, true);
- else if (ns == NAME_DEACTIVATING)
- return 0;
+ if (ns == NAME_INACTIVE) {
+ job_finish_and_invalidate(n->meta.job, true);
+ return;
+ } else if (ns == NAME_DEACTIVATING)
+ return;
else
job_finish_and_invalidate(n->meta.job, false);
@@ -696,6 +699,4 @@ int name_notify(Name *n, NameActiveState os, NameActiveState ns) {
retroactively_start_dependencies(n);
else if (NAME_IS_ACTIVE_OR_ACTIVATING(os) && NAME_IS_INACTIVE_OR_DEACTIVATING(ns))
retroactively_stop_dependencies(n);
-
- return 0;
}
diff --git a/name.h b/name.h
index 33baea1b5d..8c526baed6 100644
--- a/name.h
+++ b/name.h
@@ -189,6 +189,6 @@ int name_start(Name *n);
int name_stop(Name *n);
int name_reload(Name *n);
-int name_notify(Name *n, NameActiveState old, NameActiveState new);
+void name_notify(Name *n, NameActiveState os, NameActiveState ns);
#endif
diff --git a/socket-util.c b/socket-util.c
index 525c33474d..1024ecbed3 100644
--- a/socket-util.c
+++ b/socket-util.c
@@ -115,13 +115,13 @@ int socket_address_parse(SocketAddress *a, const char *s) {
idx = if_nametoindex(n);
free(n);
- if (n == 0)
+ if (idx == 0)
return -EINVAL;
a->sockaddr.in6.sin6_family = AF_INET6;
a->sockaddr.in6.sin6_port = htons((uint16_t) u);
a->sockaddr.in6.sin6_scope_id = idx;
- memcpy(&a->sockaddr.in6.sin6_addr, &in6addr_any, INET6_ADDRSTRLEN);
+ a->sockaddr.in6.sin6_addr = in6addr_any;
a->size = sizeof(struct sockaddr_in6);
}
} else {
@@ -135,7 +135,7 @@ int socket_address_parse(SocketAddress *a, const char *s) {
a->sockaddr.in6.sin6_family = AF_INET6;
a->sockaddr.in6.sin6_port = htons((uint16_t) u);
- memcpy(&a->sockaddr.in6.sin6_addr, &in6addr_any, INET6_ADDRSTRLEN);
+ a->sockaddr.in6.sin6_addr = in6addr_any;
a->size = sizeof(struct sockaddr_in6);
}
}
@@ -274,9 +274,10 @@ int socket_address_print(const SocketAddress *a, char **p) {
}
}
-int socket_address_listen(const SocketAddress *a, int backlog, SocketAddressBindIPv6Only only) {
+int socket_address_listen(const SocketAddress *a, int backlog, SocketAddressBindIPv6Only only, int *ret) {
int r, fd;
assert(a);
+ assert(ret);
if ((r = socket_address_verify(a)) < 0)
return r;
@@ -304,5 +305,6 @@ int socket_address_listen(const SocketAddress *a, int backlog, SocketAddressBind
return -errno;
}
+ *ret = fd;
return 0;
}
diff --git a/socket-util.h b/socket-util.h
index bea3a8931b..1f939ae2d5 100644
--- a/socket-util.h
+++ b/socket-util.h
@@ -38,6 +38,6 @@ typedef enum SocketAddressBindIPv6Only {
int socket_address_parse(SocketAddress *a, const char *s);
int socket_address_print(const SocketAddress *a, char **p);
int socket_address_verify(const SocketAddress *a);
-int socket_address_listen(const SocketAddress *a, int backlog, SocketAddressBindIPv6Only only);
+int socket_address_listen(const SocketAddress *a, int backlog, SocketAddressBindIPv6Only only, int *ret);
#endif
diff --git a/socket.c b/socket.c
index 9b694872bd..ed14db55f9 100644
--- a/socket.c
+++ b/socket.c
@@ -1,7 +1,25 @@
/*-*- Mode: C; c-basic-offset: 8 -*-*/
+#include <sys/types.h>
+#include <sys/stat.h>
+#include <unistd.h>
+#include <errno.h>
+#include <fcntl.h>
+
#include "name.h"
#include "socket.h"
+#include "log.h"
+
+static const NameActiveState state_table[_SOCKET_STATE_MAX] = {
+ [SOCKET_DEAD] = NAME_INACTIVE,
+ [SOCKET_START_PRE] = NAME_ACTIVATING,
+ [SOCKET_START_POST] = NAME_ACTIVATING,
+ [SOCKET_LISTENING] = NAME_ACTIVE,
+ [SOCKET_RUNNING] = NAME_ACTIVE,
+ [SOCKET_STOP_PRE] = NAME_DEACTIVATING,
+ [SOCKET_STOP_POST] = NAME_DEACTIVATING,
+ [SOCKET_MAINTAINANCE] = NAME_INACTIVE,
+};
static int socket_load(Name *n) {
Socket *s = SOCKET(n);
@@ -87,28 +105,130 @@ static void socket_dump(Name *n, FILE *f, const char *prefix) {
}
}
+static void socket_set_state(Socket *s, SocketState state) {
+ SocketState old_state;
+ assert(s);
+
+ old_state = s->state;
+ s->state = state;
+
+ name_notify(NAME(s), state_table[old_state], state_table[s->state]);
+}
+
+static void close_fds(Socket *s) {
+ SocketPort *p;
+
+ assert(s);
+
+ LIST_FOREACH(p, s->ports) {
+ if (p->fd < 0)
+ continue;
+
+ close_nointr(p->fd);
+ p->fd = -1;
+ }
+}
+
static int socket_start(Name *n) {
+ Socket *s = SOCKET(n);
+ SocketPort *p;
+ int r;
+
+ assert(s);
+
+ if (s->state == SOCKET_START_PRE ||
+ s->state == SOCKET_START_POST)
+ return 0;
+
+ if (s->state == SOCKET_LISTENING ||
+ s->state == SOCKET_RUNNING)
+ return -EALREADY;
+
+ if (s->state == SOCKET_STOP_PRE ||
+ s->state == SOCKET_STOP_POST)
+ return -EAGAIN;
+
+ assert(s->state == SOCKET_DEAD || s->state == SOCKET_MAINTAINANCE);
+
+ LIST_FOREACH(p, s->ports) {
+
+ assert(p->fd < 0);
+
+ if (p->type == SOCKET_SOCKET) {
+
+ if ((r = socket_address_listen(&p->address, s->backlog, s->bind_ipv6_only, &p->fd)) < 0)
+ goto rollback;
+
+ } else {
+ struct stat st;
+ assert(p->type == SOCKET_FIFO);
+
+ if (mkfifo(p->path, 0666 & ~s->exec_context.umask) < 0 && errno != EEXIST) {
+ r = -errno;
+ goto rollback;
+ }
+
+ if ((p->fd = open(p->path, O_RDWR|O_CLOEXEC|O_NOCTTY|O_NONBLOCK|O_NOFOLLOW)) < 0) {
+ r = -errno;
+ goto rollback;
+ }
+
+ if (fstat(p->fd, &st) < 0) {
+ r = -errno;
+ goto rollback;
+ }
+
+ /* FIXME verify user, access mode */
+
+ if (!S_ISFIFO(st.st_mode)) {
+ r = -EEXIST;
+ goto rollback;
+ }
+ }
+ }
+
+ socket_set_state(s, SOCKET_LISTENING);
+
return 0;
+
+rollback:
+ close_fds(s);
+
+ socket_set_state(s, SOCKET_MAINTAINANCE);
+
+ return r;
}
static int socket_stop(Name *n) {
+ Socket *s = SOCKET(n);
+
+ assert(s);
+
+ if (s->state == SOCKET_START_PRE ||
+ s->state == SOCKET_START_POST)
+ return -EAGAIN;
+
+ if (s->state == SOCKET_DEAD ||
+ s->state == SOCKET_MAINTAINANCE)
+ return -EALREADY;
+
+ if (s->state == SOCKET_STOP_PRE ||
+ s->state == SOCKET_STOP_POST)
+ return 0;
+
+ assert(s->state == SOCKET_LISTENING || s->state == SOCKET_RUNNING);
+
+ close_fds(s);
+
+ socket_set_state(s, SOCKET_DEAD);
+
return 0;
}
static NameActiveState socket_active_state(Name *n) {
+ assert(n);
- static const NameActiveState table[_SOCKET_STATE_MAX] = {
- [SOCKET_DEAD] = NAME_INACTIVE,
- [SOCKET_START_PRE] = NAME_ACTIVATING,
- [SOCKET_START_POST] = NAME_ACTIVATING,
- [SOCKET_LISTENING] = NAME_ACTIVE,
- [SOCKET_RUNNING] = NAME_ACTIVE,
- [SOCKET_STOP_PRE] = NAME_DEACTIVATING,
- [SOCKET_STOP_POST] = NAME_DEACTIVATING,
- [SOCKET_MAINTAINANCE] = NAME_INACTIVE,
- };
-
- return table[SOCKET(n)->state];
+ return state_table[SOCKET(n)->state];
}
static void socket_free_hook(Name *n) {
diff --git a/test1/postfix.socket b/test1/postfix.socket
index 81c2b07a30..1baae5b566 100644
--- a/test1/postfix.socket
+++ b/test1/postfix.socket
@@ -2,5 +2,5 @@
Description=Postfix SMTP Socket
[Socket]
-ListenStream=25
-ListenFIFO=/dev/test
+ListenStream=53333
+ListenFIFO=/tmp/systemd-fifo
diff --git a/test1/syslog.socket b/test1/syslog.socket
index 73a5662019..8d7baa5a00 100644
--- a/test1/syslog.socket
+++ b/test1/syslog.socket
@@ -2,5 +2,5 @@
Description=Syslog Socket
[Socket]
-ListenDatagram=/dev/log
-ListenStream=eth0:4711
+ListenDatagram=/tmp/systemd-socket
+ListenStream=eth0:3456