Enable compiler sanitizers for test builds, fixed new bugs found with them#1154
Conversation
|
tried to add the feature sanitizer in cli issue #1137 |
|
the feature has two sanitizers AddressSanitizer (ASan) and UndefinedBehaviorSanitizer (UBSan) and works only for ci not for production |
|
fixes the error |
gittiver
left a comment
There was a problem hiding this comment.
There are changes in query_string and tests which are not directly related to sanitizers.
Why are they there - warnings by sanitizers and can we have these fixes separately?
These changes are directly caused by the sanitizer build configuration. The ci-sanitize preset inherits from ci-linux, which uses the flags-gcc-clang compiler flags — including -Wold-style-cast, -Wconversion, and -Wsign-conversion with -Werror. The |
|
If you want I can solve this in a seperate pr but I am new to git hub so I am still figuring out how things work here |
No description provided.