Commit fb18a6e2 authored by Martin Åberg's avatar Martin Åberg Committed by Jukka Rissanen
Browse files

net: ipv6_fragment: fixed endian issue 



A cast made assumption on how data is stored.

This commit makes the following test pass on qemu_leon3:
- net.ipv6.fragment

Signed-off-by: default avatarMartin Åberg <martin.aberg@gaisler.com>
parent b9d04142

subsys/net/ip/ipv6_fragment.c

+10 −9
Original line number Diff line number Diff line
@@ -53,7 +53,6 @@ int net_ipv6_find_last_ext_hdr(struct net_pkt *pkt, uint16_t *next_hdr_off, 
	struct net_ipv6_hdr *hdr;

	uint8_t next_nexthdr;

	uint8_t nexthdr;

	uint16_t length;



	if (!pkt || !pkt->frags || !next_hdr_off || !last_hdr_off) {

		return -EINVAL;
@@ -83,18 +82,20 @@ int net_ipv6_find_last_ext_hdr(struct net_pkt *pkt, uint16_t *next_hdr_off, 
		switch (nexthdr) {

		case NET_IPV6_NEXTHDR_HBHO:

		case NET_IPV6_NEXTHDR_DESTO:

			length = 0U;

			{

				uint8_t val = 0U;

				uint16_t length;



			if (net_pkt_read_u8(pkt, (uint8_t *)&length)) {

				if (net_pkt_read_u8(pkt, &val)) {

					goto fail;

				}



			length = length * 8U + 8 - 2;

				length = val * 8U + 8 - 2;



				if (net_pkt_skip(pkt, length)) {

					goto fail;

				}



			}

			break;

		case NET_IPV6_NEXTHDR_FRAG:

			if (net_pkt_skip(pkt, 7)) {

CRA Git | Maintained and supported by SUSTech CRA and CCSE