diff mbox

validation: packet: do assert on newly created pkt reference

Message ID 1490775317-3699-1-git-send-email-balakrishna.garapati@linaro.org
State Accepted
Commit ce61337e6a7e955913fa64412c75d2066e0f121c
Headers show

Commit Message

Balakrishna Garapati March 29, 2017, 8:15 a.m. UTC
Signed-off-by: Balakrishna Garapati <balakrishna.garapati@linaro.org>

---
 test/common_plat/validation/api/packet/packet.c | 3 +++
 1 file changed, 3 insertions(+)

-- 
1.9.1

Comments

Bill Fischofer March 29, 2017, 5:42 p.m. UTC | #1
On Wed, Mar 29, 2017 at 3:15 AM, Balakrishna Garapati
<balakrishna.garapati@linaro.org> wrote:
> Signed-off-by: Balakrishna Garapati <balakrishna.garapati@linaro.org>


Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org>


> ---

>  test/common_plat/validation/api/packet/packet.c | 3 +++

>  1 file changed, 3 insertions(+)

>

> diff --git a/test/common_plat/validation/api/packet/packet.c b/test/common_plat/validation/api/packet/packet.c

> index 669122a..2ffd924 100644

> --- a/test/common_plat/validation/api/packet/packet.c

> +++ b/test/common_plat/validation/api/packet/packet.c

> @@ -2234,12 +2234,15 @@ void packet_test_ref(void)

>

>         /* Create references */

>         ref_pkt[0] = odp_packet_ref(segmented_base_pkt, offset[0]);

> +       CU_ASSERT_FATAL(ref_pkt[0] != ODP_PACKET_INVALID);

> +

>         if (odp_packet_has_ref(ref_pkt[0]) == 1) {

>                 /* CU_ASSERT needs braces */

>                 CU_ASSERT(odp_packet_has_ref(segmented_base_pkt) == 1);

>         }

>

>         ref_pkt[1] = odp_packet_ref(segmented_base_pkt, offset[1]);

> +       CU_ASSERT_FATAL(ref_pkt[1] != ODP_PACKET_INVALID);

>

>         if (odp_packet_has_ref(ref_pkt[1]) == 1) {

>                 /* CU_ASSERT needs braces */

> --

> 1.9.1

>
diff mbox

Patch

diff --git a/test/common_plat/validation/api/packet/packet.c b/test/common_plat/validation/api/packet/packet.c
index 669122a..2ffd924 100644
--- a/test/common_plat/validation/api/packet/packet.c
+++ b/test/common_plat/validation/api/packet/packet.c
@@ -2234,12 +2234,15 @@  void packet_test_ref(void)
 
 	/* Create references */
 	ref_pkt[0] = odp_packet_ref(segmented_base_pkt, offset[0]);
+	CU_ASSERT_FATAL(ref_pkt[0] != ODP_PACKET_INVALID);
+
 	if (odp_packet_has_ref(ref_pkt[0]) == 1) {
 		/* CU_ASSERT needs braces */
 		CU_ASSERT(odp_packet_has_ref(segmented_base_pkt) == 1);
 	}
 
 	ref_pkt[1] = odp_packet_ref(segmented_base_pkt, offset[1]);
+	CU_ASSERT_FATAL(ref_pkt[1] != ODP_PACKET_INVALID);
 
 	if (odp_packet_has_ref(ref_pkt[1]) == 1) {
 		/* CU_ASSERT needs braces */