lynxis lazus has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43469?usp=email ) Change subject: gprs_routing_area: add FIXMEs for unit tests ...................................................................... gprs_routing_area: add FIXMEs for unit tests Change-Id: Ia8bb1c4b6ae77bd0e5a44171d99851c74c083835 --- M src/sgsn/gprs_routing_area.c 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/sgsn/gprs_routing_area.c b/src/sgsn/gprs_routing_area.c index 4400d6d..abb613e 100644 --- a/src/sgsn/gprs_routing_area.c +++ b/src/sgsn/gprs_routing_area.c @@ -327,6 +327,11 @@ ra_created = true; } + /* FIXME: write a unit tests + * move a present cell into a new routing area + * The old cell isn't found then! + */ + if (!ra_created) { cell = sgsn_ra_geran_get_cell_by_ra(ra, cgi_ps->cell_identity); if (cell && cell->ran_type == RA_TYPE_GERAN_Gb) { -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43469?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: Ia8bb1c4b6ae77bd0e5a44171d99851c74c083835 Gerrit-Change-Number: 43469 Gerrit-PatchSet: 19 Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> Gerrit-Reviewer: pespin <pespin@sysmocom.de>