fdo-mirrors/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git synced 2026-02-14 16:50:33 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

libnm: fix leak with self assignment in nm_connection_add_setting()

Browse source 
We must consume the reference, like we would in the other case.

Interestingly, I am unable to reproduce a case where valgrind would
complain about the leak. But it is there nonetheless.

Fixes: 0a22f4e490 ('libnm: refactor tracking of NMSetting in NMConnection')
This commit is contained in:
Thomas Haller 2022-11-16 19:10:56 +01:00
parent 3b2eb689f3
commit 3e3b629586
No known key found for this signature in database
GPG key ID: 29C2366E4DFC5728
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/libnm-core-impl/nm-connection.c
View file

@ -202,8 +202,10 @@ _nm_connection_add_setting(NMConnection *connection, NMSetting *setting)
priv = NM_CONNECTION_GET_PRIVATE(connection);
s_old = priv->settings[setting_info->meta_type];
if (s_old == setting)
if (s_old == setting) {
g_object_unref(s_old);
return;
}
priv->settings[setting_info->meta_type] = setting;