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, next has been updated via 05815782de21ea4086842131ec8b351ce0299d1d (commit) via e8199256e7dec948f6b13269d297acbcaee4b731 (commit) from eb600e3f40fe95b986ddee1ba80faed7c0953445 (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 05815782de21ea4086842131ec8b351ce0299d1d 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 e8199256e7dec948f6b13269d297acbcaee4b731 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