lynxis lazus has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/15482 )
Change subject: gprs_ranap: release Iu UE Context when exiting PMM Connected ...................................................................... Patch Set 7: (1 comment) https://gerrit.osmocom.org/#/c/15482/6/src/sgsn/gprs_ranap.c File src/sgsn/gprs_ranap.c: https://gerrit.osmocom.org/#/c/15482/6/src/sgsn/gprs_ranap.c@138 PS6, Line 138: if (mm->iu.mm_state_fsm->state == ST_PMM_CONNECTED) > (could mm_state_fsm ever be NULL for some mm in a weird state?) no. otherwise it would follow the path in line 123 -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/15482 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: Ie05d2bdf8dfb593b4c7e837107a3a06f22e90119 Gerrit-Change-Number: 15482 Gerrit-PatchSet: 7 Gerrit-Owner: lynxis lazus <lyn...@fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <lafo...@osmocom.org> Gerrit-Reviewer: lynxis lazus <lyn...@fe80.eu> Gerrit-Reviewer: neels <nhofm...@sysmocom.de> Gerrit-Reviewer: pespin <pes...@sysmocom.de> Gerrit-Comment-Date: Thu, 03 Oct 2019 20:27:49 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <nhofm...@sysmocom.de> Gerrit-MessageType: comment