2016-03-11 16:17:38

by Szymon Janc

Subject: [PATCH] unit/test-crypto: Add test based on sample data from errata 4243

---
unit/test-crypto.c | 39 ++++++++++++++++++++++++++++++++++-----
1 file changed, 34 insertions(+), 5 deletions(-)

diff --git a/unit/test-crypto.c b/unit/test-crypto.c
index 5fb1c64..8d65707 100644
--- a/unit/test-crypto.c
+++ b/unit/test-crypto.c
@@ -38,6 +38,8 @@ struct test_data {
const uint8_t *msg;
uint16_t msg_len;
const uint8_t *t;
+ const uint8_t *key;
+ uint32_t cnt;
};

static const uint8_t key[] = {
@@ -54,7 +56,8 @@ static const uint8_t t_msg_1[] = {
static const struct test_data test_data_1 = {
.msg = msg_1,
.msg_len = 0,
- .t = t_msg_1
+ .t = t_msg_1,
+ .key = key,
};

static const uint8_t msg_2[] = {
@@ -70,7 +73,8 @@ static const uint8_t t_msg_2[] = {
static const struct test_data test_data_2 = {
.msg = msg_2,
.msg_len = 16,
- .t = t_msg_2
+ .t = t_msg_2,
+ .key = key,
};

static const uint8_t msg_3[] = {
@@ -87,7 +91,8 @@ static const uint8_t t_msg_3[12] = {
static const struct test_data test_data_3 = {
.msg = msg_3,
.msg_len = 40,
- .t = t_msg_3
+ .t = t_msg_3,
+ .key = key,
};

static const uint8_t msg_4[] = {
@@ -106,7 +111,30 @@ static const uint8_t t_msg_4[12] = {
static const struct test_data test_data_4 = {
.msg = msg_4,
.msg_len = 64,
- .t = t_msg_4
+ .t = t_msg_4,
+ .key = key,
+};
+
+static const uint8_t msg_5[] = {
+ 0xd2, 0x12, 0x00, 0x13, 0x37
+};
+
+static const uint8_t key_5[] = {
+ 0x50, 0x5E, 0x42, 0xDF, 0x96, 0x91, 0xEC, 0x72, 0xD3, 0x1F,
+ 0xCD, 0xFB, 0xEB, 0x64, 0x1B, 0x61
+};
+
+static const uint8_t t_msg_5[] = {
+ 0x01, 0x00, 0x00, 0x00, 0xF1, 0x87, 0x1E, 0x93, 0x3C, 0x90,
+ 0x0F, 0xf2
+};
+
+static const struct test_data test_data_5 = {
+ .msg = msg_5,
+ .msg_len = sizeof(msg_5),
+ .t = t_msg_5,
+ .cnt = 1,
+ .key = key_5,
};

static void print_debug(const char *str, void *user_data)
@@ -130,7 +158,7 @@ static void test_sign(gconstpointer data)
const struct test_data *d = data;

memset(t, 0, 12);
- if (!bt_crypto_sign_att(crypto, key, d->msg, d->msg_len, 0, t))
+ if (!bt_crypto_sign_att(crypto, d->key, d->msg, d->msg_len, d->cnt, t))
g_assert(true);

tester_debug("Result T:");
@@ -157,6 +185,7 @@ int main(int argc, char *argv[])
tester_add("/crypto/sign_att_2", &test_data_2, NULL, test_sign, NULL);
tester_add("/crypto/sign_att_3", &test_data_3, NULL, test_sign, NULL);
tester_add("/crypto/sign_att_4", &test_data_4, NULL, test_sign, NULL);
+ tester_add("/crypto/sign_att_5", &test_data_5, NULL, test_sign, NULL);

exit_status = tester_run();

--
2.6.2

2016-03-18 10:04:21

by Szymon Janc

Subject: Re: [PATCH] unit/test-crypto: Add test based on sample data from errata 4243

On Friday 11 March 2016 17:17:38 Szymon Janc wrote:
> ---
> unit/test-crypto.c | 39 ++++++++++++++++++++++++++++++++++-----
> 1 file changed, 34 insertions(+), 5 deletions(-)
>
> diff --git a/unit/test-crypto.c b/unit/test-crypto.c
> index 5fb1c64..8d65707 100644
> --- a/unit/test-crypto.c
> +++ b/unit/test-crypto.c
> @@ -38,6 +38,8 @@ struct test_data {
> const uint8_t *msg;
> uint16_t msg_len;
> const uint8_t *t;
> + const uint8_t *key;
> + uint32_t cnt;
> };
>
> static const uint8_t key[] = {
> @@ -54,7 +56,8 @@ static const uint8_t t_msg_1[] = {
> static const struct test_data test_data_1 = {
> .msg = msg_1,
> .msg_len = 0,
> - .t = t_msg_1
> + .t = t_msg_1,
> + .key = key,
> };
>
> static const uint8_t msg_2[] = {
> @@ -70,7 +73,8 @@ static const uint8_t t_msg_2[] = {
> static const struct test_data test_data_2 = {
> .msg = msg_2,
> .msg_len = 16,
> - .t = t_msg_2
> + .t = t_msg_2,
> + .key = key,
> };
>
> static const uint8_t msg_3[] = {
> @@ -87,7 +91,8 @@ static const uint8_t t_msg_3[12] = {
> static const struct test_data test_data_3 = {
> .msg = msg_3,
> .msg_len = 40,
> - .t = t_msg_3
> + .t = t_msg_3,
> + .key = key,
> };
>
> static const uint8_t msg_4[] = {
> @@ -106,7 +111,30 @@ static const uint8_t t_msg_4[12] = {
> static const struct test_data test_data_4 = {
> .msg = msg_4,
> .msg_len = 64,
> - .t = t_msg_4
> + .t = t_msg_4,
> + .key = key,
> +};
> +
> +static const uint8_t msg_5[] = {
> + 0xd2, 0x12, 0x00, 0x13, 0x37
> +};
> +
> +static const uint8_t key_5[] = {
> + 0x50, 0x5E, 0x42, 0xDF, 0x96, 0x91, 0xEC, 0x72, 0xD3, 0x1F,
> + 0xCD, 0xFB, 0xEB, 0x64, 0x1B, 0x61
> +};
> +
> +static const uint8_t t_msg_5[] = {
> + 0x01, 0x00, 0x00, 0x00, 0xF1, 0x87, 0x1E, 0x93, 0x3C, 0x90,
> + 0x0F, 0xf2
> +};
> +
> +static const struct test_data test_data_5 = {
> + .msg = msg_5,
> + .msg_len = sizeof(msg_5),
> + .t = t_msg_5,
> + .cnt = 1,
> + .key = key_5,
> };
>
> static void print_debug(const char *str, void *user_data)
> @@ -130,7 +158,7 @@ static void test_sign(gconstpointer data)
> const struct test_data *d = data;
>
> memset(t, 0, 12);
> - if (!bt_crypto_sign_att(crypto, key, d->msg, d->msg_len, 0, t))
> + if (!bt_crypto_sign_att(crypto, d->key, d->msg, d->msg_len, d->cnt, t))
> g_assert(true);
>
> tester_debug("Result T:");
> @@ -157,6 +185,7 @@ int main(int argc, char *argv[])
> tester_add("/crypto/sign_att_2", &test_data_2, NULL, test_sign, NULL);
> tester_add("/crypto/sign_att_3", &test_data_3, NULL, test_sign, NULL);
> tester_add("/crypto/sign_att_4", &test_data_4, NULL, test_sign, NULL);
> + tester_add("/crypto/sign_att_5", &test_data_5, NULL, test_sign, NULL);
>
> exit_status = tester_run();

Applied.

--
pozdrawiam
Szymon Janc