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;
 }


--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32483
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: Ic60e73d8ba13681217bcb35e4b66cc565239cfd2
Gerrit-Change-Number: 32483
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-MessageType: newchange

Reply via email to