This repository was archived by the owner on Mar 22, 2023. It is now read-only.
Merge stable-1.7 into stable-1.8#1129
Merged
lukaszstolarczuk merged 20 commits intopmem:stable-1.8from 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
…nto-stable-1.7 Merge stable-1.6 into stable-1.7
igchor
approved these changes
Jul 6, 2021
Contributor
igchor
left a comment
igchor
left a comment
There was a problem hiding this comment.
Reviewable status: 0 of 10 files reviewed, all discussions resolved
Codecov Report
@@ Coverage Diff @@
## stable-1.8 #1129 +/- ##
==============================================
+ Coverage 92.18% 92.19% +0.01%
==============================================
Files 34 34
Lines 3658 3665 +7
==============================================
+ Hits 3372 3379 +7
Misses 286 286
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.
2 participants
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