diff options
| author | Maciej Czekaj <mczekaj@marvell.com> | 2020-05-13 16:13:39 +0200 |
|---|---|---|
| committer | Matias Elo <matias.elo@nokia.com> | 2020-06-16 21:45:14 +0300 |
| commit | 245dda8129b26aa66f7ff56a3ccb7a989f8792b5 (patch) | |
| tree | 551903d0e107399b8f91a885a7537ceee1fa7fc3 | |
| parent | a651f63fd25969806e178da4618bdbdaf6411a5d (diff) | |
validation: fix compiler warnings
Some ODP APIs do not specify what to do with output arguments if the call
fails. Implementation may be free to leave them uninitialized. Therefore
make sure output parameters are initialized at the start of test
function.
Signed-off-by: Maciej Czekaj <mczekaj@marvell.com>
Signed-off-by: Stanislaw Kardach <skardach@marvell.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
| -rw-r--r-- | test/validation/api/packet/packet.c | 4 | ||||
| -rw-r--r-- | test/validation/api/pktio/pktio.c | 2 | ||||
| -rw-r--r-- | test/validation/api/pool/pool.c | 3 | ||||
| -rw-r--r-- | test/validation/api/queue/queue.c | 4 |
4 files changed, 8 insertions, 5 deletions
diff --git a/test/validation/api/packet/packet.c b/test/validation/api/packet/packet.c index 6026ca32a..51d251630 100644 --- a/test/validation/api/packet/packet.c +++ b/test/validation/api/packet/packet.c @@ -862,7 +862,7 @@ static void _verify_headroom_shift(odp_packet_t *pkt, uint32_t room = odp_packet_headroom(*pkt); uint32_t seg_data_len = odp_packet_seg_len(*pkt); uint32_t pkt_data_len = odp_packet_len(*pkt); - void *data; + void *data = NULL; char *data_orig = odp_packet_data(*pkt); char *head_orig = odp_packet_head(*pkt); uint32_t seg_len; @@ -1571,7 +1571,7 @@ static void packet_test_concatsplit(void) { odp_packet_t pkt, pkt2; uint32_t pkt_len; - odp_packet_t splits[4]; + odp_packet_t splits[4] = {ODP_PACKET_INVALID}; odp_pool_t pool; pool = odp_packet_pool(test_packet); diff --git a/test/validation/api/pktio/pktio.c b/test/validation/api/pktio/pktio.c index 20c33b461..b40987aae 100644 --- a/test/validation/api/pktio/pktio.c +++ b/test/validation/api/pktio/pktio.c @@ -649,7 +649,7 @@ static int recv_packets_tmo(odp_pktio_t pktio, odp_packet_t pkt_tbl[], int num_q; int i; int n; - unsigned from_val; + unsigned int from_val = 0; unsigned *from = NULL; if (mode == RECV_MQ_TMO) diff --git a/test/validation/api/pool/pool.c b/test/validation/api/pool/pool.c index 24b47d4fb..7ac92ff0e 100644 --- a/test/validation/api/pool/pool.c +++ b/test/validation/api/pool/pool.c @@ -83,6 +83,7 @@ static void pool_test_lookup_info_print(void) odp_pool_info_t info; odp_pool_param_t param; + memset(&info, 0, sizeof(info)); odp_pool_param_init(¶m); param.type = ODP_POOL_BUFFER; @@ -321,6 +322,7 @@ static void pool_test_info_packet(void) odp_pool_param_t param; const char pool_name[] = "test_pool_name"; + memset(&info, 0, sizeof(info)); odp_pool_param_init(¶m); param.type = ODP_POOL_PACKET; @@ -351,6 +353,7 @@ static void pool_test_info_data_range(void) uint32_t i, num; uintptr_t pool_len; + memset(&info, 0, sizeof(info)); odp_pool_param_init(¶m); param.type = ODP_POOL_PACKET; diff --git a/test/validation/api/queue/queue.c b/test/validation/api/queue/queue.c index 9b4ebd44c..a5ec628e4 100644 --- a/test/validation/api/queue/queue.c +++ b/test/validation/api/queue/queue.c @@ -580,8 +580,8 @@ static void queue_test_pair_lf_spsc(void) static void queue_test_param(void) { odp_queue_t queue, null_queue; - odp_event_t enev[BURST_SIZE]; - odp_event_t deev[BURST_SIZE]; + odp_event_t enev[BURST_SIZE] = {ODP_EVENT_INVALID}; + odp_event_t deev[BURST_SIZE] = {ODP_EVENT_INVALID}; odp_buffer_t buf; odp_event_t ev; odp_pool_t msg_pool; |