diff options
author | Beniamino Galvani <bgalvani@redhat.com> | 2016-02-22 10:01:30 +0100 |
---|---|---|
committer | Beniamino Galvani <bgalvani@redhat.com> | 2016-03-04 14:27:09 +0100 |
commit | f87d25f7a1cce12e9906f2fa74b3c8b2a24b8da2 (patch) | |
tree | 77c43c934b75d66f64f56d398bae38d1edbedb76 /src | |
parent | 4090716d96180caaeb807033e0c9a3b4a3092f9b (diff) | |
download | NetworkManager-f87d25f7a1cce12e9906f2fa74b3c8b2a24b8da2.tar.gz |
policy: simplify set_system_hostname()
Move some conditionals to the beginning of the function.
Diffstat (limited to 'src')
-rw-r--r-- | src/nm-policy.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/src/nm-policy.c b/src/nm-policy.c index c564ff0a52..c331df2dfe 100644 --- a/src/nm-policy.c +++ b/src/nm-policy.c @@ -157,8 +157,13 @@ set_system_hostname (NMPolicy *self, const char *new_hostname, const char *msg) const char *name; int ret; - if (new_hostname) - g_warn_if_fail (strlen (new_hostname)); + if (!new_hostname) + name = FALLBACK_HOSTNAME4; + else if (!new_hostname[0]) { + g_warn_if_reached (); + name = FALLBACK_HOSTNAME4; + } else + name = new_hostname; old_hostname[HOST_NAME_MAX] = '\0'; errno = 0; @@ -168,12 +173,10 @@ set_system_hostname (NMPolicy *self, const char *new_hostname, const char *msg) errno, strerror (errno)); } else { /* Don't set the hostname if it isn't actually changing */ - if ( (new_hostname && !strcmp (old_hostname, new_hostname)) - || (!new_hostname && !strcmp (old_hostname, FALLBACK_HOSTNAME4))) + if (nm_streq (name, old_hostname)) return; } - name = (new_hostname && strlen (new_hostname)) ? new_hostname : FALLBACK_HOSTNAME4; _LOGI (LOGD_DNS, "setting system hostname to '%s' (%s)", name, msg); nm_settings_set_transient_hostname (priv->settings, name, |