Fix : Null deference issue 87/135787/1
authorSeungyoun Ju <sy39.ju@samsung.com>
Tue, 20 Jun 2017 12:06:59 +0000 (21:06 +0900)
committerInjun Yang <injun.yang@samsung.com>
Mon, 26 Jun 2017 11:28:14 +0000 (20:28 +0900)
[Problem] If remote device is not connected but _bt_le_conn_update is
 called, crash happens.
[Cause & Measure] There is no connected device info. But it is
 deferenced. This patch checks NULL for connected device info.
[Checking Method] Try to update LE connection parameter to non-connected
 BLE device

Change-Id: Id4fe04ff17392585ec544bcd4f22e33ad5906820

bt-service/bt-service-device.c

index df90920..fd1466b 100644 (file)
@@ -2759,6 +2759,11 @@ int _bt_le_conn_update(const char *sender,
        _bt_add_le_conn_param_info(address, sender, interval_min, interval_max, 0, 2000);
 
        dev = __bt_get_le_connected_dev_info(address);
+       if (dev == NULL) {
+               ret = BLUETOOTH_ERROR_NOT_CONNECTED;
+               goto fail;
+       }
+
        if (g_slist_length(dev->senders) == 1)
                goto update;
        else {