libosmocore.git branch master updated. 0.10.2-33-g19ec7b94

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "An utility library for Open Source Mobile Communications". The branch, master has been updated via 19ec7b948322bbc9457a2b22219c93558a6f931e (commit) from b4718fd233a00a950cef1965d13afccd6c6c0e77 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://cgit.osmocom.org/libosmocore/commit/?id=19ec7b948322bbc9457a2b22219c… commit 19ec7b948322bbc9457a2b22219c93558a6f931e Author: Neels Hofmeyr <neels(a)hofmeyr.de> Date: Sat Nov 18 23:10:24 2017 +0100 fsm_tmr_cb: don't set T=0, the fi may no longer exist When calling the timer_cb, that may have effected an fi termination and deallocation, e.g. from dispatching events and/or complex choices made. Current timer_cb implementations expect T to reflect the fired timer number, so we can't actually set T=0 before calling the timer_cb. Instead, never reset T to zero, let it always reflect the timer that last fired. When a new timer starts, T will be set to its new value. Adding a T arg to the timer_cb() would have been the cleanest solution, so that fi->T can be set to zero before dispatching the timer_cb. But since we've already rolled out this FSM API, we should stay backwards compatible. In the case where the timer returned 1 to request termination, we can assume that the fi still exists, but to be consistent, don't set T = 0 in that code path either. Change-Id: I18626b55a1491098b3ed602df1b331f08d25625a ----------------------------------------------------------------------- Summary of changes: src/fsm.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) hooks/post-receive -- An utility library for Open Source Mobile Communications