From: Lennart Poettering Date: Wed, 1 Sep 2010 01:35:04 +0000 (+0200) Subject: unit: unify some code X-Git-Tag: v9~11 X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=18ffdfda48c327f56cac0aabdc6bada91cc9b7df;p=systemd unit: unify some code --- diff --git a/src/manager.c b/src/manager.c index ff1c70b4..e2f13b20 100644 --- a/src/manager.c +++ b/src/manager.c @@ -2591,17 +2591,10 @@ bool manager_unit_pending_inactive(Manager *m, const char *name) { assert(name); /* Returns true if the unit is inactive or going down */ - if (!(u = manager_get_unit(m, name))) return true; - if (UNIT_IS_INACTIVE_OR_DEACTIVATING(unit_active_state(u))) - return true; - - if (u->meta.job && u->meta.job->type == JOB_STOP) - return true; - - return false; + return unit_pending_inactive(u); } static const char* const manager_running_as_table[_MANAGER_RUNNING_AS_MAX] = { diff --git a/src/socket.c b/src/socket.c index fd975fd9..34068b91 100644 --- a/src/socket.c +++ b/src/socket.c @@ -1167,7 +1167,7 @@ static void socket_enter_running(Socket *s, int cfd) { /* We don't take connections anymore if we are supposed to * shut down anyway */ - if (s->meta.job && s->meta.job->type == JOB_STOP) { + if (unit_pending_inactive(UNIT(s))) { if (cfd >= 0) close_nointr_nofail(cfd); else { diff --git a/src/unit.c b/src/unit.c index bca4d979..d5ed5e14 100644 --- a/src/unit.c +++ b/src/unit.c @@ -2125,6 +2125,20 @@ Unit *unit_following(Unit *u) { return NULL; } +bool unit_pending_inactive(Unit *u) { + assert(u); + + /* Returns true if the unit is inactive or going down */ + + if (UNIT_IS_INACTIVE_OR_DEACTIVATING(unit_active_state(u))) + return true; + + if (u->meta.job && u->meta.job->type == JOB_STOP) + return true; + + return false; +} + static const char* const unit_load_state_table[_UNIT_LOAD_STATE_MAX] = { [UNIT_STUB] = "stub", [UNIT_LOADED] = "loaded", diff --git a/src/unit.h b/src/unit.h index 15e7fedd..c85d968b 100644 --- a/src/unit.h +++ b/src/unit.h @@ -499,6 +499,8 @@ void unit_reset_failed(Unit *u); Unit *unit_following(Unit *u); +bool unit_pending_inactive(Unit *u); + const char *unit_load_state_to_string(UnitLoadState i); UnitLoadState unit_load_state_from_string(const char *s);