diff options
| author | Thomas Haller <thaller@redhat.com> | 2013-11-13 18:24:42 +0100 |
|---|---|---|
| committer | Thomas Haller <thaller@redhat.com> | 2013-11-13 18:43:18 +0100 |
| commit | 709bc90b6bf163d07245669451cc5d97e32bda7f (patch) | |
| tree | 78780adeaf2b73e264b00dc7a3b7ce4c5aacb8e5 /src/devices/nm-device-vlan.c | |
| parent | 97935382f4aca80b8f952ea9fe3ce205253758b7 (diff) | |
| download | NetworkManager-709bc90b6bf163d07245669451cc5d97e32bda7f.tar.gz | |
coverity: fix various warnings detected with Coverity (fixup)
I missed to implement the remarks from
https://bugzilla.redhat.com/show_bug.cgi?id=1025894#c4
Signed-off-by: Thomas Haller <thaller@redhat.com>
Diffstat (limited to 'src/devices/nm-device-vlan.c')
| -rw-r--r-- | src/devices/nm-device-vlan.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/devices/nm-device-vlan.c b/src/devices/nm-device-vlan.c index 0cf2800b73..f64fd87eb7 100644 --- a/src/devices/nm-device-vlan.c +++ b/src/devices/nm-device-vlan.c @@ -318,7 +318,12 @@ update_connection (NMDevice *device, NMConnection *connection) g_object_set (s_vlan, NM_SETTING_VLAN_INTERFACE_NAME, nm_device_get_iface (device), NULL); } - (void) nm_platform_vlan_get_info (ifindex, &parent_ifindex, &vlan_id); + if (!nm_platform_vlan_get_info (ifindex, &parent_ifindex, &vlan_id)) { + nm_log_warn (LOGD_VLAN, "(%s): failed to get VLAN interface info while updating connection.", + nm_device_get_iface (device)); + return; + } + if (priv->vlan_id != vlan_id) { priv->vlan_id = vlan_id; g_object_notify (G_OBJECT (device), NM_DEVICE_VLAN_ID); |