From 7079e698481705c82c7aff58ea9c63469626af80 Mon Sep 17 00:00:00 2001 From: Neels Hofmeyr Date: Wed, 5 Dec 2018 21:02:36 +0100 Subject: add osmo_bcd2str() Add a standalone bcd-to-string conversion function with generic parameters. Add a regression test in utils_test.c. So far there is no single universal implementation that converts a BCD to a string. I could only find gsm48_mi_to_string(), which also interprets surrounding bytes, MI type and TMSI as non-BCD value. The idea is to use this function from gsm48_mi_to_string() and similar implementations in subsequent commits. Root cause: in osmo-msc, I want to have an alternative MI-to-string function for composing an FSM name, which needs the BCD part of gsm48_mi_to_string() but not the TMSI part. Change-Id: I86b09d37ceef33331c1a56046a5443127d6c6be0 --- tests/utils/utils_test.c | 114 ++++++++++++++++++++++++++++++++++++++++++++++ tests/utils/utils_test.ok | 29 ++++++++++++ 2 files changed, 143 insertions(+) (limited to 'tests') diff --git a/tests/utils/utils_test.c b/tests/utils/utils_test.c index 2bb1f9ca..a773b3f5 100644 --- a/tests/utils/utils_test.c +++ b/tests/utils/utils_test.c @@ -32,6 +32,7 @@ #include #include #include +#include static void hexdump_test(void) { @@ -383,6 +384,118 @@ static void bcd_test(void) } } +struct bcd2str_test { + const char *bcd_hex; + int start_nibble; + int end_nibble; + bool allow_hex; + size_t str_size; + const char *expect_str; + int expect_rc; +}; + +static const struct bcd2str_test bcd2str_tests[] = { + { + .bcd_hex = "1a 32 54 76 98 f0", + .start_nibble = 1, + .end_nibble = 11, + .expect_str = "1234567890", + .expect_rc = 10, + }, + { + .bcd_hex = "1a 32 a4 cb 9d f0", + .start_nibble = 1, + .end_nibble = 11, + .expect_str = "1234ABCD90", + .expect_rc = -EINVAL, + }, + { + .bcd_hex = "1a 32 a4 cb 9d f0", + .start_nibble = 1, + .end_nibble = 11, + .allow_hex = true, + .expect_str = "1234ABCD90", + .expect_rc = 10, + }, + { + .bcd_hex = "1a 32 54 76 98 f0", + .start_nibble = 1, + .end_nibble = 12, + .expect_str = "1234567890F", + .expect_rc = -EINVAL, + }, + { + .bcd_hex = "1a 32 54 76 98 f0", + .start_nibble = 1, + .end_nibble = 12, + .allow_hex = true, + .expect_str = "1234567890F", + .expect_rc = 11, + }, + { + .bcd_hex = "1a 32 54 76 98 f0", + .start_nibble = 0, + .end_nibble = 12, + .allow_hex = true, + .expect_str = "A1234567890F", + .expect_rc = 12, + }, + { + .bcd_hex = "1a 32 54 76 98 f0", + .start_nibble = 1, + .end_nibble = 12, + .str_size = 5, + .expect_str = "1234", + .expect_rc = 11, + }, + { + .bcd_hex = "", + .start_nibble = 1, + .end_nibble = 1, + .expect_str = "", + .expect_rc = 0, + }, +}; + +static void bcd2str_test(void) +{ + int i; + uint8_t bcd[64]; + int rc; + + printf("\nTesting bcd to string conversion\n"); + + for (i = 0; i < ARRAY_SIZE(bcd2str_tests); i++) { + const struct bcd2str_test *t = &bcd2str_tests[i]; + char str[64] = {}; + size_t str_size = t->str_size ? : sizeof(str); + + osmo_hexparse(t->bcd_hex, bcd, sizeof(bcd)); + + printf("- BCD-input='%s' nibbles=[%d..%d[ str_size=%zu\n", t->bcd_hex, + t->start_nibble, t->end_nibble, str_size); + rc = osmo_bcd2str(str, str_size, bcd, t->start_nibble, t->end_nibble, t->allow_hex); + + printf(" rc=%d\n", rc); + + OSMO_ASSERT(str[str_size-1] == '\0'); + printf(" -> %s\n", osmo_quote_str(str, -1)); + + if (rc != t->expect_rc) + printf(" ERROR: expected rc=%d\n", t->expect_rc); + if (strcmp(str, t->expect_str)) + printf(" ERROR: expected result %s\n", osmo_quote_str(t->expect_str, -1)); + } + + printf("- zero output buffer\n"); + rc = osmo_bcd2str(NULL, 100, bcd, 1, 2, false); + printf(" bcd2str(NULL, ...) -> %d\n", rc); + OSMO_ASSERT(rc < 0); + rc = osmo_bcd2str((char*)23, 0, bcd, 1, 2, false); + printf(" bcd2str(dst, 0, ...) -> %d\n", rc); + OSMO_ASSERT(rc < 0); +} + static void str_escape_test(void) { int i; @@ -810,6 +923,7 @@ int main(int argc, char **argv) test_ipa_ccm_id_resp_parsing(); test_is_hexstr(); bcd_test(); + bcd2str_test(); str_escape_test(); str_quote_test(); isqrt_test(); diff --git a/tests/utils/utils_test.ok b/tests/utils/utils_test.ok index 3ea8ec6a..65e32ed2 100644 --- a/tests/utils/utils_test.ok +++ b/tests/utils/utils_test.ok @@ -80,6 +80,35 @@ Testing BCD conversion val=0xe, expected=E, found=E val=0xf, expected=F, found=F +Testing bcd to string conversion +- BCD-input='1a 32 54 76 98 f0' nibbles=[1..11[ str_size=64 + rc=10 + -> "1234567890" +- BCD-input='1a 32 a4 cb 9d f0' nibbles=[1..11[ str_size=64 + rc=-22 + -> "1234ABCD90" +- BCD-input='1a 32 a4 cb 9d f0' nibbles=[1..11[ str_size=64 + rc=10 + -> "1234ABCD90" +- BCD-input='1a 32 54 76 98 f0' nibbles=[1..12[ str_size=64 + rc=-22 + -> "1234567890F" +- BCD-input='1a 32 54 76 98 f0' nibbles=[1..12[ str_size=64 + rc=11 + -> "1234567890F" +- BCD-input='1a 32 54 76 98 f0' nibbles=[0..12[ str_size=64 + rc=12 + -> "A1234567890F" +- BCD-input='1a 32 54 76 98 f0' nibbles=[1..12[ str_size=5 + rc=11 + -> "1234" +- BCD-input='' nibbles=[1..1[ str_size=64 + rc=0 + -> "" +- zero output buffer + bcd2str(NULL, ...) -> -12 + bcd2str(dst, 0, ...) -> -12 + Testing string escaping - all chars from 0 to 255 in batches of 16: "\0\1\2\3\4\5\6\a\b\t\n\v\f\r\14\15" -- cgit v1.2.3