diff options
author | Jacob Erlbeck <jerlbeck@sysmocom.de> | 2014-11-07 14:17:44 +0100 |
---|---|---|
committer | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2014-11-14 10:23:54 +0100 |
commit | f951a01bb227e524eb369051c95fbebace7570d0 (patch) | |
tree | 9a01dc28a3675f95e4b43ab70e05da8a25eb442f /openbsc/src/gprs/gprs_sgsn.c | |
parent | f6e7d99d54cc75fdc19433011afb5eaaa8a2a002 (diff) |
sgsn: Refactor sgsn_auth to separate request and authorization
Currently the authorization is done in sgsn_auth_request for ACL
based authorization. This doesn't match the way remote authorization
would work, so that there is a second call to sgsn_auth_state already
present in sgsn_auth_update.
This patch removes the autorization check completely from
sgsn_auth_request which in turn calls sgsn_auth_update directly now.
Sponsored-by: On-Waves ehf
Diffstat (limited to 'openbsc/src/gprs/gprs_sgsn.c')
-rw-r--r-- | openbsc/src/gprs/gprs_sgsn.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/openbsc/src/gprs/gprs_sgsn.c b/openbsc/src/gprs/gprs_sgsn.c index daf9483bd..9fc4b86f3 100644 --- a/openbsc/src/gprs/gprs_sgsn.c +++ b/openbsc/src/gprs/gprs_sgsn.c @@ -451,11 +451,9 @@ int sgsn_force_reattach_oldmsg(struct msgb *oldmsg) return gsm0408_gprs_force_reattach_oldmsg(oldmsg); } -void sgsn_update_subscriber_data(struct sgsn_mm_ctx *mmctx, - struct sgsn_subscriber_data *sd) +void sgsn_update_subscriber_data(struct sgsn_mm_ctx *mmctx) { OSMO_ASSERT(mmctx); - if (sd->auth_state != mmctx->auth_state) - sgsn_auth_update(mmctx, sd); + sgsn_auth_update(mmctx, sgsn); } |