system/NetworkManager
system/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager synced 2024-10-15 04:24:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

ifcfg-rh: fix signature of link_changed() callback

Browse source 
Depending on how arguments are passed to the called function,
this could lead to a crash.

Maybe not on 32 bit machines where the size of the pointer is
the size of an int.

Maybe not on x86_64, where the arguments are passed in registers.

Fixes: b88c309167
This commit is contained in:
Thomas Haller 2016-10-22 13:46:02 +02:00
parent 9f11610606
commit 548a5440e9
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/settings/plugins/ifcfg-rh/nms-ifcfg-rh-connection.c
View file

@ -116,7 +116,7 @@ devtimeout_ready (gpointer user_data)
}
static void
link_changed (NMPlatform *platform, NMPObjectType *obj_type, int ifindex, const NMPlatformLink *link,
link_changed (NMPlatform *platform, NMPObjectType obj_type, int ifindex, const NMPlatformLink *link,
NMPlatformSignalChangeType change_type,
NMConnection *self)
{
@ -127,7 +127,6 @@ link_changed (NMPlatform *platform, NMPObjectType *obj_type, int ifindex, const
if (g_strcmp0 (link->name, ifname) != 0)
return;
/* Shouldn't happen, but... */
if (change_type == NM_PLATFORM_SIGNAL_REMOVED)
return;