openbsc.git branch jerlbeck/wip/gprs-patching updated. 0.13.0-918-g8b77456

gitosis at gitosis at
Fri Jan 30 15:52:56 UTC 2015

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 "The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)".

The branch, jerlbeck/wip/gprs-patching has been updated
  discards  0ee763fe1b99390f23eb62cc86d1848e89cdbeb4 (commit)
  discards  97113dccfc7af33e7443e1086932306718f87554 (commit)
  discards  b2186d39d547280a73ff24cc0a4d657301cd0cc4 (commit)
  discards  367e17ab4fac2388bba6329101fce721fcec28ca (commit)
  discards  5bb5439eaf769bc63159df1875e11da158695f69 (commit)
  discards  5d2620dd4a233dafcf1c8e170a0934521e3ae8ba (commit)
  discards  01a346c7320c9f9eceae8245d75700242cf65151 (commit)
  discards  09283010307a88941f1c47746bbdce29e0e3b783 (commit)
       via  8b77456f406e8215f92150868062edf1122e3aa7 (commit)
       via  9b9c689b7f4895d12030105b804ab5ac20a3b2bd (commit)
       via  16a341029d698b58db0b42a5b454b34fc0fb6589 (commit)
       via  8bd72827e74c49ebb504f31bc2118bfe6bdd65a7 (commit)
       via  b922a3dcb059cdf9aefd1574d74cdf1e88cb1ea2 (commit)
       via  df2627ad89e6ed2d217e960832545e56bc639b35 (commit)
       via  6a3d6fdb84c7e5d91df2c1b340cd28c20fe1c503 (commit)
       via  a16c1ef5660370c88f20df65bc593b63d3bcfcb7 (commit)
       via  6cc717596131558b374520aaa42d9c70decab786 (commit)

This update added new revisions after undoing existing revisions.  That is
to say, the old revision is not a strict subset of the new revision.  This
situation occurs when you --force push a change and generate a repository
containing something like this:

 * -- * -- B -- O -- O -- O (0ee763fe1b99390f23eb62cc86d1848e89cdbeb4)
             N -- N -- N (8b77456f406e8215f92150868062edf1122e3aa7)

When this happens we assume that you've already had alert emails for all
of the O revisions, and so we here report only the revisions in the N
branch from the common base, B.

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 -----------------------------------------------------------------

