fdo-mirrors/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git synced 2026-03-27 02:20:45 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

settings: remove an unnecessary check

Browse source 
Coverity gets confused and thinks we are potentially leaking bssid_str
here. Given that nm_utils_hwaddr_ntoa() never returns NULL anyway,
just drop the check.

(cherry picked from commit f4f4ecf159)
This commit is contained in:
Dan Winship 2014-06-12 12:21:59 -04:00 • committed by Dan Williams
parent 45851e0309
commit a429276035
1 changed files with 0 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
src/settings/nm-settings-connection.c
View file

@ -1815,7 +1815,6 @@ nm_settings_connection_add_seen_bssid (NMSettingsConnection *connection,
/* Add the new BSSID; let the hash take ownership of the allocated BSSID string */
bssid_str = nm_utils_hwaddr_ntoa (seen_bssid, ARPHRD_ETHER);
g_return_if_fail (bssid_str != NULL);
g_hash_table_insert (priv->seen_bssids, mac_dup (seen_bssid), bssid_str);
/* Build up a list of all the BSSIDs in string form */