osmith has posted comments on this change. ( https://gerrit.osmocom.org/14011 )
Change subject: Use gtp_delete_context_req2() everywhere ...................................................................... Patch Set 1: I am new to the osmo-ggsn codebase. But from what I can tell, and from looking at the similar sgsn patch [1], this is the only change that needs to be done. The sgsn patch also had a "pctx->lib = NULL;" line removed in its delete_pdp_conf() function, however in ggsn.c's gtp_delete_pdp_conf() and sgsnemu.c's delete_pdp_conf(), there is no similar code. [1]: https://gerrit.osmocom.org/#/c/osmo-sgsn/+/10019/ -- To view, visit https://gerrit.osmocom.org/14011 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ggsn Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iecc8c5ac45207e7e20129559c4ac7f3c67dfb36a Gerrit-Change-Number: 14011 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <[email protected]> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: osmith <[email protected]> Gerrit-Comment-Date: Mon, 13 May 2019 09:49:40 +0000 Gerrit-HasComments: No Gerrit-HasLabels: No
