Skip to content
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

tracer: regressions #4416

Closed
markus2330 opened this issue Jul 16, 2022 · 2 comments
Closed

tracer: regressions #4416

markus2330 opened this issue Jul 16, 2022 · 2 comments
Assignees
Labels

Comments

@markus2330
Copy link
Contributor

markus2330 commented Jul 16, 2022

As uncovered while looking for the problem in #4414:

The searchKey allocated in line 28 of src/plugins/tracer/tracer.c does not get freed. This was introduced in 7c803e1

Furthermore, a bug gets introduced: ksSearch does not support cascading keys, so the logic fails, see #4415 .

Please also check why the tests that caused #4414 didn't show up in our CI.

@github-actions
Copy link

I mark this stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping by writing a message here or create a new issue with the remainder of this issue.
Thank you for your contributions 💖

@github-actions github-actions bot added the stale label Jul 17, 2023
@github-actions
Copy link

github-actions bot commented Aug 1, 2023

I closed this now because it has been inactive for more than one year. If I closed it by mistake, please do not hesitate to reopen it or create a new issue with the remainder of this issue.
Thank you for your contributions 💖

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants