Skip to content

NULL pointer dereference during NHC encoding

High
miri64 published GHSA-r2pv-3jqc-vh7w May 30, 2023

Package

RIOT-OS

Affected versions

<= 2023.01

Patched versions

None

Description

Impact

RIOT-OS contains a network stack with the ability to process 6LoWPAN frames. An attacker can send a crafted frame which is forwarded by the device. During encoding of the packet a NULL pointer dereference occurs. This crashes the device leading to denial of service.

Patches

None

Workarounds

  • None

For more information

If you have any questions or comments about this advisory:

Bug Details

The function _iphc_encode checks if the next header exists (source):

    if (pkt->next->next == NULL) {
        DEBUG("6lo iphc: packet next header missing data");
        gnrc_pktbuf_release(dispatch);
        return NULL;
    }

If the next header is an encapsulated IPv6 header _nhc_ipv6_encode_snip will be called (source):

    case PROTNUM_IPV6: {    /* encapsulated IPv6 header */
        local_pos = _nhc_ipv6_encode_snip(pkt, netif_hdr, iface,
                                          &iphc_hdr[inline_pos], &nh);
        break;
    }

The function extracts the header (source):

    gnrc_pktsnip_t *hdr = pkt->next->next;

The header is then passed to _iphc_ipv6_encode (source):

    tmp = (ssize_t)_iphc_ipv6_encode(hdr, netif_hdr, iface, &nhc_data[nhc_len]);

This function tries to access the actual IPv6 header (source):

    ipv6_hdr_t *ipv6_hdr = pkt->next->data;

But it is not checked if the packet actually contains a next snippet and thus next might be NULL and the usage of ->data will result in a NULL pointer dereference.

Severity

High

CVSS overall score

This score calculates overall vulnerability severity from 0 to 10 and is based on the Common Vulnerability Scoring System (CVSS).
/ 10

CVSS v3 base metrics

Attack vector
Network
Attack complexity
Low
Privileges required
None
User interaction
None
Scope
Unchanged
Confidentiality
None
Integrity
None
Availability
High

CVSS v3 base metrics

Attack vector: More severe the more the remote (logically and physically) an attacker can be in order to exploit the vulnerability.
Attack complexity: More severe for the least complex attacks.
Privileges required: More severe if no privileges are required.
User interaction: More severe when no user interaction is required.
Scope: More severe when a scope change occurs, e.g. one vulnerable component impacts resources in components beyond its security scope.
Confidentiality: More severe when loss of data confidentiality is highest, measuring the level of data access available to an unauthorized user.
Integrity: More severe when loss of data integrity is the highest, measuring the consequence of data modification possible by an unauthorized user.
Availability: More severe when the loss of impacted component availability is highest.
CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:H

CVE ID

CVE-2023-33973

Weaknesses

Credits