diff options
| author | Jacob Erlbeck <jerlbeck@sysmocom.de> | 2015-05-21 16:58:22 +0200 |
|---|---|---|
| committer | Jacob Erlbeck <jerlbeck@sysmocom.de> | 2015-05-27 13:30:01 +0200 |
| commit | 0e50ce614569b9ae7bcc6e7becc4aeec46e8369d (patch) | |
| tree | 71ce7ecfc92c291cd2362036b0d2df343ffb448b /src/gprs_ms.h | |
| parent | 767193e20b4172dfb0e76ec63444115dc5ae8806 (diff) | |
tbf: Always call set_tlli/confirm_tlli in update_ms
Currently the m_tlli member in GprsMs is set by the constructor,
circumventing the TLLI confirmation mechanism.
This commit replaces the get_or_create_ms() method by a create_ms()
method which takes the TLLI and the direction (UL or DL) as
parameters to select either set_tlli() or confirm_tlli(). The MS
object is instantiated with TLLI = 0, and therefore GprsMs::tlli() is
extended to return the DL TLLI if both of the other TLLI are not set.
Note that create_ms() will not check whether an MS object with a
matching TLLI is already stored in the list, so it should only be
called after a corresponding get_ms() in general.
Sponsored-by: On-Waves ehf
Diffstat (limited to 'src/gprs_ms.h')
| -rw-r--r-- | src/gprs_ms.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/gprs_ms.h b/src/gprs_ms.h index ac1928e7..e84ff9e7 100644 --- a/src/gprs_ms.h +++ b/src/gprs_ms.h @@ -89,7 +89,9 @@ private: inline uint32_t GprsMs::tlli() const { - return m_new_ul_tlli ? m_new_ul_tlli : m_tlli; + return m_new_ul_tlli ? m_new_ul_tlli : + m_tlli ? m_tlli : + m_new_dl_tlli; } inline bool GprsMs::check_tlli(uint32_t tlli) |