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

tests: internal: stream_processor: fixed memory leak #7975

Merged
merged 2 commits into from
Sep 26, 2023

Conversation

leonardo-albertovich
Copy link
Collaborator

@leonardo-albertovich leonardo-albertovich commented Sep 26, 2023

This PR addresses the following leak reported by ASAN :

Direct leak of 65536 byte(s) in 8 object(s) allocated from:
    #0 0x7fb487452d40 in realloc (/lib/x86_64-linux-gnu/libasan.so.4+0xdfd40)
    #1 0x56098a10d88a in msgpack_sbuffer_write /home/runner/work/fluent-bit/fluent-bit/lib/msgpack-c/include/msgpack/sbuffer.h:81
    #2 0x56098a10d3ef in msgpack_pack_array /home/runner/work/fluent-bit/fluent-bit/lib/msgpack-c/include/msgpack/pack_template.h:724
    #3 0x56098a10fd70 in package_results /home/runner/work/fluent-bit/fluent-bit/src/stream_processor/flb_sp.c:1139
    #4 0x56098a101e80 in flb_sp_do_test /home/runner/work/fluent-bit/fluent-bit/tests/internal/stream_processor.c:125
    #5 0x56098a1040d4 in test_window /home/runner/work/fluent-bit/fluent-bit/tests/internal/stream_processor.c:506
    #6 0x56098a0fb83c in acutest_do_run_ /home/runner/work/fluent-bit/fluent-bit/tests/internal/../lib/acutest/acutest.h:1034
    #7 0x56098a0fbc68 in acutest_run_ /home/runner/work/fluent-bit/fluent-bit/tests/internal/../lib/acutest/acutest.h:1130
    #8 0x56098a0fe5d4 in main /home/runner/work/fluent-bit/fluent-bit/tests/internal/../lib/acutest/acutest.h:1769
    #9 0x7fb486bd5082 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x24082)

@leonardo-albertovich leonardo-albertovich temporarily deployed to pr September 26, 2023 08:15 — with GitHub Actions Inactive
@leonardo-albertovich leonardo-albertovich temporarily deployed to pr September 26, 2023 08:15 — with GitHub Actions Inactive
@leonardo-albertovich leonardo-albertovich temporarily deployed to pr September 26, 2023 08:15 — with GitHub Actions Inactive
@leonardo-albertovich leonardo-albertovich temporarily deployed to pr September 26, 2023 08:45 — with GitHub Actions Inactive
@leonardo-albertovich leonardo-albertovich temporarily deployed to pr September 26, 2023 08:46 — with GitHub Actions Inactive
@leonardo-albertovich leonardo-albertovich temporarily deployed to pr September 26, 2023 08:46 — with GitHub Actions Inactive
@leonardo-albertovich leonardo-albertovich temporarily deployed to pr September 26, 2023 08:46 — with GitHub Actions Inactive
@leonardo-albertovich leonardo-albertovich temporarily deployed to pr September 26, 2023 09:14 — with GitHub Actions Inactive
@leonardo-albertovich leonardo-albertovich merged commit 2610ef8 into master Sep 26, 2023
39 of 42 checks passed
@leonardo-albertovich leonardo-albertovich deleted the leonardo-master-sp-test-leak-fix branch September 26, 2023 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant