On 10/27/2011 01:19 PM, [email protected] wrote: > From: Patrik Flykt <[email protected]> > > deselect_service() and select_offline_service() want to clear > up the session state machinery by setting up a zero second > timeout function thereby letting the main loop run once through > all waiting actions. Unfortunately, if a service is being > removed meantime, the service structure stored in the timeout > functions will become invalid. > > This is fixed by adding a list of pending connect/disconnect > actions per struct service_entry and cancelling all of them > if the service is removed. > > Fixes BMC#22879
Obvious bug. I should have seen this when I wrote this code. Your patch looks good. Unfortunatly, on my home F14 system g_slist_free_full() not available (glib 2.26). Okay, that is quite old and soon deprecated. Though I fear debian system also ship still 2.26. Maybe you should use an open coded version as we do in other places too. cheers, daniel _______________________________________________ connman mailing list [email protected] http://lists.connman.net/listinfo/connman
