You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
when it goes out of scope at line 85 the wrong de-allocator is used
the fixed line is: std::unique_ptr<char[]> outputBuff_uptr = std::unique_ptr<char[]>(new char[requiredLength]); (added array brackets to the types)
there are many other asan warnings when running integration and unit tests (see #15 for instructions). the rest seem to be leaks. generally, valgrind should catch most of these same errors, if you'd prefer to use that
The text was updated successfully, but these errors were encountered:
Hi @larsonmpdx,
Thanks for finding this issue! I have fixed this in our local repo and will be adding it to the list of fixes in the next release. I have also updated our internal repo with the sanitizers and will include that as well.
I will keep this issue open till we have released the next version with the fix.
Please let us know if you find any more issues.
Thanks!
Varun
see https://github.com/aws/aws-iot-device-sdk-cpp/blob/master/src/util/logging/FormattedLogSystem.cpp#L72
when it goes out of scope at line 85 the wrong de-allocator is used
the fixed line is:
std::unique_ptr<char[]> outputBuff_uptr = std::unique_ptr<char[]>(new char[requiredLength]);
(added array brackets to the types)there are many other asan warnings when running integration and unit tests (see #15 for instructions). the rest seem to be leaks. generally, valgrind should catch most of these same errors, if you'd prefer to use that
The text was updated successfully, but these errors were encountered: