Skip to content

Memory leak in Nanopb

Moderate severity GitHub Reviewed Published Nov 25, 2020 in nanopb/nanopb • Updated Jan 9, 2023

Package

pip nanopb (pip)

Affected versions

>= 0.3.2, <= 0.3.9.6
>= 0.4.0, < 0.4.4

Patched versions

0.3.9.7
0.4.4

Description

Impact

Decoding specifically formed message can leak memory if dynamic allocation is enabled and an oneof field contains a static submessage that contains a dynamic field, and the message being decoded contains the submessage multiple times. This is rare in normal messages, but it is a concern when untrusted data is parsed.

Patches

Preliminary patch is available on git and problem will be patched in versions 0.3.9.7 and 0.4.4 once testing has been completed.

Workarounds

Following workarounds are available:

  • Set the option no_unions for the oneof field. This will generate fields as separate instead of C union, and avoids triggering the problematic code.
  • Set the type of the submessage field inside oneof to FT_POINTER. This way the whole submessage will be dynamically allocated and the problematic code is not executed.
  • Use an arena allocator for nanopb, to make sure all memory can be released afterwards.

References

Bug report: nanopb/nanopb#615

For more information

If you have any questions or comments about this advisory, comment on the bug report linked above.

References

@PetteriAimonen PetteriAimonen published to nanopb/nanopb Nov 25, 2020
Reviewed Nov 25, 2020
Published to the GitHub Advisory Database Nov 25, 2020
Last updated Jan 9, 2023

Severity

Moderate

EPSS score

0.340%
(72nd percentile)

CVE ID

CVE-2020-26243

GHSA ID

GHSA-85rr-4rh9-hhwh

Source code

No known source code
Loading Checking history
See something to contribute? Suggest improvements for this vulnerability.