Hi, A small patch to avoid ConnMan crash, NULL pointer reference. When dbus removes timer and calls dbus_timeout_set_data, timeout_handler_free didn't check whether its handler is NULL or not. Following backtrace of ConnMan crash is included:
#0 timeout_handler_free (data=0x0) at gdbus/mainloop.c:203 #1 0x42272654 in dbus_timeout_set_data (timeout=0x76f28, data=0x0, free_data_function=0) at dbus-timeout.c:451 #2 0x422726cc in _dbus_timeout_list_remove_timeout (timeout_list=0x6cab0, timeout=0x76f28) at dbus-timeout.c:347 Please review, From: Jeongseok Seo <[email protected]> Date: Wed, 7 Dec 2011 22:18:10 +0900 Subject: [PATCH] gdbus: Check NULL pointer Add NULL check to avoid ConnMan crash --- gdbus/mainloop.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) ----------------------------------------------- Danny Jeongseok Seo Senior Engineer Platform R&D Team Mobile & Communication Division Telecommunication Network SAMSUNG Electronics Co., Ltd. Phone: +82-31-301-4561 E-mail: [email protected] _______________________________________________ connman mailing list [email protected] http://lists.connman.net/listinfo/connman
