pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmo-gprs/+/32644 )

Change subject: sm: Fix APN IE content not transmitted in Act PDP Ctx Req
......................................................................

sm: Fix APN IE content not transmitted in Act PDP Ctx Req

Change-Id: I70be2ff5e7e0fea5e93d7b9963064c38a0382f30
---
M src/sm/sm_pdu.c
M tests/sm/sm_prim_test.ok
2 files changed, 11 insertions(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, approved




diff --git a/src/sm/sm_pdu.c b/src/sm/sm_pdu.c
index f8512c0..8b9dd68 100644
--- a/src/sm/sm_pdu.c
+++ b/src/sm/sm_pdu.c
@@ -209,6 +209,7 @@
                if (rc < 0)
                        return -EINVAL;
                *l = rc;
+               msgb_put(msg, *l);
        }

        /* 10.5.6.3 Protocol configuration options (Optional) */
diff --git a/tests/sm/sm_prim_test.ok b/tests/sm/sm_prim_test.ok
index 33f7291..d713308 100644
--- a/tests/sm/sm_prim_test.ok
+++ b/tests/sm/sm_prim_test.ok
@@ -1,6 +1,6 @@
 ==== test_sm_prim_ms() [start] ====
 test_sm_prim_gmm_down_cb(): Rx GMMSM-ESTABLISH.request sess_id=0
-test_sm_prim_gmm_down_cb(): Rx GMMSM-UNITDATA.request sess_id=0 SMPDU=[8a 41 
06 03 01 00 02 01 21 28 04 27 01 00 ]
+test_sm_prim_gmm_down_cb(): Rx GMMSM-UNITDATA.request sess_id=0 SMPDU=[8a 41 
06 03 01 00 02 01 21 28 04 03 61 70 6e 27 01 00 ]
 test_sm_prim_sndcp_up_cb(): Rx SNSM-ACTIVATE.indication
 test_sm_prim_up_cb(): Rx SMREG-PDP_ACTIVATE.confirm
 ==== test_sm_prim_ms() [end] ====

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

Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I70be2ff5e7e0fea5e93d7b9963064c38a0382f30
Gerrit-Change-Number: 32644
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-MessageType: merged

Reply via email to