pespin submitted this change.

View Change


Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified laforge: Looks good to me, approved
layer23: modem: sndcp: Properly inform SNDCP layer of ownership transfer

Change-Id: Icc3351f360e41d4a95e056b4fd797c7d133e8a83
---
M src/host/layer23/src/modem/sndcp.c
1 file changed, 10 insertions(+), 0 deletions(-)

diff --git a/src/host/layer23/src/modem/sndcp.c b/src/host/layer23/src/modem/sndcp.c
index fd6aada..b4b7761 100644
--- a/src/host/layer23/src/modem/sndcp.c
+++ b/src/host/layer23/src/modem/sndcp.c
@@ -146,6 +146,7 @@
case OSMO_PRIM(OSMO_GPRS_SM_SMREG_PDP_ACTIVATE, PRIM_OP_RESPONSE):
LOGP(DSNDCP, LOGL_INFO, "%s(): Rx %s\n", __func__, npdu_name);
rc = osmo_gprs_sm_prim_sndcp_upper_down(sndcp_prim);
+ rc = 1; /* Tell SNDCP layer we took msgb ownership and transfer it to SM */
break;
default:
LOGP(DSNDCP, LOGL_ERROR, "%s(): Unexpected Rx %s\n", __func__, npdu_name);

To view, visit change 32593. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Icc3351f360e41d4a95e056b4fd797c7d133e8a83
Gerrit-Change-Number: 32593
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged