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

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/osmocom-commitlog@lists.osmocom.org/.

gitosis at osmocom.org gitosis at osmocom.org
Sat Nov 25 17:45:22 UTC 2017


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=19ec7b948322bbc9457a2b22219c93558a6f931e

commit 19ec7b948322bbc9457a2b22219c93558a6f931e
Author: Neels Hofmeyr <neels at 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



More information about the osmocom-commitlog mailing list