Skip to content

Conversation

@masaori335
Copy link
Contributor

Suppress the leak we're facing on PRB.

117/136 Test #114: verify_global_magick ...................***Failed    0.35 sec
[Jun 12 13:10:18.447] traffic_server NOTE: records parsing completed.
[Jun 12 13:10:18.447] traffic_server NOTE: /tmp/ats-quiche/etc/trafficserver/records.yaml finished loading
[Jun 12 13:10:18.460] traffic_server NOTE: Traffic Server is running unprivileged, not switching to user 'nobody'
NOTE: verifying plugin '/home/jenkins/workspace/Github_Builds/rocky/src/build/plugins/experimental/magick/magick.so'...
NOTE: verifying plugin '/home/jenkins/workspace/Github_Builds/rocky/src/build/plugins/experimental/magick/magick.so' Success

=================================================================
==6867==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 8 byte(s) in 1 object(s) allocated from:
    #0 0x7fbb3b08d9a7 in __interceptor_malloc (/lib64/libasan.so.6+0xb49a7)
    #1 0x7fbb31796dcc  (/opt/h3-tools-boringssl/boringssl/lib64/libssl.so+0x715dcc)

SUMMARY: AddressSanitizer: 8 byte(s) leaked in 1 allocation(s).

@masaori335 masaori335 added Tests ASan Address Sanitizer Leak labels Jun 12, 2024
@masaori335 masaori335 added this to the 10.1.0 milestone Jun 12, 2024
@masaori335 masaori335 self-assigned this Jun 12, 2024
Copy link
Contributor

@bneradt bneradt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know @JosiahWI was investigating this. We weren't quite 100% sure yet why this leak was being reported and whether we could ignore it yet. We thought we probably could, but weren't sure.

Can you share your thoughts? Do you know why the leak is sometimes being reported?

@masaori335 masaori335 merged commit c28ee67 into apache:master Jun 12, 2024
@masaori335
Copy link
Contributor Author

115/136 Test #114: verify_global_magick ...................***Failed    0.37 sec
[Jun 13 01:45:44.985] traffic_server NOTE: records parsing completed.
[Jun 13 01:45:44.985] traffic_server NOTE: /tmp/ats-quiche/etc/trafficserver/records.yaml finished loading
[Jun 13 01:45:44.989] traffic_server NOTE: Traffic Server is running unprivileged, not switching to user 'nobody'
NOTE: verifying plugin '/home/jenkins/workspace/Github_Builds/rocky/src/build/plugins/experimental/magick/magick.so'...
NOTE: verifying plugin '/home/jenkins/workspace/Github_Builds/rocky/src/build/plugins/experimental/magick/magick.so' Success

=================================================================
==6860==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 8 byte(s) in 1 object(s) allocated from:
    #0 0x7fe5b47859a7 in __interceptor_malloc (/lib64/libasan.so.6+0xb49a7)
    #1 0x7fe5aae96dcc  (<unknown module>)

SUMMARY: AddressSanitizer: 8 byte(s) leaked in 1 allocation(s).

With this PR, still a leak is detected from <unknown module> 😞

@JosiahWI
Copy link
Contributor

JosiahWI commented Jun 13, 2024

I was unable to reproduce either of the leaks before and after this change on one of the CI docker environments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ASan Address Sanitizer Leak Tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants