mirror of
https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git
synced 2026-05-05 11:08:03 +02:00
vpn-plugin-old,dispatcher: cast unchecked g_variant_lookup() calls to void
This is done to silence coverity. In the dispatcher the existence of the key is checked before and we're fine with leaving the value untouched in the vpn-plugin-old.
This commit is contained in:
parent
da612acc6a
commit
a9996c4f1d
2 changed files with 7 additions and 7 deletions
|
|
@ -388,7 +388,7 @@ nm_dispatcher_utils_construct_envp (const char *action,
|
|||
return NULL;
|
||||
}
|
||||
g_variant_unref (value);
|
||||
g_variant_lookup (device_props, NMD_DEVICE_PROPS_IP_INTERFACE, "&s", &ip_iface);
|
||||
(void) g_variant_lookup (device_props, NMD_DEVICE_PROPS_IP_INTERFACE, "&s", &ip_iface);
|
||||
}
|
||||
|
||||
/* Device type */
|
||||
|
|
|
|||
|
|
@ -280,8 +280,8 @@ nm_vpn_plugin_old_set_config (NMVpnPluginOld *plugin,
|
|||
|
||||
priv->got_config = TRUE;
|
||||
|
||||
g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_HAS_IP4, "b", &priv->has_ip4);
|
||||
g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_HAS_IP6, "b", &priv->has_ip6);
|
||||
(void) g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_HAS_IP4, "b", &priv->has_ip4);
|
||||
(void) g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_HAS_IP6, "b", &priv->has_ip6);
|
||||
|
||||
g_warn_if_fail (priv->has_ip4 || priv->has_ip6);
|
||||
|
||||
|
|
@ -289,13 +289,13 @@ nm_vpn_plugin_old_set_config (NMVpnPluginOld *plugin,
|
|||
* ip4config, for compatibility with older daemons.
|
||||
*/
|
||||
g_clear_pointer (&priv->banner, g_free);
|
||||
g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_BANNER, "&s", &priv->banner);
|
||||
(void) g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_BANNER, "&s", &priv->banner);
|
||||
g_clear_pointer (&priv->tundev, g_free);
|
||||
g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_TUNDEV, "&s", &priv->tundev);
|
||||
(void) g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_TUNDEV, "&s", &priv->tundev);
|
||||
g_clear_pointer (&priv->gateway, g_free);
|
||||
g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_EXT_GATEWAY, "&s", &priv->gateway);
|
||||
(void) g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_EXT_GATEWAY, "&s", &priv->gateway);
|
||||
g_clear_pointer (&priv->mtu, g_free);
|
||||
g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_MTU, "&s", &priv->mtu);
|
||||
(void) g_variant_lookup (config, NM_VPN_PLUGIN_CONFIG_MTU, "&s", &priv->mtu);
|
||||
|
||||
g_signal_emit (plugin, signals[CONFIG], 0, config);
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue