korg/bluez
0
0
Fork 0
mirror of https://git.kernel.org/pub/scm/bluetooth/bluez.git synced 2024-11-15 08:14:28 +08:00
Code Issues Packages Projects Releases Wiki Activity

client: Fix core dump when using set-advertise-name

Browse Source 
If repeating to set on/off with set-advertise-name after setting local
name, and then may dump core by double free. This patch uses g_free()
instead of free().
This commit is contained in:
ERAMOTO Masaya 2017-09-25 13:30:56 +09:00 committed by Luiz Augusto von Dentz
parent 115fd96871
commit 15225b1e0b
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
client/advertising.c
View File

@ -547,8 +547,10 @@ void ad_advertise_name(DBusConnection *conn, bool value)
ad.name = value;
if (!value)
free(ad.local_name);
if (!value) {
g_free(ad.local_name);
ad.local_name = NULL;
}
g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE, "Includes");
}
@ -558,7 +560,7 @@ void ad_advertise_local_name(DBusConnection *conn, const char *name)
if (ad.local_name && !strcmp(name, ad.local_name))
return;
free(ad.local_name);
g_free(ad.local_name);
ad.local_name = strdup(name);
g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE, "LocalName");