test: pktio: correct mac addr len check

Message ID	1466439518-6539-1-git-send-email-maxim.uvarov@linaro.org
State	Accepted
Commit	08487413ead04cb5fef8e813af06e5bc29207334
Headers	show Delivered-To: patch@linaro.org Received-SPF: pass (google.com: domain of lng-odp-bounces@lists.linaro.org designates 54.225.227.206 as permitted sender) client-ip=54.225.227.206; From: Maxim Uvarov <maxim.uvarov@linaro.org> To: lng-odp@lists.linaro.org Date: Mon, 20 Jun 2016 19:18:38 +0300 Message-Id: <1466439518-6539-1-git-send-email-maxim.uvarov@linaro.org> Subject: [lng-odp] [PATCH] test: pktio: correct mac addr len check Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: lng-odp-bounces@lists.linaro.org Sender: "lng-odp" <lng-odp-bounces@lists.linaro.org>

Message ID

1466439518-6539-1-git-send-email-maxim.uvarov@linaro.org

State

Accepted

Commit

08487413ead04cb5fef8e813af06e5bc29207334

Headers

Received-SPF: pass (google.com: domain of lng-odp-bounces@lists.linaro.org
	designates 54.225.227.206 as permitted sender)
	client-ip=54.225.227.206; 
From: Maxim Uvarov <maxim.uvarov@linaro.org>
To: lng-odp@lists.linaro.org
Date: Mon, 20 Jun 2016 19:18:38 +0300
Message-Id: <1466439518-6539-1-git-send-email-maxim.uvarov@linaro.org>
Subject: [lng-odp] [PATCH] test: pktio: correct mac addr len check
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
Errors-To: lng-odp-bounces@lists.linaro.org
Sender: "lng-odp" <lng-odp-bounces@lists.linaro.org>

Comments

Bill Fischofer June 23, 2016, 10:38 a.m. UTC | #1

On Mon, Jun 20, 2016 at 11:18 PM, Maxim Uvarov <maxim.uvarov@linaro.org>
wrote:

> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
>

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


