diff options
| author | Jukka Rissanen <jukka.rissanen@linux.intel.com> | 2017-03-10 10:37:45 +0200 |
|---|---|---|
| committer | Anas Nashif <nashif@linux.intel.com> | 2017-03-10 10:10:21 +0000 |
| commit | d031cd35c444459b6220a06cc4d878b6cc6c19e5 (patch) | |
| tree | 9c060ee880dee026d52e4972b4a7ce282c2362a5 | |
| parent | 9be5966dc92ca2e6300d6f4d7010e3ccc1b1aa64 (diff) | |
net: tests: nbuf: Fix compiler warnings
LLVM/icx compiler gives warnings for uint16_t vs. size_t
values.
Jira: ZEP-1886
Change-Id: I63c5deea672568946d91421a873c470af3cec6df
Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
(cherry picked from commit 22b370c19980c605b778355bb61823f1ea7448c3)
| -rw-r--r-- | tests/net/nbuf/src/main.c | 28 |
1 files changed, 16 insertions, 12 deletions
diff --git a/tests/net/nbuf/src/main.c b/tests/net/nbuf/src/main.c index 8a09f3730..b46d8dc4b 100644 --- a/tests/net/nbuf/src/main.c +++ b/tests/net/nbuf/src/main.c @@ -615,25 +615,27 @@ static int test_nbuf_read_append(void) tfrag = net_buf_frag_last(buf->frags); off = tfrag->len; - if (!net_nbuf_append(buf, sizeof(test_rw_short), test_rw_short, - K_FOREVER)) { + if (!net_nbuf_append(buf, (uint16_t)sizeof(test_rw_short), + test_rw_short, K_FOREVER)) { printk("net_nbuf_append failed\n"); return -EINVAL; } - if (!net_nbuf_append(buf, sizeof(test_rw_short), test_rw_short, - K_FOREVER)) { + if (!net_nbuf_append(buf, (uint16_t)sizeof(test_rw_short), + test_rw_short, K_FOREVER)) { printk("net_nbuf_append failed\n"); return -EINVAL; } - tfrag = net_nbuf_skip(tfrag, off, &tpos, sizeof(test_rw_short)); + tfrag = net_nbuf_skip(tfrag, off, &tpos, + (uint16_t)sizeof(test_rw_short)); if (!tfrag) { printk("net_nbuf_skip failed\n"); return -EINVAL; } - tfrag = net_nbuf_read(tfrag, tpos, &tpos, sizeof(test_rw_short), + tfrag = net_nbuf_read(tfrag, tpos, &tpos, + (uint16_t)sizeof(test_rw_short), verify_rw_short); if (memcmp(test_rw_short, verify_rw_short, sizeof(test_rw_short))) { printk("net_nbuf_read failed with mismatch data"); @@ -651,13 +653,13 @@ static int test_nbuf_read_append(void) tfrag = net_buf_frag_last(buf->frags); off = tfrag->len; - if (!net_nbuf_append(buf, sizeof(test_rw_long), test_rw_long, + if (!net_nbuf_append(buf, (uint16_t)sizeof(test_rw_long), test_rw_long, K_FOREVER)) { printk("net_nbuf_append failed\n"); return -EINVAL; } - if (!net_nbuf_append(buf, sizeof(test_rw_long), test_rw_long, + if (!net_nbuf_append(buf, (uint16_t)sizeof(test_rw_long), test_rw_long, K_FOREVER)) { printk("net_nbuf_append failed\n"); return -EINVAL; @@ -666,19 +668,21 @@ static int test_nbuf_read_append(void) /* Try to pass fragment to net_nbuf_append(), this should fail * as we always need to pass the first buf into it. */ - if (net_nbuf_append(buf->frags, sizeof(test_rw_short), test_rw_short, - K_FOREVER)) { + if (net_nbuf_append(buf->frags, (uint16_t)sizeof(test_rw_short), + test_rw_short, K_FOREVER)) { printk("net_nbuf_append succeed but should have failed\n"); return -EINVAL; } - tfrag = net_nbuf_skip(tfrag, off, &tpos, sizeof(test_rw_long)); + tfrag = net_nbuf_skip(tfrag, off, &tpos, + (uint16_t)sizeof(test_rw_long)); if (!tfrag) { printk("net_nbuf_skip failed\n"); return -EINVAL; } - tfrag = net_nbuf_read(tfrag, tpos, &tpos, sizeof(test_rw_long), + tfrag = net_nbuf_read(tfrag, tpos, &tpos, + (uint16_t)sizeof(test_rw_long), verify_rw_long); if (memcmp(test_rw_long, verify_rw_long, sizeof(test_rw_long))) { printk("net_nbuf_read failed with mismatch data"); |