diff options
author | Pavel Šimerda <pavlix@pavlix.net> | 2018-01-17 17:39:24 +0100 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2018-01-17 19:20:03 +0100 |
commit | c4f655579c0d6bdb4b61a80c49071e81fd035089 (patch) | |
tree | 6aa4d35e780998173f81879a42bd23b61c58db05 | |
parent | f2aa286fa92a8fa6c96d8b7db09e6fb5519bea13 (diff) | |
download | NetworkManager-c4f655579c0d6bdb4b61a80c49071e81fd035089.tar.gz |
libnm: fix empty statements that were supposed to return NULL
Discovered thanks to `-Wunused-value` when building on openSUSE Leap 42.3.
-rw-r--r-- | libnm/nm-client.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/libnm/nm-client.c b/libnm/nm-client.c index 0cf4d4361c..6936c53fe9 100644 --- a/libnm/nm-client.c +++ b/libnm/nm-client.c @@ -1410,7 +1410,7 @@ nm_client_get_connection_by_id (NMClient *client, const char *id) g_return_val_if_fail (id != NULL, NULL); if (!nm_client_get_nm_running (client)) - NULL; + return NULL; return nm_remote_settings_get_connection_by_id (NM_CLIENT_GET_PRIVATE (client)->settings, id); } @@ -1435,7 +1435,7 @@ nm_client_get_connection_by_path (NMClient *client, const char *path) g_return_val_if_fail (path != NULL, NULL); if (!nm_client_get_nm_running (client)) - NULL; + return NULL; return nm_remote_settings_get_connection_by_path (NM_CLIENT_GET_PRIVATE (client)->settings, path); } @@ -1460,7 +1460,7 @@ nm_client_get_connection_by_uuid (NMClient *client, const char *uuid) g_return_val_if_fail (uuid != NULL, NULL); if (!nm_client_get_nm_running (client)) - NULL; + return NULL; return nm_remote_settings_get_connection_by_uuid (NM_CLIENT_GET_PRIVATE (client)->settings, uuid); } |