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.
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.
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.
Bug report: https://github.com/nanopb/nanopb/issues/615
If you have any questions or comments about this advisory, comment on the bug report linked above.
{ "nvd_published_at": null, "github_reviewed_at": "2020-11-25T16:52:43Z", "severity": "MODERATE", "github_reviewed": true, "cwe_ids": [ "CWE-119", "CWE-20" ] }