openbsc.git branch jerlbeck/wip/gprs-patching updated. 0.13.0-772-gdc30ded

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 4193f174dd6cdaabf2d4319faec463261a37d5af (commit) discards 95f84db2402d458fd5b3d899f08cb33ec9cd3212 (commit) discards 7518cdd42e257e70d9ec52ebb3567ab2df1472d6 (commit) discards dd9645c9e2cf910bd30abedecc086e7207bdce92 (commit) discards aa5dd818449d27659cb9e8b10d69ba3e9f875097 (commit) discards 6e8c0bd7c97273f0ab171471a4725447ade17ae3 (commit) discards a374e69d392dc4cfccf7a367c74961b2786bbc90 (commit) discards 4cc973bce2eded371712ec73475d6d62c4aa91e7 (commit) discards 9cc7a20c65bc881ea8e0f4084efac7180266b239 (commit) discards e99d24e9f00bc55e3c215fb3c0380171ccacebfd (commit) discards 744c7874f51c98c1ccf33814b2a88e0aeab34472 (commit) discards 2b35c1709abf42c9d250e280a18eb7886a3f4b37 (commit) discards 5465f915bf4dcebb89171a9cf9f78a9e1b7c4f7f (commit) discards ad30d8d3b3af86a789e42ab3804f14aba98c43e0 (commit) discards 49ac6fcf3bd39aa6f1fa19fb199de6e45bf1e2f6 (commit) discards c11c6b97f6c20739a39edc10cf5988c2583b951e (commit) discards 40415c5655d5f5b67c557f733f0c47b2d60eae88 (commit) discards be35395defcd81d7661365b97c072711e0e55d23 (commit) discards a00c3bd090b024f116c6e134b668cade04b535c1 (commit) discards 62a432c7db59c3004a9fc3272d46a6cc9c4269aa (commit) discards 584bf98d87497381ea30526d73899d3a5542eb61 (commit) discards b8242b0c2422a948fdce005558fc8e5738555ff0 (commit) discards acc28f5fa7b9c1ad607bddb47429a04a068c14ef (commit) via dc30dedcb857c9cfa9efb451a725ed83c0774def (commit) via aa78094b4bc11caff7b3c3c90366448a75269d84 (commit) via 4d1f467f4b78c5d630dc5aa0da847d7124e5b645 (commit) via 2b8b8aeb85f20782be66e7968b63f0bb94139483 (commit) via e920272bb4ef66dbc662c81ef852dec03bf3c124 (commit) via 230c37ff165ee2cff7d9098fd15c7989861421df (commit) via a464776f1513218645c5b4284c0b1fc1e1a17b59 (commit) via 51ad560afe535d4c7528eca2aebb9e9de8a12df7 (commit) via deddfdb54f878e64df11a8151239efd70b975015 (commit) via 95617192e94c178ac36f3f379cbbcbfbaadb9939 (commit) via 175a240285897317fff4a94e4f9b62408797b4b6 (commit) via a49b2c010e05eeaa979c76c9a277bb5aaea31968 (commit) via b92a538d236ccffe97803279ecbaa242f8faa66a (commit) via 8a64141a533c0b090bbeb756bb8eac2eee457ad4 (commit) via b1461152e6945a747ef2d65ee5908cc5d59acbbd (commit) via b1edf7b64f6c05932c83b4d31b1127f2a833601e (commit) via 619b014d3aacea1b33b70432af01e80eb2252af0 (commit) via 02ab91e6a73e16da51918b52548a0f4be4c6a0f6 (commit) via a0b6efb3685ea90190cfbd9423a74c181ea4f12f (commit) via f951a01bb227e524eb369051c95fbebace7570d0 (commit) via f6e7d99d54cc75fdc19433011afb5eaaa8a2a002 (commit) via c4f9bf3142f54b63174f653b07103dd6ab36b1ae (commit) via 80d07e30c76f2bd72ee382b4b7941262c1d5865d (commit) via 106f547733450afda1ddbd7e886dc8c902fed4d4 (commit) via 144b8b1ca77f628ea4cf87ff903b7e79f0abf9dd (commit) via 423f8bfa027d544867dcf7afc188a8ea5221e2bc (commit) via 0c06f98ced2b70801147eb52d6596079e473dfc9 (commit) via b1c074f62c1d04138b751f71e3b07fbb0a521898 (commit) via abdf02b9b988a816201b4530f36f709ca4c146ac (commit) via a790456f1b826ea204f47c4aa4973dc777d00d2a (commit) via 0074a77424c95d2d7d0608dc0dfabc1948e62fe0 (commit) via 93eae8ec78d145cf87b7858ebee60c72da7a2d20 (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 (4193f174dd6cdaabf2d4319faec463261a37d5af) \ N -- N -- N (dc30dedcb857c9cfa9efb451a725ed83c0774def) 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=dc30dedcb857c9cfa9efb451a725ed83… commit dc30dedcb857c9cfa9efb451a725ed83c0774def 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=aa78094b4bc11caff7b3c3c90366448a… commit aa78094b4bc11caff7b3c3c90366448a75269d84 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Tue Nov 4 12:44:15 2014 +0100 sgsn: Notify an affected MM context if an ACL rule is removed (TODO) Currently if an ACL access rule is removed, MS already attached are not affected until they try to reattach to the SGSN. This patch adds sgsn_auth_update_all() that iterates through allMM contexts and calls sgsn_auth_update() on each of them. This function is then called when 'imsi-acl del IMSI' is executed after the ACL has been removed. TODO: - decide, whether ito replace that by a dedicated update command in the VTY ENABLE node Sponsored-by: On-Waves ehf http://cgit.osmocom.org/openbsc/commit/?id=4d1f467f4b78c5d630dc5aa0da847d71… commit 4d1f467f4b78c5d630dc5aa0da847d7124e5b645 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Fri Oct 17 10:08:02 2014 +0200 sgsn: Do not die in _bssgp_tx_dl_ud on TLLI mismatch (TODO) Currently an OSMO_ASSERT fails, if the message's and the mmctx's TLLIs do not match. This commit turns the assertion into an ERROR log message and uses the default values for IMSI, DRX, and RA CAP instead of the MM context values in this case. TODO: - find the real source for the problem - add test cases derived from what is being described in the ticket Ticket: OW#1322 Sponsored-by: On-Waves ehf http://cgit.osmocom.org/openbsc/commit/?id=2b8b8aeb85f20782be66e7968b63f0bb… commit 2b8b8aeb85f20782be66e7968b63f0bb94139483 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Tue Nov 11 14:01:48 2014 +0100 sgsn: Add VTY commands to manage subscriber cache This adds the following commands to the ENABLE node: - show subscriber cache - update-subscriber imsi IMSI insert authorized (0|1) - update-subscriber imsi IMSI cancel These commands are mainly testing tools and maintenance helpers. The update commands work asynchronously and can be used to complete a pending update request or to terminate an existing connection. Note that the subscriber cache is not stored persistently and will always be empty after an SGSN restart. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/openbsc/commit/?id=e920272bb4ef66dbc662c81ef852dec0… commit e920272bb4ef66dbc662c81ef852dec03bf3c124 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Tue Nov 11 14:47:38 2014 +0100 sgsn: Add gsm0408_gprs_access_cancelled (TODO) This function is called to delete an established MM context without invoking a detach procedure that could disable the GPRS functionality of the MS until the next power cycle. It is called when a subscriber is cancelled. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/openbsc/commit/?id=230c37ff165ee2cff7d9098fd15c7989… commit 230c37ff165ee2cff7d9098fd15c7989861421df Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Wed Nov 12 10:18:09 2014 +0100 sgsn: Integrate subscriber handling into the SGSN (TODO) This commit adds a new authorization policy 'remote' and uses the subscriber cache for authorization when this policy is being used. Note that there is no remote backend implemented yet. TODO: - tests - decide, whether 'remote' could be named better, e.g. sth based on 'subscriber' Ticket: OW#???? Sponsored-by: On-Waves ehf http://cgit.osmocom.org/openbsc/commit/?id=a464776f1513218645c5b4284c0b1fc1… commit a464776f1513218645c5b4284c0b1fc1e1a17b59 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Wed Nov 12 10:12:11 2014 +0100 sgsn: Add gprs_subscriber.c This patch adds GPRS specific functions for gsm_subscriber objects (allocation, retrieval, deletion) and subscriber data requests/updates. The sgsn_update_subscriber_data callback is used to notify the sgsn about updates and is extended by a parameter that passes a reference to a gsm_subscriber. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/openbsc/commit/?id=51ad560afe535d4c7528eca2aebb9e9d… commit 51ad560afe535d4c7528eca2aebb9e9de8a12df7 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Mon Nov 24 15:04:15 2014 +0100 sgsn/test: Add test case for unexpected Detach Accepts The commit "sgsn: Don't send XID reset after Detach Accept" fixed the SGSN to not respond to a Detach Accept message when there is no MM context. This patch adds a test case to verify, that there is really no message sent in that case. The test fails when the commit mentioned aboved is reverted. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/openbsc/commit/?id=deddfdb54f878e64df11a8151239efd7… commit deddfdb54f878e64df11a8151239efd70b975015 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Mon Nov 24 14:40:28 2014 +0100 sgsn/test: Don't rely on the actual PRNG sequence Currently the expected P-TMSI generated by the SGSN is hard-coded into the test. This adds a dependency on the implementation of rand() and thus the libc used. This breaks the test e.g. on FreeBSD. This patch modifies test_gmm_attach() to srand(1) first, generate the P-TMSI, and finally srand(1) again before starting the test. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/openbsc/commit/?id=95617192e94c178ac36f3f379cbbcbfb… commit 95617192e94c178ac36f3f379cbbcbfbaadb9939 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Thu Oct 30 17:15:43 2014 +0100 gbproxy: Reset TLLIs when the link_info is found by IMSI/P-TMSI Currently when the MS does a re-attach without doing a proper detach first, the gbproxy uses the old local TLLI if patching and the keep mode are enabled. This leads to a failing attachment procedure when TLLI patching is also enabled. This patch changes gbproxy_update_link_state_ul to reset all TLLIs within the link_info if the message contains an unknown TLLI and an MI. This is generally the case with Attach Request messages. The gbproxy_get_link_info_ul gets an additional tlli_is_valid output parameter that is set, when a TLLI was present and found. This flag is then used instead of checking tlli.current == 0 to set TLLI/P-TMSI e.g. Attach Requests when a link_info was already present for the P-TMSI/IMSI used in such a request. Ticket: OW#1324 Sponsored-by: On-Waves ehf ----------------------------------------------------------------------- Summary of changes: openbsc/include/openbsc/gprs_sgsn.h | 4 +- openbsc/include/openbsc/gsm_data.h | 1 + openbsc/include/openbsc/gsm_data_shared.h | 3 +- openbsc/include/openbsc/gsm_subscriber.h | 1 + openbsc/include/openbsc/mgcp.h | 1 + openbsc/src/gprs/gb_proxy_tlli.c | 13 +++--- openbsc/src/gprs/gprs_subscriber.c | 2 +- openbsc/src/libbsc/bsc_ctrl_commands.c | 73 ++++++++++++++++++++++++++++++- openbsc/src/libbsc/bsc_init.c | 4 +- openbsc/src/libbsc/system_information.c | 3 ++ openbsc/src/libcommon/gsm_data_shared.c | 3 ++ openbsc/src/libmgcp/mgcp_protocol.c | 5 ++- openbsc/src/libmgcp/mgcp_vty.c | 50 +++++++++++++++++++++ openbsc/tests/ctrl_test_runner.py | 59 +++++++++++++++++++++++++ openbsc/tests/mgcp/mgcp_test.c | 58 ++++++++++++++++++++++++ openbsc/tests/mgcp/mgcp_test.ok | 1 + openbsc/tests/sgsn/sgsn_test.c | 45 ++++++++++++++++++- openbsc/tests/sgsn/sgsn_test.ok | 1 + openbsc/tests/vty_test_runner.py | 15 +++++++ 19 files changed, 326 insertions(+), 16 deletions(-) hooks/post-receive -- The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)