summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Haller <thaller@redhat.com>2013-10-22 17:11:24 +0200
committerThomas Haller <thaller@redhat.com>2013-10-22 18:45:22 +0200
commitc67f978df09cbf25c3c9e2fae6003f291c7aa184 (patch)
tree1dabd37bbed87864a44f1c717b55500e9073194b
parentba781ced5701922c7e8f05e5b451a0b464977d88 (diff)
downloadNetworkManager-c67f978df09cbf25c3c9e2fae6003f291c7aa184.tar.gz
core: fix compiler warnings -Werror=shadow by trivial renaming of variables
https://bugzilla.gnome.org/show_bug.cgi?id=710497 Signed-off-by: Thomas Haller <thaller@redhat.com>
-rw-r--r--src/nm-dbus-manager.c6
-rw-r--r--src/nm-manager.c46
-rw-r--r--src/platform/nm-linux-platform.c18
3 files changed, 35 insertions, 35 deletions
diff --git a/src/nm-dbus-manager.c b/src/nm-dbus-manager.c
index 2c219861e8..ca2182d325 100644
--- a/src/nm-dbus-manager.c
+++ b/src/nm-dbus-manager.c
@@ -787,7 +787,7 @@ nm_dbus_manager_register_exported_type (NMDBusManager *self,
GType object_type,
const DBusGObjectInfo *info)
{
- const char *properties_info, *dbus_name, *gobject_name, *access;
+ const char *properties_info, *dbus_name, *gobject_name, *tmp_access;
dbus_g_object_type_install_info (object_type, info);
if (!info->exported_properties)
@@ -798,8 +798,8 @@ nm_dbus_manager_register_exported_type (NMDBusManager *self,
/* The format is: "interface\0DBusPropertyName\0gobject_property_name\0access\0" */
dbus_name = strchr (properties_info, '\0') + 1;
gobject_name = strchr (dbus_name, '\0') + 1;
- access = strchr (gobject_name, '\0') + 1;
- properties_info = strchr (access, '\0') + 1;
+ tmp_access = strchr (gobject_name, '\0') + 1;
+ properties_info = strchr (tmp_access, '\0') + 1;
/* Note that nm-properties-changed-signal takes advantage of the
* fact that @dbus_name and @gobject_name are static data that won't
diff --git a/src/nm-manager.c b/src/nm-manager.c
index 810f5a15f0..8427c20589 100644
--- a/src/nm-manager.c
+++ b/src/nm-manager.c
@@ -174,7 +174,7 @@ static void rfkill_change_wifi (const char *desc, gboolean enabled);
static void
platform_link_added_cb (NMPlatform *platform,
int ifindex,
- NMPlatformLink *link,
+ NMPlatformLink *plink,
NMPlatformReason reason,
gpointer user_data);
@@ -2401,7 +2401,7 @@ load_device_factories (NMManager *self)
static void
platform_link_added_cb (NMPlatform *platform,
int ifindex,
- NMPlatformLink *link,
+ NMPlatformLink *plink,
NMPlatformReason reason,
gpointer user_data)
{
@@ -2421,7 +2421,7 @@ platform_link_added_cb (NMPlatform *platform,
NMDeviceFactoryCreateFunc create_func = iter->data;
g_clear_error (&error);
- device = (NMDevice *) create_func (link, &error);
+ device = (NMDevice *) create_func (plink, &error);
if (device && NM_IS_DEVICE (device)) {
g_assert_no_error (error);
break; /* success! */
@@ -2429,7 +2429,7 @@ platform_link_added_cb (NMPlatform *platform,
if (error) {
nm_log_warn (LOGD_HW, "%s: factory failed to create device: (%d) %s",
- link->udi,
+ plink->udi,
error ? error->code : -1,
error ? error->message : "(unknown)");
g_clear_error (&error);
@@ -2441,63 +2441,63 @@ platform_link_added_cb (NMPlatform *platform,
int parent_ifindex = -1;
NMDevice *parent;
- switch (link->type) {
+ switch (plink->type) {
case NM_LINK_TYPE_ETHERNET:
- device = nm_device_ethernet_new (link);
+ device = nm_device_ethernet_new (plink);
break;
case NM_LINK_TYPE_INFINIBAND:
- device = nm_device_infiniband_new (link);
+ device = nm_device_infiniband_new (plink);
break;
case NM_LINK_TYPE_OLPC_MESH:
- device = nm_device_olpc_mesh_new (link);
+ device = nm_device_olpc_mesh_new (plink);
break;
case NM_LINK_TYPE_WIFI:
- device = nm_device_wifi_new (link);
+ device = nm_device_wifi_new (plink);
break;
case NM_LINK_TYPE_BOND:
- device = nm_device_bond_new (link);
+ device = nm_device_bond_new (plink);
break;
case NM_LINK_TYPE_TEAM:
- device = nm_device_team_new (link);
+ device = nm_device_team_new (plink);
break;
case NM_LINK_TYPE_BRIDGE:
/* FIXME: always create device when we handle bridges non-destructively */
- if (bridge_created_by_nm (self, link->name))
- device = nm_device_bridge_new (link);
+ if (bridge_created_by_nm (self, plink->name))
+ device = nm_device_bridge_new (plink);
else
- nm_log_info (LOGD_BRIDGE, "(%s): ignoring bridge not created by NetworkManager", link->name);
+ nm_log_info (LOGD_BRIDGE, "(%s): ignoring bridge not created by NetworkManager", plink->name);
break;
case NM_LINK_TYPE_VLAN:
/* Have to find the parent device */
if (nm_platform_vlan_get_info (ifindex, &parent_ifindex, NULL)) {
parent = find_device_by_ifindex (self, parent_ifindex);
if (parent)
- device = nm_device_vlan_new (link, parent);
+ device = nm_device_vlan_new (plink, parent);
else {
/* If udev signaled the VLAN interface before it signaled
* the VLAN's parent at startup we may not know about the
* parent device yet. But we'll find it on the second pass
* from nm_manager_start().
*/
- nm_log_dbg (LOGD_HW, "(%s): VLAN parent interface unknown", link->name);
+ nm_log_dbg (LOGD_HW, "(%s): VLAN parent interface unknown", plink->name);
}
} else
- nm_log_err (LOGD_HW, "(%s): failed to get VLAN parent ifindex", link->name);
+ nm_log_err (LOGD_HW, "(%s): failed to get VLAN parent ifindex", plink->name);
break;
case NM_LINK_TYPE_VETH:
- device = nm_device_veth_new (link);
+ device = nm_device_veth_new (plink);
break;
case NM_LINK_TYPE_TUN:
case NM_LINK_TYPE_TAP:
- device = nm_device_tun_new (link);
+ device = nm_device_tun_new (plink);
break;
case NM_LINK_TYPE_MACVLAN:
case NM_LINK_TYPE_MACVTAP:
- device = nm_device_macvlan_new (link);
+ device = nm_device_macvlan_new (plink);
break;
case NM_LINK_TYPE_GRE:
case NM_LINK_TYPE_GRETAP:
- device = nm_device_gre_new (link);
+ device = nm_device_gre_new (plink);
break;
case NM_LINK_TYPE_WWAN_ETHERNET:
@@ -2513,7 +2513,7 @@ platform_link_added_cb (NMPlatform *platform,
break;
default:
- device = nm_device_generic_new (link);
+ device = nm_device_generic_new (plink);
break;
}
}
@@ -2525,7 +2525,7 @@ platform_link_added_cb (NMPlatform *platform,
static void
platform_link_removed_cb (NMPlatform *platform,
int ifindex,
- NMPlatformLink *link,
+ NMPlatformLink *plink,
NMPlatformReason reason,
gpointer user_data)
{
diff --git a/src/platform/nm-linux-platform.c b/src/platform/nm-linux-platform.c
index 44304bbeeb..821a8cd911 100644
--- a/src/platform/nm-linux-platform.c
+++ b/src/platform/nm-linux-platform.c
@@ -505,7 +505,7 @@ link_type_from_udev (NMPlatform *platform, int ifindex, int arptype, const char
}
static gboolean
-link_is_software (struct rtnl_link *link)
+link_is_software (struct rtnl_link *rtnllink)
{
const char *type;
@@ -514,11 +514,11 @@ link_is_software (struct rtnl_link *link)
* device causes.
*/
- if ( rtnl_link_get_arptype (link) == ARPHRD_INFINIBAND
- && strchr (rtnl_link_get_name (link), '.'))
+ if ( rtnl_link_get_arptype (rtnllink) == ARPHRD_INFINIBAND
+ && strchr (rtnl_link_get_name (rtnllink), '.'))
return TRUE;
- type = rtnl_link_get_type (link);
+ type = rtnl_link_get_type (rtnllink);
if (type == NULL)
return FALSE;
@@ -1646,7 +1646,7 @@ link_supports_vlans (NMPlatform *platform, int ifindex)
auto_nl_object struct rtnl_link *rtnllink = link_get (platform, ifindex);
const char *name = nm_platform_link_get_name (ifindex);
auto_g_free struct ethtool_gfeatures *features = NULL;
- int index, block, bit, size;
+ int idx, block, bit, size;
/* Only ARPHRD_ETHER links can possibly support VLANs. */
if (!rtnllink || rtnl_link_get_arptype (rtnllink) != ARPHRD_ETHER)
@@ -1655,14 +1655,14 @@ link_supports_vlans (NMPlatform *platform, int ifindex)
if (!name)
return FALSE;
- index = ethtool_get_stringset_index (name, ETH_SS_FEATURES, "vlan-challenged");
- if (index == -1) {
+ idx = ethtool_get_stringset_index (name, ETH_SS_FEATURES, "vlan-challenged");
+ if (idx == -1) {
debug ("vlan-challenged ethtool feature does not exist?");
return FALSE;
}
- block = index / 32;
- bit = index % 32;
+ block = idx / 32;
+ bit = idx % 32;
size = block + 1;
features = g_malloc0 (sizeof (*features) + size * sizeof (struct ethtool_get_features_block));