In case bluetoothd crashes randomly the data pointer can be NULL. So we shouldn't derefernce it. --- plugins/hfp.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/plugins/hfp.c b/plugins/hfp.c index 12d3457..acb297f 100644 --- a/plugins/hfp.c +++ b/plugins/hfp.c @@ -569,6 +569,9 @@ static void hfp_connect_reply(DBusPendingCall *call, gpointer user_data) DBG("Connect reply: %s", derr.message); + if (data == NULL) + goto poweroff; + if (dbus_error_has_name(&derr, DBUS_ERROR_NO_REPLY)) { msg = dbus_message_new_method_call(BLUEZ_SERVICE, data->handsfree_path, @@ -579,6 +582,7 @@ static void hfp_connect_reply(DBusPendingCall *call, gpointer user_data) g_dbus_send_message(connection, msg); } +poweroff: ofono_modem_set_powered(modem, FALSE); dbus_error_free(&derr); -- 1.7.3.3 _______________________________________________ ofono mailing list ofono@ofono.org http://lists.ofono.org/listinfo/ofono