This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "".
The branch, api-next has been updated via 6303c7d0e98fafe0f14c8c4dd9989b3b7633ebf4 (commit) via 5b1ec6a624c8ea4b4645ace62ba11fc6701aa5b3 (commit) from 68735b312926a44ddf42f9ecf96a0badd941a247 (commit)
Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below.
- Log ----------------------------------------------------------------- commit 6303c7d0e98fafe0f14c8c4dd9989b3b7633ebf4 Author: Dmitry Eremin-Solenikov dmitry.ereminsolenikov@linaro.org Date: Thu Dec 28 13:03:21 2017 +0300
validation: pktio: clean shutdown in case pkt sending failed
Provide clean shutdown path for the case of odp_pktout_send() returning an error during pktio_test_send_failure test.
Signed-off-by: Dmitry Eremin-Solenikov dmitry.ereminsolenikov@linaro.org Reviewed-by: Bill Fischofer bill.fischofer@linaro.org Signed-off-by: Maxim Uvarov maxim.uvarov@linaro.org
diff --git a/test/validation/api/pktio/pktio.c b/test/validation/api/pktio/pktio.c index a6820a1d..a2364103 100644 --- a/test/validation/api/pktio/pktio.c +++ b/test/validation/api/pktio/pktio.c @@ -1846,7 +1846,9 @@ void pktio_test_send_failure(void) * the initial short packets should be sent successfully */ odp_errno_zero(); ret = odp_pktout_send(pktout, pkt_tbl, TX_BATCH_LEN); - CU_ASSERT_FATAL(ret == long_pkt_idx); + CU_ASSERT(ret == long_pkt_idx); + if (ret != long_pkt_idx) + goto cleanup; CU_ASSERT(odp_errno() == 0);
info_rx.id = pktio_rx; @@ -1897,6 +1899,7 @@ void pktio_test_send_failure(void) odp_packet_free(pkt_tbl[i]); }
+cleanup: if (pktio_rx != pktio_tx) { CU_ASSERT(odp_pktio_stop(pktio_rx) == 0); CU_ASSERT(odp_pktio_close(pktio_rx) == 0);
commit 5b1ec6a624c8ea4b4645ace62ba11fc6701aa5b3 Author: Dmitry Eremin-Solenikov dmitry.ereminsolenikov@linaro.org Date: Thu Dec 28 13:03:21 2017 +0300
validation: crypto: clean shutdown in case pkt allocation failed
Provide clean shutdown path for the case of odp_packet_alloc() returning ODP_PACKET_INVALID.
Signed-off-by: Dmitry Eremin-Solenikov dmitry.ereminsolenikov@linaro.org Reviewed-by: Bill Fischofer bill.fischofer@linaro.org Signed-off-by: Maxim Uvarov maxim.uvarov@linaro.org
diff --git a/test/validation/api/crypto/odp_crypto_test_inp.c b/test/validation/api/crypto/odp_crypto_test_inp.c index f1da6989..821bdc7e 100644 --- a/test/validation/api/crypto/odp_crypto_test_inp.c +++ b/test/validation/api/crypto/odp_crypto_test_inp.c @@ -458,6 +458,8 @@ static void alg_test(odp_crypto_op_t op, odp_packet_t pkt = odp_packet_alloc(suite_context.pool, ref->length + ref->digest_length); CU_ASSERT(pkt != ODP_PACKET_INVALID); + if (pkt == ODP_PACKET_INVALID) + goto cleanup;
for (iteration = NORMAL_TEST; iteration < MAX_TEST; iteration++) { /* checking against wrong digest is meaningless for NULL digest @@ -523,10 +525,11 @@ static void alg_test(odp_crypto_op_t op, } }
+ odp_packet_free(pkt); + +cleanup: rc = odp_crypto_session_destroy(session); CU_ASSERT(!rc); - - odp_packet_free(pkt); }
/**
-----------------------------------------------------------------------
Summary of changes: test/validation/api/crypto/odp_crypto_test_inp.c | 7 +++++-- test/validation/api/pktio/pktio.c | 5 ++++- 2 files changed, 9 insertions(+), 3 deletions(-)
hooks/post-receive