[PATCH] osmo-pcu[master]: EGPRS: EPDAN decoding: add test case to show no ack_nack len...

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

arvind.sirsikar gerrit-no-reply at lists.osmocom.org
Thu Aug 25 08:48:35 UTC 2016


Hello Jenkins Builder,

I'd like you to reexamine a change.  Please visit

    https://gerrit.osmocom.org/765

to look at the new patch set (#2).

EGPRS: EPDAN decoding: add test case to show no ack_nack len issue

This patch adds a test case itest_epdan_noack_nack_len_issue which
expects a current bug with EGPRS EPDAN decoding Ack/Nack dissector
length is not received. The test's expectation is corrected along
with the bug fix in a subsequent commit

Related: OS#1805

Change-Id: I8da84794748f2e81722c3eca69b46ccc3a21552e
---
M tests/rlcmac/RLCMACTest.cpp
1 file changed, 31 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/65/765/2

diff --git a/tests/rlcmac/RLCMACTest.cpp b/tests/rlcmac/RLCMACTest.cpp
index 466b89e..74a8ad3 100644
--- a/tests/rlcmac/RLCMACTest.cpp
+++ b/tests/rlcmac/RLCMACTest.cpp
@@ -25,8 +25,12 @@
 #include <cstring>
 #include "csn1.h"
 #include "gsm_rlcmac.h"
+#include "pcu_utils.h"
+#include "bts.h"
+#include "tbf.h"
+#include "gprs_debug.h"
+
 extern "C" {
-extern const struct log_info gprs_log_info;
 #include "pcu_vty.h"
 #include <osmocom/vty/telnet_interface.h>
 #include <osmocom/vty/logging.h>
@@ -211,6 +215,31 @@
 	bitvec_free(resultVector);
 }
 
+void test_epdan_noack_nack_len_issue()
+{
+	std::string testData_epdan[] = {
+	"40200bffd161003e0e519ffffffb800000000000000000" //EPDAN No ACKNACK len
+	};
+
+	bitvec *vector = bitvec_alloc(23);
+
+	bitvec_unhex(vector, testData_epdan[0].c_str());
+	RlcMacUplink_t *data = (RlcMacUplink_t *)malloc(sizeof(RlcMacUplink_t));
+
+	decode_gsm_rlcmac_uplink(vector, data);
+
+	EGPRS_AckNack_Desc_t *urbb =
+		&data->u.Egprs_Packet_Downlink_Ack_Nack.EGPRS_AckNack.Desc;
+	/*
+	 * TODO: URBB len is decoded as 102. so there is total 12 bytes and
+	 * 6 bits in the 13th byte. in 13th byte we expect value
+	 * as 0x00. But we see 0xea. Which is incorrect
+	 */
+	OSMO_ASSERT(!strcmp(osmo_hexdump(urbb->URBB, (urbb->URBB_LENGTH + 7)/8),
+	"7f ff ff ee 00 00 00 00 00 00 00 00 ea "
+	));
+}
+
 int main(int argc, char *argv[])
 {
 	osmo_init_logging(&gprs_log_info);
@@ -218,5 +247,5 @@
 	//printSizeofRLCMAC();
 	testRlcMacDownlink();
 	testRlcMacUplink();
-
+	test_epdan_noack_nack_len_issue();
 }

-- 
To view, visit https://gerrit.osmocom.org/765
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8da84794748f2e81722c3eca69b46ccc3a21552e
Gerrit-PatchSet: 2
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Owner: arvind.sirsikar <arvind.sirsikar at radisys.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: arvind.sirsikar <arvind.sirsikar at radisys.com>



More information about the gerrit-log mailing list