pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/32483 )
Change subject: gmm: Allow cb callers to take ownsership of msgb primitive ......................................................................
gmm: Allow cb callers to take ownsership of msgb primitive
Change-Id: Ic60e73d8ba13681217bcb35e4b66cc565239cfd2 --- M src/gmm/gmm_prim.c 1 file changed, 17 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/83/32483/1
diff --git a/src/gmm/gmm_prim.c b/src/gmm/gmm_prim.c index 69e74a2..b1119fe 100644 --- a/src/gmm/gmm_prim.c +++ b/src/gmm/gmm_prim.c @@ -346,7 +346,10 @@ rc = g_gmm_ctx->gmm_up_cb(gmm_prim, g_gmm_ctx->gmm_up_cb_user_data); else rc = gmm_up_cb_dummy(gmm_prim, g_gmm_ctx->gmm_up_cb_user_data); - msgb_free(gmm_prim->oph.msg); + if (rc != 1) + msgb_free(gmm_prim->oph.msg); + else + rc = 0; return rc; }
@@ -505,7 +508,10 @@ rc = g_gmm_ctx->gmm_down_cb(gmm_prim, g_gmm_ctx->gmm_down_cb_user_data); else rc = gmm_down_cb_dummy(gmm_prim, g_gmm_ctx->gmm_down_cb_user_data); - msgb_free(gmm_prim->oph.msg); + if (rc != 1) + msgb_free(gmm_prim->oph.msg); + else + rc = 0; return rc; }