[PATCH] libosmocore[master]: gsm0808: Add create functions for CIPHER MODE COMMAND

2017-04-05 Thread dexter
Hello Harald Welte, Jenkins Builder,

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

https://gerrit.osmocom.org/2182

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

gsm0808: Add create functions for CIPHER MODE COMMAND

gsm0808.h/c lacks functionality to generate CIPHER MODE COMMAND messages. These
messages are required if the code is used in an MSC implementation.

This commit adds a gsm0808_create_cipher() function, that generates an A/AoiP
CIPHER MODE COMMAND message.

Change-Id: I8eb1c357860c3e740b0f5d17e1c256bc87920958
---
M include/osmocom/gsm/gsm0808.h
M src/gsm/gsm0808.c
M src/gsm/libosmogsm.map
M tests/gsm0808/gsm0808_test.c
M tests/gsm0808/gsm0808_test.ok
5 files changed, 78 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/82/2182/5

diff --git a/include/osmocom/gsm/gsm0808.h b/include/osmocom/gsm/gsm0808.h
index fd73376..9738a55 100644
--- a/include/osmocom/gsm/gsm0808.h
+++ b/include/osmocom/gsm/gsm0808.h
@@ -35,6 +35,8 @@
 struct msgb *gsm0808_create_reset_ack(void);
 struct msgb *gsm0808_create_clear_command(uint8_t reason);
 struct msgb *gsm0808_create_clear_complete(void);
+struct msgb *gsm0808_create_cipher(const struct gsm0808_encrypt_info *ei,
+  const uint8_t *cipher_response_mode);
 struct msgb *gsm0808_create_cipher_complete(struct msgb *layer3, uint8_t 
alg_id);
 struct msgb *gsm0808_create_cipher_reject(uint8_t cause);
 struct msgb *gsm0808_create_classmark_update(const uint8_t *cm2, uint8_t 
cm2_len,
diff --git a/src/gsm/gsm0808.c b/src/gsm/gsm0808.c
index b8ab79b..c952a9f 100644
--- a/src/gsm/gsm0808.c
+++ b/src/gsm/gsm0808.c
@@ -131,6 +131,39 @@
return msg;
 }
 
