diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-04-25 10:41:59 +0200 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-04-25 11:13:39 +0200 |
| commit | 4232cf049f5f41f47f2b8e8ad6ca8c66bf7c4f51 (patch) | |
| tree | 154e946ca821b2687ee3231d31c76996a634000f | |
| parent | c5322608a5b78d55381fc87985304ec515cd723d (diff) | |
| download | systemd-4232cf049f5f41f47f2b8e8ad6ca8c66bf7c4f51.tar.gz | |
network: drop "return 1" when the return value is ignored by all callers
The reader is tricked into thinking that this has some meaning...
| -rw-r--r-- | src/network/networkd-link.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/network/networkd-link.c b/src/network/networkd-link.c index d3928e2862..ec2190fd13 100644 --- a/src/network/networkd-link.c +++ b/src/network/networkd-link.c @@ -3365,7 +3365,7 @@ static int link_initialized_and_synced(Link *link) { /* We may get called either from the asynchronous netlink callback, * or directly for link_add() if running in a container. See link_add(). */ if (!IN_SET(link->state, LINK_STATE_PENDING, LINK_STATE_INITIALIZED)) - return 1; + return 0; log_link_debug(link, "Link state is up-to-date"); link_set_state(link, LINK_STATE_INITIALIZED); @@ -3383,7 +3383,7 @@ static int link_initialized_and_synced(Link *link) { &link->mac, &network); if (r == -ENOENT) { link_enter_unmanaged(link); - return 1; + return 0; } else if (r == 0 && network->unmanaged) { link_enter_unmanaged(link); return 0; @@ -3420,7 +3420,7 @@ static int link_initialized_and_synced(Link *link) { if (r < 0) return r; - return 1; + return 0; } static int link_initialized_handler(sd_netlink *rtnl, sd_netlink_message *m, Link *link) { |