This repository was archived by the owner on Mar 22, 2023. It is now read-only.
Merge stable-1.6 into stable-1.7#1128
Merged
lukaszstolarczuk merged 18 commits intopmem:stable-1.7from Jul 6, 2021
Merged
Conversation
it makes more sense to rebuild dockers all the time on stable branches, because we don't have to count on DockerHub's policy to keep our docker images on their servers.
…r-on-stables add "rebuild" param to utils' script, to force dockers rebuilding
…nto-stable-1.6 Merge back stable-1.5 to stable-1.6
Casting unaligned ptr_offset_magic address to pointers is undefined behavior for types which require alignemnt > 1. It results in following error under UBSAN for ptr_0_none test: "runtime error: upcast of misaligned address 0x0000deadbeef for type 'C', which requires 8 byte alignment" Also, add test for offset calculation for objects with different alignment requirements.
…ed iterator Origianl implementation resulted in the following array_iterator_noconv_0_test failure under UBSAN: "runtime error: applying non-zero offset 4 to null pointer"
[stable-1.6] Ubsan fixes
Memcheck reports an error in libunwind exception raising.
tests: add libunwind suppresion
igchor
approved these changes
Jul 6, 2021
Contributor
igchor
left a comment
There was a problem hiding this comment.
Reviewable status: 0 of 9 files reviewed, all discussions resolved
Codecov Report
@@ Coverage Diff @@
## stable-1.7 #1128 +/- ##
==============================================
+ Coverage 91.96% 92.00% +0.03%
==============================================
Files 32 32
Lines 2951 2951
==============================================
+ Hits 2714 2715 +1
+ Misses 237 236 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This change is