+struct msgb *gsm0808_create_cipher(const struct gsm0808_encrypt_info *ei,
+  const uint8_t *cipher_response_mode)
+{
+   /* See also: 3GPP TS 48.008 3.2.1.30 CIPHER MODE COMMAND */
+   struct msgb *msg;
+
+   /* Mandatory emelent! */
+   OSMO_ASSERT(ei);
+
+   msg =
+   msgb_alloc_headroom(BSSMAP_MSG_SIZE, BSSMAP_MSG_HEADROOM,
+   "cipher-mode-command");
+   if (!msg)
+   return NULL;
+
+   /* Message Type 3.2.2.1 */
+   msgb_v_put(msg, BSS_MAP_MSG_CIPHER_MODE_CMD);
+
+   /* Encryption Information 3.2.2.10 */
+   gsm0808_enc_encrypt_info(msg, ei);
+
+   /* Cipher Response Mode 3.2.2.34 */
+   if (cipher_response_mode)
+   msgb_tv_put(msg, GSM0808_IE_CIPHER_RESPONSE_MODE,
+   *cipher_response_mode);
+
+   /* pre-pend the header */
+   msg->l3h =
+   msgb_tv_push(msg, BSSAP_MSG_BSS_MANAGEMENT, msgb_length(msg));
+
+   return msg;
+}
+
 struct msgb *gsm0808_create_cipher_complete(struct msgb *layer3, uint8_t 
alg_id)
 {
struct msgb *msg = msgb_alloc_headroom(BSSMAP_MSG_SIZE, 
BSSMAP_MSG_HEADROOM,
diff --git a/src/gsm/libosmogsm.map b/src/gsm/libosmogsm.map
index cf0a7fe..786bf08 100644
--- a/src/gsm/libosmogsm.map
+++ b/src/gsm/libosmogsm.map
@@ -128,6 +128,7 @@
 gsm0808_create_ass_compl;
 gsm0808_create_assignment_failure;
 gsm0808_create_ass_fail;
+gsm0808_create_cipher;
 gsm0808_create_cipher_complete;
 gsm0808_create_cipher_reject;
 gsm0808_create_classmark_update;
diff --git a/tests/gsm0808/gsm0808_test.c b/tests/gsm0808/gsm0808_test.c
index af457b4..f33e0bd 100644
--- a/tests/gsm0808/gsm0808_test.c
+++ b/tests/gsm0808/gsm0808_test.c
@@ -144,6 +144,46 @@
msgb_free(msg);
 }
 
+static void test_create_cipher()
+{
+   static const uint8_t res[] =
+   { 0x00, 0x0c, 0x53, 0x0a, 0x09, 0x03, 0xaa,
+ 0xbb, 0xcc, 0xdd, 0xee, 0xff, 0x23, 0x42 };
+   static const uint8_t res2[] =
+   { 0x00, 0x0e, 0x53, 0x0a, 0x09, 0x03, 0xaa,
+ 0xbb, 0xcc, 0xdd, 0xee, 0xff, 0x23, 0x42,
+ GSM0808_IE_CIPHER_RESPONSE_MODE, 0x01 };
+   struct msgb *msg;
+   struct gsm0808_encrypt_info ei;
+   uint8_t include_imeisv;
+
+   memset(, 0, sizeof(ei));
+   ei.perm_algo[0] = GSM0808_ALG_ID_A5_0;
+   ei.perm_algo[1] = GSM0808_ALG_ID_A5_1;
+   ei.perm_algo_len = 2;
+   ei.key[0] = 0xaa;
+   ei.key[1] = 0xbb;
+   ei.key[2] = 0xcc;
+   ei.key[3] = 0xdd;
+   ei.key[4] = 0xee;
+   ei.key[5] = 0xff;
+   ei.key[6] = 0x23;
+   ei.key[7] = 0x42;
+   ei.key_len = 8;
+   include_imeisv = 1;
+
+   printf("Testing creating Chipher Mode Command\n");
+   msg = gsm0808_create_cipher(, NULL);
+   OSMO_ASSERT(msg);
+   VERIFY(msg, res, ARRAY_SIZE(res));
+   msgb_free(msg);
+
+   msg = gsm0808_create_cipher(, _imeisv);
+   OSMO_ASSERT(msg);
+   VERIFY(msg, res2, ARRAY_SIZE(res2));
+   msgb_free(msg);
+}
+
 static void test_create_cipher_complete()
 {
static const uint8_t res1[] = {
@@ -700,6 +740,7 @@
test_create_reset();
test_create_clear_command();
test_create_clear_complete();
+   

[PATCH] libosmocore[master]: gsm0808: Add create functions for CIPHER MODE COMMAND

2017-04-04 Thread dexter
Hello Jenkins Builder,

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

https://gerrit.osmocom.org/2182

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

gsm0808: Add create functions for CIPHER MODE COMMAND

gsm0808.h/c lacks functionality to generate CIPHER MODE COMMAND messages. These
messages are required if the code is used in an MSC implementation.

This commit adds a gsm0808_create_cipher() function, that generates an A/AoiP
CIPHER MODE COMMAND message.

Change-Id: I8eb1c357860c3e740b0f5d17e1c256bc87920958
---
M include/osmocom/gsm/gsm0808.h
M src/gsm/gsm0808.c
M src/gsm/libosmogsm.map
M tests/gsm0808/gsm0808_test.c
M tests/gsm0808/gsm0808_test.ok
5 files changed, 78 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/82/2182/3

diff --git a/include/osmocom/gsm/gsm0808.h b/include/osmocom/gsm/gsm0808.h
index fd0d5ee..7e7cd46 100644
--- a/include/osmocom/gsm/gsm0808.h
+++ b/include/osmocom/gsm/gsm0808.h
@@ -34,6 +34,8 @@
 struct msgb *gsm0808_create_reset_ack(void);
 struct msgb *gsm0808_create_clear_command(uint8_t reason);
 struct msgb *gsm0808_create_clear_complete(void);
+struct msgb *gsm0808_create_cipher(struct gsm0808_encrypt_info *ei,
+  uint8_t *cipher_response_mode);
 struct msgb *gsm0808_create_cipher_complete(struct msgb *layer3, uint8_t 
alg_id);
 struct msgb *gsm0808_create_cipher_reject(uint8_t cause);
 struct msgb *gsm0808_create_classmark_update(const uint8_t *cm2, uint8_t 
cm2_len,
diff --git a/src/gsm/gsm0808.c b/src/gsm/gsm0808.c
index 0dc7b8e..b71a0a1 100644
--- a/src/gsm/gsm0808.c
+++ b/src/gsm/gsm0808.c
@@ -131,6 +131,39 @@
return msg;
 }
 
+struct msgb *gsm0808_create_cipher(struct gsm0808_encrypt_info *ei,
+  uint8_t *cipher_response_mode)
+{
+   /* See also: 3GPP TS 48.008 3.2.1.30 CIPHER MODE COMMAND */
+   struct msgb *msg;
+
+   /* Mandatory emelent! */
+   OSMO_ASSERT(ei);
+
+   msg =
+   msgb_alloc_headroom(BSSMAP_MSG_SIZE, BSSMAP_MSG_HEADROOM,
+   "cipher-mode-command");
+   if (!msg)
+   return NULL;
+
+   /* Message Type 3.2.2.1 */
+   msgb_v_put(msg, BSS_MAP_MSG_CIPHER_MODE_CMD);
+
+   /* Encryption Information 3.2.2.10 */
+   gsm0808_enc_encrypt_info(msg, ei);
+
+   /* Cipher Response Mode 3.2.2.34 */
+   if (cipher_response_mode)
+   msgb_tv_put(msg, GSM0808_IE_CIPHER_RESPONSE_MODE,
+   *cipher_response_mode);
+
+   /* pre-pend the header */
+   msg->l3h =
+   msgb_tv_push(msg, BSSAP_MSG_BSS_MANAGEMENT, msgb_length(msg));
+
+   return msg;
+}
+
 struct msgb *gsm0808_create_cipher_complete(struct msgb *layer3, uint8_t 
alg_id)
 {
struct msgb *msg = msgb_alloc_headroom(BSSMAP_MSG_SIZE, 
BSSMAP_MSG_HEADROOM,
diff --git a/src/gsm/libosmogsm.map b/src/gsm/libosmogsm.map
index ac8d467..2826cd8 100644
--- a/src/gsm/libosmogsm.map
+++ b/src/gsm/libosmogsm.map
@@ -128,6 +128,7 @@
 gsm0808_create_assignment_completed_aoip;
 gsm0808_create_assignment_failure;
 gsm0808_create_assignment_failure_aoip;
+gsm0808_create_cipher;
 gsm0808_create_cipher_complete;
 gsm0808_create_cipher_reject;
 gsm0808_create_classmark_update;
diff --git a/tests/gsm0808/gsm0808_test.c b/tests/gsm0808/gsm0808_test.c
index ab555d2..bf979bb 100644
--- a/tests/gsm0808/gsm0808_test.c
+++ b/tests/gsm0808/gsm0808_test.c
@@ -144,6 +144,46 @@
msgb_free(msg);
 }
 
+static void test_create_cipher()
+{
+   static const uint8_t res[] =
+   { 0x00, 0x0c, 0x53, 0x0a, 0x09, 0x03, 0xaa,
+ 0xbb, 0xcc, 0xdd, 0xee, 0xff, 0x23, 0x42 };
+   static const uint8_t res2[] =
+   { 0x00, 0x0e, 0x53, 0x0a, 0x09, 0x03, 0xaa,
+ 0xbb, 0xcc, 0xdd, 0xee, 0xff, 0x23, 0x42,
+ GSM0808_IE_CIPHER_RESPONSE_MODE, 0x01 };
+   struct msgb *msg;
+   struct gsm0808_encrypt_info ei;
+   uint8_t include_imeisv;
+
+   memset(, 0, sizeof(ei));
+   ei.perm_algo[0] = GSM0808_ALG_ID_A5_0;
+   ei.perm_algo[1] = GSM0808_ALG_ID_A5_1;
+   ei.perm_algo_len = 2;
+   ei.key[0] = 0xaa;
+   ei.key[1] = 0xbb;
+   ei.key[2] = 0xcc;
+   ei.key[3] = 0xdd;
+   ei.key[4] = 0xee;
+   ei.key[5] = 0xff;
+   ei.key[6] = 0x23;
+   ei.key[7] = 0x42;
+   ei.key_len = 8;
+   include_imeisv = 1;
+
+   printf("Testing creating Chipher Mode Command\n");
+   msg = gsm0808_create_cipher(, NULL);
+   OSMO_ASSERT(msg);
+   VERIFY(msg, res, ARRAY_SIZE(res));
+   msgb_free(msg);
+
+   msg = gsm0808_create_cipher(, _imeisv);
+   OSMO_ASSERT(msg);
+   VERIFY(msg, res2, ARRAY_SIZE(res2));
+   msgb_free(msg);
+}
+
 static void test_create_cipher_complete()
 {
static const uint8_t res1[] = {
@@ -701,6 +741,7 @@
test_create_reset();
test_create_clear_command();
test_create_clear_complete();
+   

[PATCH] libosmocore[master]: gsm0808: Add create functions for CIPHER MODE COMMAND

2017-03-30 Thread dexter

Review at  https://gerrit.osmocom.org/2182

gsm0808: Add create functions for CIPHER MODE COMMAND

gsm0808.h/c lacks functionality to generate CIPHER MODE COMMAND messages. These
messages are required if the code is used in an MSC implementation.

This commit adds a gsm0808_create_cipher() function, that generates an A/AoiP
CIPHER MODE COMMAND message.

Change-Id: I8eb1c357860c3e740b0f5d17e1c256bc87920958
---
M include/osmocom/gsm/gsm0808.h
M src/gsm/gsm0808.c
M src/gsm/libosmogsm.map
M tests/gsm0808/gsm0808_test.c
M tests/gsm0808/gsm0808_test.ok
5 files changed, 86 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/82/2182/1

diff --git a/include/osmocom/gsm/gsm0808.h b/include/osmocom/gsm/gsm0808.h
index 7ef62b2..44ce763 100644
--- a/include/osmocom/gsm/gsm0808.h
+++ b/include/osmocom/gsm/gsm0808.h
@@ -34,6 +34,8 @@
 struct msgb *gsm0808_create_reset_ack(void);
 struct msgb *gsm0808_create_clear_command(uint8_t reason);
 struct msgb *gsm0808_create_clear_complete(void);
+struct msgb *gsm0808_create_cipher(struct gsm0808_encrypt_info *ei,
+  uint8_t *cipher_response_mode);
 struct msgb *gsm0808_create_cipher_complete(struct msgb *layer3, uint8_t 
alg_id);
 struct msgb *gsm0808_create_cipher_reject(uint8_t cause);
 struct msgb *gsm0808_create_classmark_update(const uint8_t *cm2, uint8_t 
cm2_len,
diff --git a/src/gsm/gsm0808.c b/src/gsm/gsm0808.c
index c37ce3e..7e91df8 100644
--- a/src/gsm/gsm0808.c
+++ b/src/gsm/gsm0808.c
@@ -140,6 +140,47 @@
return msg;
 }
 
+struct msgb *gsm0808_create_cipher(struct gsm0808_encrypt_info *ei,
+  uint8_t *cipher_response_mode)
+{
+   /* See also: 3GPP TS 48.008 3.2.1.30 CIPHER MODE COMMAND */
+   struct msgb *msg;
+   struct msgb *ei_encoded;
+
+   /* Mandatory emelent! */
+   OSMO_ASSERT(ei);
+
+   msg =
+   msgb_alloc_headroom(BSSMAP_MSG_SIZE, BSSMAP_MSG_HEADROOM,
+   "cipher-mode-command");
+   if (!msg)
+   return NULL;
+
+   /* Message Type 3.2.2.1 */
+   msgb_v_put(msg, BSS_MAP_MSG_CIPHER_MODE_CMD);
+
+   /* Encryption Information 3.2.2.10 */
+   ei_encoded = gsm0808_enc_encrypt_info(ei);
+   if (!ei_encoded) {
+   msgb_free(msg);
+   return NULL;
+   }
+   msgb_tlv_put(msg, GSM0808_IE_ENCRYPTION_INFORMATION,
+ei_encoded->len, ei_encoded->data);
+   msgb_free(ei_encoded);
+
+   /* Cipher Response Mode 3.2.2.34 */
+   if (cipher_response_mode)
+   msgb_tv_put(msg, GSM0808_IE_CIPHER_RESPONSE_MODE,
+   *cipher_response_mode);
+
+   /* pre-pend the header */
+   msg->l3h =
+   msgb_tv_push(msg, BSSAP_MSG_BSS_MANAGEMENT, msgb_length(msg));
+
+   return msg;
+}
+
 struct msgb *gsm0808_create_cipher_complete(struct msgb *layer3, uint8_t 
alg_id)
 {
struct msgb *msg = msgb_alloc_headroom(BSSMAP_MSG_SIZE, 
BSSMAP_MSG_HEADROOM,
diff --git a/src/gsm/libosmogsm.map b/src/gsm/libosmogsm.map
index ac8d467..2826cd8 100644
--- a/src/gsm/libosmogsm.map
+++ b/src/gsm/libosmogsm.map
@@ -128,6 +128,7 @@
 gsm0808_create_assignment_completed_aoip;
 gsm0808_create_assignment_failure;
 gsm0808_create_assignment_failure_aoip;
+gsm0808_create_cipher;
 gsm0808_create_cipher_complete;
 gsm0808_create_cipher_reject;
 gsm0808_create_classmark_update;
diff --git a/tests/gsm0808/gsm0808_test.c b/tests/gsm0808/gsm0808_test.c
index c7e0d18..b3f1f40 100644
--- a/tests/gsm0808/gsm0808_test.c
+++ b/tests/gsm0808/gsm0808_test.c
@@ -162,6 +162,46 @@
msgb_free(msg);
 }
 
+static void test_create_cipher()
+{
+   static const uint8_t res[] =
+   { 0x00, 0x0c, 0x53, 0x0a, 0x09, 0x03, 0xaa,
+ 0xbb, 0xcc, 0xdd, 0xee, 0xff, 0x23, 0x42 };
+   static const uint8_t res2[] =
+   { 0x00, 0x0e, 0x53, 0x0a, 0x09, 0x03, 0xaa,
+ 0xbb, 0xcc, 0xdd, 0xee, 0xff, 0x23, 0x42,
+ GSM0808_IE_CIPHER_RESPONSE_MODE, 0x01 };
+   struct msgb *msg;
+   struct gsm0808_encrypt_info ei;
+   uint8_t include_imeisv;
+
+   memset(, 0, sizeof(ei));
+   ei.perm_algo[0] = GSM0808_ALG_ID_A5_0;
+   ei.perm_algo[1] = GSM0808_ALG_ID_A5_1;
+   ei.perm_algo_len = 2;
+   ei.key[0] = 0xaa;
+   ei.key[1] = 0xbb;
+   ei.key[2] = 0xcc;
+   ei.key[3] = 0xdd;
+   ei.key[4] = 0xee;
+   ei.key[5] = 0xff;
+   ei.key[6] = 0x23;
+   ei.key[7] = 0x42;
+   ei.key_len = 8;
+   include_imeisv = 1;
+
+   printf("Testing creating Chipher Mode Command\n");
+   msg = gsm0808_create_cipher(, NULL);
+   OSMO_ASSERT(msg);
+   VERIFY(msg, res, ARRAY_SIZE(res));
+   msgb_free(msg);
+
+   msg = gsm0808_create_cipher(, _imeisv);
+   OSMO_ASSERT(msg);
+   VERIFY(msg, res2, ARRAY_SIZE(res2));
+   msgb_free(msg);
+}
+
 static void test_create_cipher_complete()
 {