commit 8b77456f406e8215f92150868062edf1122e3aa7
Author: Jacob Erlbeck <jerlbeck at>
Date:   Tue Sep 30 13:53:26 2014 +0200

    sgsn: Added TODOs (TODO)
      - fix and remove TODOs
      - add test cases for Detach(reattach)
      - add test cases for PDP context deletion (OW#1311)
      - add test cases for Cancel pending timer in sgsn_mm_ctx_free
      - remove this commit
    Sponsored-by: On-Waves ehf

commit 9b9c689b7f4895d12030105b804ab5ac20a3b2bd
Author: Jacob Erlbeck <jerlbeck at>
Date:   Wed Dec 17 14:03:35 2014 +0100

    sgsn: Add PDP info to subscriber data (TODO)
      - commit message
      - raw APN representation ???
      - tests
      - check PDP data list on PDP Ctx Act (other commit)
    Sponsored-by: On-Waves ehf

commit 16a341029d698b58db0b42a5b454b34fc0fb6589
Author: Jacob Erlbeck <jerlbeck at>
Date:   Fri Dec 19 19:11:20 2014 +0100

    sgsn: Add logging to connect_timer_cb (TODO)
      - LOGLevel ?
    Sponsored-by: On-Waves ehf

commit 8bd72827e74c49ebb504f31bc2118bfe6bdd65a7
Author: Jacob Erlbeck <jerlbeck at>
Date:   Fri Jan 30 11:57:25 2015 +0100

    sgsn/test: Fix memory leak in test_subscriber_gsup
    Currently the MM context is not deleted when a GSUP location
    cancellation message is processed, because the real
    sgsn_update_subscriber_data function has been wrapped to a dummy
    This commit adds an explicit call to sgsn_mm_ctx_cleanup_free which
    also unassigns the LLME, so the call to gprs_llgmm_assign is removed.
    It also adds an assertion to check that there are no talloc'ed blocks
    left in tall_bsc_ctx.
    == 372 bytes in 1 blocks are possibly lost in loss record 7 of 9
    ==    at 0x402A17C: malloc (in /usr/lib/valgrind/
    ==    by 0x4059FB8: _talloc_zero (talloc.c:354)
    ==    by 0x8055B82: sgsn_mm_ctx_alloc (gprs_sgsn.c:167)
    ==    by 0x804A336: alloc_mm_ctx (sgsn_test.c:140)
    ==    by 0x804B24D: test_subscriber_gsup (sgsn_test.c:503)
    ==    by 0x804EC99: main (sgsn_test.c:1853)
    Sponsored-by: On-Waves ehf

commit b922a3dcb059cdf9aefd1574d74cdf1e88cb1ea2
Author: Jacob Erlbeck <jerlbeck at>
Date:   Thu Jan 29 14:55:34 2015 +0100

    sgsn/test: Add checks for subscr->error_cause
    This commits adds a few asserts that check the value of
    subscr->error_cause after a GSUP message has been received.
    Sponsored-by: On-Waves ehf

commit df2627ad89e6ed2d217e960832545e56bc639b35
Author: Jacob Erlbeck <jerlbeck at>
Date:   Thu Jan 29 14:17:51 2015 +0100

    gprs: Use 'Network failure' as default cause
    This commit adds a check after a GSUP message has been decoded
    whether it is an error message and does not contain a cause value.
    If his is the case, the cause value is set to 'Network failure', so
    that this cause if effectively the default value for error messages.
    Sponsored-by: On-Waves ehf

commit 6a3d6fdb84c7e5d91df2c1b340cd28c20fe1c503
Author: Jacob Erlbeck <jerlbeck at>
Date:   Thu Jan 29 14:15:54 2015 +0100

    gprs/test: Fix GSUP Purge MS response messages
    Currently the message types for these message types are wrong.
    This patch inserts the correct message types.
    Sponsored-by: On-Waves ehf

commit a16c1ef5660370c88f20df65bc593b63d3bcfcb7
Author: Jacob Erlbeck <jerlbeck at>
Date:   Thu Jan 29 14:12:29 2015 +0100

    sgsn/test: Add tests for PurgeMs responses
    Currently there are not any test cases for PurgeMS GSUP messages in
    This commit adds tests for incoming PurgeMSResult and -Error
    Sponsored-by: On-Waves ehf

commit 6cc717596131558b374520aaa42d9c70decab786
Author: Jacob Erlbeck <jerlbeck at>
Date:   Thu Jan 29 14:00:28 2015 +0100

    gprs: Send GSUP error reply for requests without IMSI
    Currently gprs_subscr_rx_gsup_message returns immediately after it
    detects that an IMSI has not been given in the received GSUP message.
    While this is ok for responses (result or error), a request should
    always be answered.
    This commit adds code to reply with a corresponding error message
    ("Invalid mandatory information") when it receives a request without
    an IMSI.
    Note that the generated error message will not contain an IMSI either.
    Sponsored-by: On-Waves ehf


Summary of changes:
 openbsc/include/openbsc/gprs_sgsn.h | 13 +++----
 openbsc/src/gprs/gprs_subscriber.c  | 72 ++++++++++++++++++++++++++++++++-----
 openbsc/src/gprs/sgsn_vty.c         |  7 ++++
 openbsc/tests/sgsn/sgsn_test.c      | 32 +++++++++++------
 4 files changed, 96 insertions(+), 28 deletions(-)

The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)

More information about the osmocom-commitlog mailing list