selftests/bpf: Change type from void * to struct generic_data *
authorBjörn Töpel <bjorn.topel@intel.com>
Fri, 22 Jan 2021 15:47:21 +0000 (16:47 +0100)
committerDaniel Borkmann <daniel@iogearbox.net>
Mon, 25 Jan 2021 23:05:02 +0000 (00:05 +0100)
Instead of casting from void *, let us use the actual type in
gen_udp_hdr().

Signed-off-by: Björn Töpel <bjorn.topel@intel.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Link: https://lore.kernel.org/bpf/20210122154725.22140-9-bjorn.topel@gmail.com
tools/testing/selftests/bpf/xdpxceiver.c

index c2cfc0b6d19ea31301c53b3f6cf2bb2cb1e62d9f..993ce9b7aa76e98b2d6ef4aa2713f450007d235c 100644 (file)
@@ -246,13 +246,13 @@ static void gen_ip_hdr(struct ifobject *ifobject, struct iphdr *ip_hdr)
        ip_hdr->check = 0;
 }
 
-static void gen_udp_hdr(void *data, struct ifobject *ifobject, struct udphdr *udp_hdr)
+static void gen_udp_hdr(struct generic_data *data, struct ifobject *ifobject,
+                       struct udphdr *udp_hdr)
 {
        udp_hdr->source = htons(ifobject->src_port);
        udp_hdr->dest = htons(ifobject->dst_port);
        udp_hdr->len = htons(UDP_PKT_SIZE);
-       memset32_htonl(pkt_data + PKT_HDR_SIZE,
-                      htonl(((struct generic_data *)data)->seqnum), UDP_PKT_DATA_SIZE);
+       memset32_htonl(pkt_data + PKT_HDR_SIZE, htonl(data->seqnum), UDP_PKT_DATA_SIZE);
 }
 
 static void gen_udp_csum(struct udphdr *udp_hdr, struct iphdr *ip_hdr)
@@ -841,7 +841,7 @@ static void *worker_testapp_validate(void *arg)
                                data->seqnum = -1;
                        else
                                data->seqnum = i;
-                       gen_udp_hdr((void *)data, ifobject, udp_hdr);
+                       gen_udp_hdr(data, ifobject, udp_hdr);
                        gen_ip_hdr(ifobject, ip_hdr);
                        gen_udp_csum(udp_hdr, ip_hdr);
                        gen_eth_hdr(ifobject, eth_hdr);