diff options
author | Alexander Couzens <lynxis@fe80.eu> | 2018-09-18 16:53:42 +0200 |
---|---|---|
committer | Alexander Couzens <lynxis@fe80.eu> | 2018-09-19 12:11:22 +0200 |
commit | b62653a635f46b7e95378a8ce96dc1ecaee34149 (patch) | |
tree | 7c60679331b6d4fe07f388ffa00d073d37e2da02 | |
parent | a66f0f20f26cebc76b8552ccf01316063466a46b (diff) |
sgsn_ggsn_ctx_drop_pdp: protect against nullpointer when MM is gone
When the GGSN crashs, the SGSN will be notified after
it comes back. Because of the async operation,
the mm ctx could be already gone.
Change-Id: I507a8c2193c84f8dff7f5d669adcd3583331f289
-rw-r--r-- | src/gprs/gprs_sgsn.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/gprs/gprs_sgsn.c b/src/gprs/gprs_sgsn.c index 977ae4898..dc0e7c075 100644 --- a/src/gprs/gprs_sgsn.c +++ b/src/gprs/gprs_sgsn.c @@ -707,7 +707,9 @@ failed: void sgsn_ggsn_ctx_drop_pdp(struct sgsn_pdp_ctx *pctx) { - if (pctx->mm->gmm_state == GMM_REGISTERED_NORMAL) { + /* the MM context can be deleted while the GGSN is not reachable or + * if has been crashed. */ + if (pctx->mm && pctx->mm->gmm_state == GMM_REGISTERED_NORMAL) { gsm48_tx_gsm_deact_pdp_req(pctx, GSM_CAUSE_NET_FAIL, true); sgsn_ggsn_ctx_remove_pdp(pctx->ggsn, pctx); } else { |