> ---
>  test/validation/pktio/pktio.c | 14 +++++++++-----
>  1 file changed, 9 insertions(+), 5 deletions(-)
>
> diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c
> index 5922ba5..d53a419 100644
> --- a/test/validation/pktio/pktio.c
> +++ b/test/validation/pktio/pktio.c
> @@ -143,11 +143,13 @@ static void pktio_pkt_set_macs(odp_packet_t pkt,
>         odph_ethhdr_t *eth = (odph_ethhdr_t *)odp_packet_l2_ptr(pkt, &len);
>         int ret;
>
> -       ret = odp_pktio_mac_addr(src, &eth->src, sizeof(eth->src));
> +       ret = odp_pktio_mac_addr(src, &eth->src,
> ODP_PKTIO_MACADDR_MAXSIZE);
>         CU_ASSERT(ret == ODPH_ETHADDR_LEN);
> +       CU_ASSERT(ret <= ODP_PKTIO_MACADDR_MAXSIZE);
>
> -       ret = odp_pktio_mac_addr(dst, &eth->dst, sizeof(eth->dst));
> +       ret = odp_pktio_mac_addr(dst, &eth->dst,
> ODP_PKTIO_MACADDR_MAXSIZE);
>         CU_ASSERT(ret == ODPH_ETHADDR_LEN);
> +       CU_ASSERT(ret <= ODP_PKTIO_MACADDR_MAXSIZE);
>  }
>
>  static uint32_t pktio_pkt_set_seq(odp_packet_t pkt)
> @@ -239,7 +241,7 @@ static uint32_t pktio_init_packet(odp_packet_t pkt)
>         odph_udphdr_t *udp;
>         char *buf;
>         uint16_t seq;
> -       uint8_t mac[ODPH_ETHADDR_LEN] = {0};
> +       uint8_t mac[ODP_PKTIO_MACADDR_MAXSIZE] = {0};
>         int pkt_len = odp_packet_len(pkt);
>
>         buf = odp_packet_data(pkt);
> @@ -1036,7 +1038,7 @@ void pktio_test_promisc(void)
>
>  void pktio_test_mac(void)
>  {
> -       unsigned char mac_addr[ODPH_ETHADDR_LEN];
> +       unsigned char mac_addr[ODP_PKTIO_MACADDR_MAXSIZE];
>         int mac_len;
>         int ret;
>         odp_pktio_t pktio;
> @@ -1047,8 +1049,10 @@ void pktio_test_mac(void)
>
>         printf("testing mac for %s\n", iface_name[0]);
>
> -       mac_len = odp_pktio_mac_addr(pktio, mac_addr, sizeof(mac_addr));
> +       mac_len = odp_pktio_mac_addr(pktio, mac_addr,
> +                                    ODP_PKTIO_MACADDR_MAXSIZE);
>         CU_ASSERT(ODPH_ETHADDR_LEN == mac_len);
> +       CU_ASSERT(ODP_PKTIO_MACADDR_MAXSIZE >= mac_len);
>
>         printf(" %X:%X:%X:%X:%X:%X ",
>                mac_addr[0], mac_addr[1], mac_addr[2],
> --
> 2.7.1.250.gff4ea60
>
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> https://lists.linaro.org/mailman/listinfo/lng-odp
>

Maxim Uvarov June 23, 2016, 3:23 p.m. UTC | #2

Merged,
Maxim.

On 06/23/16 13:38, Bill Fischofer wrote:
>
>
> On Mon, Jun 20, 2016 at 11:18 PM, Maxim Uvarov 
> <maxim.uvarov@linaro.org <mailto:maxim.uvarov@linaro.org>> wrote:
>
>     Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org
>     <mailto:maxim.uvarov@linaro.org>>
>
>
> Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org 
> <mailto:bill.fischofer@linaro.org>>
>
>     ---
>      test/validation/pktio/pktio.c | 14 +++++++++-----
>      1 file changed, 9 insertions(+), 5 deletions(-)
>
>     diff --git a/test/validation/pktio/pktio.c
>     b/test/validation/pktio/pktio.c
>     index 5922ba5..d53a419 100644
>     --- a/test/validation/pktio/pktio.c
>     +++ b/test/validation/pktio/pktio.c
>     @@ -143,11 +143,13 @@ static void pktio_pkt_set_macs(odp_packet_t pkt,
>             odph_ethhdr_t *eth = (odph_ethhdr_t
>     *)odp_packet_l2_ptr(pkt, &len);
>             int ret;
>
>     -       ret = odp_pktio_mac_addr(src, &eth->src, sizeof(eth->src));
>     +       ret = odp_pktio_mac_addr(src, &eth->src,
>     ODP_PKTIO_MACADDR_MAXSIZE);
>             CU_ASSERT(ret == ODPH_ETHADDR_LEN);
>     +       CU_ASSERT(ret <= ODP_PKTIO_MACADDR_MAXSIZE);
>
>     -       ret = odp_pktio_mac_addr(dst, &eth->dst, sizeof(eth->dst));
>     +       ret = odp_pktio_mac_addr(dst, &eth->dst,
>     ODP_PKTIO_MACADDR_MAXSIZE);
>             CU_ASSERT(ret == ODPH_ETHADDR_LEN);
>     +       CU_ASSERT(ret <= ODP_PKTIO_MACADDR_MAXSIZE);
>      }
>
>      static uint32_t pktio_pkt_set_seq(odp_packet_t pkt)
>     @@ -239,7 +241,7 @@ static uint32_t pktio_init_packet(odp_packet_t
>     pkt)
>             odph_udphdr_t *udp;
>             char *buf;
>             uint16_t seq;
>     -       uint8_t mac[ODPH_ETHADDR_LEN] = {0};
>     +       uint8_t mac[ODP_PKTIO_MACADDR_MAXSIZE] = {0};
>             int pkt_len = odp_packet_len(pkt);
>
>             buf = odp_packet_data(pkt);
>     @@ -1036,7 +1038,7 @@ void pktio_test_promisc(void)
>
>      void pktio_test_mac(void)
>      {
>     -       unsigned char mac_addr[ODPH_ETHADDR_LEN];
>     +       unsigned char mac_addr[ODP_PKTIO_MACADDR_MAXSIZE];
>             int mac_len;
>             int ret;
>             odp_pktio_t pktio;
>     @@ -1047,8 +1049,10 @@ void pktio_test_mac(void)
>
>             printf("testing mac for %s\n", iface_name[0]);
>
>     -       mac_len = odp_pktio_mac_addr(pktio, mac_addr,
>     sizeof(mac_addr));
>     +       mac_len = odp_pktio_mac_addr(pktio, mac_addr,
>     + ODP_PKTIO_MACADDR_MAXSIZE);
>             CU_ASSERT(ODPH_ETHADDR_LEN == mac_len);
>     +       CU_ASSERT(ODP_PKTIO_MACADDR_MAXSIZE >= mac_len);
>
>             printf(" %X:%X:%X:%X:%X:%X ",
>                    mac_addr[0], mac_addr[1], mac_addr[2],
>     --
>     2.7.1.250.gff4ea60
>
>     _______________________________________________
>     lng-odp mailing list
>     lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
>     https://lists.linaro.org/mailman/listinfo/lng-odp
>
>

diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c
index 5922ba5..d53a419 100644
--- a/test/validation/pktio/pktio.c
+++ b/test/validation/pktio/pktio.c
@@ -143,11 +143,13 @@  static void pktio_pkt_set_macs(odp_packet_t pkt,
 	odph_ethhdr_t *eth = (odph_ethhdr_t *)odp_packet_l2_ptr(pkt, &len);
 	int ret;
 
-	ret = odp_pktio_mac_addr(src, &eth->src, sizeof(eth->src));
+	ret = odp_pktio_mac_addr(src, &eth->src, ODP_PKTIO_MACADDR_MAXSIZE);
 	CU_ASSERT(ret == ODPH_ETHADDR_LEN);
+	CU_ASSERT(ret <= ODP_PKTIO_MACADDR_MAXSIZE);
 
-	ret = odp_pktio_mac_addr(dst, &eth->dst, sizeof(eth->dst));
+	ret = odp_pktio_mac_addr(dst, &eth->dst, ODP_PKTIO_MACADDR_MAXSIZE);
 	CU_ASSERT(ret == ODPH_ETHADDR_LEN);
+	CU_ASSERT(ret <= ODP_PKTIO_MACADDR_MAXSIZE);
 }
 
 static uint32_t pktio_pkt_set_seq(odp_packet_t pkt)
@@ -239,7 +241,7 @@  static uint32_t pktio_init_packet(odp_packet_t pkt)
 	odph_udphdr_t *udp;
 	char *buf;
 	uint16_t seq;
-	uint8_t mac[ODPH_ETHADDR_LEN] = {0};
+	uint8_t mac[ODP_PKTIO_MACADDR_MAXSIZE] = {0};
 	int pkt_len = odp_packet_len(pkt);
 
 	buf = odp_packet_data(pkt);
@@ -1036,7 +1038,7 @@  void pktio_test_promisc(void)
 
 void pktio_test_mac(void)
 {
-	unsigned char mac_addr[ODPH_ETHADDR_LEN];
+	unsigned char mac_addr[ODP_PKTIO_MACADDR_MAXSIZE];
 	int mac_len;
 	int ret;
 	odp_pktio_t pktio;
@@ -1047,8 +1049,10 @@  void pktio_test_mac(void)
 
 	printf("testing mac for %s\n", iface_name[0]);
 
-	mac_len = odp_pktio_mac_addr(pktio, mac_addr, sizeof(mac_addr));
+	mac_len = odp_pktio_mac_addr(pktio, mac_addr,
+				     ODP_PKTIO_MACADDR_MAXSIZE);
 	CU_ASSERT(ODPH_ETHADDR_LEN == mac_len);
+	CU_ASSERT(ODP_PKTIO_MACADDR_MAXSIZE >= mac_len);
 
 	printf(" %X:%X:%X:%X:%X:%X ",
 	       mac_addr[0], mac_addr[1], mac_addr[2],

test: pktio: correct mac addr len check

Commit Message

Comments

Patch