aboutsummaryrefslogtreecommitdiffstats
path: root/src/csn1.cpp
diff options
context:
space:
mode:
authorSaurabh Sharan <saurabh.sharan@radisys.com>2016-03-16 19:17:32 +0530
committerHolger Hans Peter Freyther <holger@moiji-mobile.com>2016-03-16 15:01:53 +0100
commit2b09c39c9cb6d8a9761539d00e21b21352272905 (patch)
tree06bcecb94118b161190dea7cdfc44173fd0e53b9 /src/csn1.cpp
parentbacb65b48b520ace03c877d3092b58e50cb68266 (diff)
Fix issue in encoding CSN_RECURSIVE_ARRAY
The remaining_bits_len is correctly decremented while encoding CSN_RECURSIVE_ARRAY for fixing the bug. Details of the bug is in https://projects.osmocom.org/issues/1641 During introduction of basic EGPRS feature new hex dump message PUASS, from a different working network log was used in Unit test. It exposed the issue of incorrect handling of recursive array encoding in osmo-pcu. Fixes: OS#1641
Diffstat (limited to 'src/csn1.cpp')
-rw-r--r--src/csn1.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/csn1.cpp b/src/csn1.cpp
index 82bf17f4..d51fe83e 100644
--- a/src/csn1.cpp
+++ b/src/csn1.cpp
@@ -2504,6 +2504,7 @@ gint16 csnStreamEncoder(csnStream_t* ar, const CSN_DESCR* pDescr, bitvec *vector
bitvec_write_field(vector, writeIndex, !Tag, 1);
LOGPC(DCSN1, LOGL_NOTICE, "%s = %u | ", pDescr->sz , (unsigned)(!Tag));
bit_offset++;
+ remaining_bits_len--;
pDescr++;
break;