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

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 ----------------------------------------------------------------- http://cgit.osmocom.org/openbsc/commit/?id=8b77456f406e8215f92150868062edf1… commit 8b77456f406e8215f92150868062edf1122e3aa7 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Tue Sep 30 13:53:26 2014 +0200 sgsn: Added TODOs (TODO) 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 http://cgit.osmocom.org/openbsc/commit/?id=9b9c689b7f4895d12030105b804ab5ac… commit 9b9c689b7f4895d12030105b804ab5ac20a3b2bd Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Wed Dec 17 14:03:35 2014 +0100 sgsn: Add PDP info to subscriber data (TODO) TODO: - commit message - raw APN representation ??? - tests - check PDP data list on PDP Ctx Act (other commit) Sponsored-by: On-Waves ehf http://cgit.osmocom.org/openbsc/commit/?id=16a341029d698b58db0b42a5b454b34f… commit 16a341029d698b58db0b42a5b454b34fc0fb6589 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Fri Dec 19 19:11:20 2014 +0100 sgsn: Add logging to connect_timer_cb (TODO) TODO: - LOGLevel ? Sponsored-by: On-Waves ehf http://cgit.osmocom.org/openbsc/commit/?id=8bd72827e74c49ebb504f31bc2118bfe… commit 8bd72827e74c49ebb504f31bc2118bfe6bdd65a7 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> 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 implementation. 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. Addresses: == 372 bytes in 1 blocks are possibly lost in loss record 7 of 9 == at 0x402A17C: malloc (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so) == 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 http://cgit.osmocom.org/openbsc/commit/?id=b922a3dcb059cdf9aefd1574d74cdf1e… commit b922a3dcb059cdf9aefd1574d74cdf1e88cb1ea2 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> 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 http://cgit.osmocom.org/openbsc/commit/?id=df2627ad89e6ed2d217e960832545e56… commit df2627ad89e6ed2d217e960832545e56bc639b35 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> 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 http://cgit.osmocom.org/openbsc/commit/?id=6a3d6fdb84c7e5d91df2c1b340cd28c2… commit 6a3d6fdb84c7e5d91df2c1b340cd28c20fe1c503 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> 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 http://cgit.osmocom.org/openbsc/commit/?id=a16c1ef5660370c88f20df65bc593b63… commit a16c1ef5660370c88f20df65bc593b63d3bcfcb7 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> 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 test_subscriber_gsup. This commit adds tests for incoming PurgeMSResult and -Error messages. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/openbsc/commit/?id=6cc717596131558b374520aaa42d9c70… commit 6cc717596131558b374520aaa42d9c70decab786 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> 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(-) hooks/post-receive -- The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)