Skip to content

Conversation

@sbiscigl
Copy link
Contributor

Description of changes:

Our CI jobs are failing infrequently with a ASAN check

==192==ERROR: LeakSanitizer: detected memory leaks
--
 
Direct leak of 2808 byte(s) in 27 object(s) allocated from:
#0 0x7fd90482657d in malloc (/lib64/libasan.so.4+0xd857d)
#1 0x7fd90395f1bf in Aws::Malloc(char const*, unsigned long) aws-sdk-cpp/src/aws-cpp-sdk-core/source/utils/memory/AWSMemory.cpp:146
#2 0x4e992a in RequestContext* Aws::New<RequestContext>(char const*) aws-sdk-cpp/src/aws-cpp-sdk-core/include/aws/core/utils/memory/AWSMemory.h:70

this tracks request contexts, storing them in a container, removing them from the map. If for any reason some get orphaned, they will be free-ed during object desctruction.

Check all that applies:

  • Did a review by yourself.
  • Added proper tests to cover this PR. (If tests are not applicable, explain.)
  • Checked if this PR is a breaking (APIs have been changed) change.
  • Checked if this PR will not introduce cross-platform inconsistent behavior.
  • Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

  • Linux
  • Windows
  • Android
  • MacOS
  • IOS
  • Other Platforms

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@sbiscigl sbiscigl force-pushed the perf-test-asan branch 2 times, most recently from bf67bac to 54cad01 Compare July 24, 2025 17:52
@sbiscigl sbiscigl marked this pull request as ready for review July 24, 2025 20:45
@sbiscigl sbiscigl merged commit a4a9baf into main Jul 24, 2025
6 checks passed
@sbiscigl sbiscigl deleted the perf-test-asan branch July 24, 2025 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants