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:
no_unions
for the oneof field. This will generate fields as separate instead of C union, and avoids triggering the problematic code.FT_POINTER
. This way the whole submessage will be dynamically allocated and the problematic code is not executed.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.