-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Filebeat copy_fields processor can recurse, leading to crash #19206
Comments
Pinging @elastic/integrations (Team:Integrations) |
copy source: https://github.com/elastic/beats/blob/master/libbeat/processors/actions/copy_fields.go#L99 slices and maps are stored by reference. The code does not handle these types correctly, which leads to a child-document pointing to it's root. Full copy semantics are required. Another bug we see due to these reference semantics |
) Closes elastic#19206 (cherry picked from commit 0940e25)
) Closes elastic#19206 (cherry picked from commit 0940e25)
…26382) Closes #19206 (cherry picked from commit 0940e25) Co-authored-by: ian woolf <[email protected]>
It's possible for the
copy_fields
processor to fall into infinite recursion if we point at two fields with the same root:The
from
andto
fields will point to the same root, leading an eventual crash. It appearscopy_fields
doesn't do an actual copy and just moves references. @urso discovered the root cause here.The text was updated successfully, but these errors were encountered: