ci: Bump codecov/codecov-action to v3 #7
Merged
Conversation
This file contains 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
The v2 version uses node12 which is deprecated. Signed-off-by: Free Ekanayaka <free@ekanayaka.io>
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #7 +/- ##
=======================================
Coverage 76.49% 76.49%
=======================================
Files 51 51
Lines 9685 9685
Branches 2451 2451
=======================================
Hits 7409 7409
Misses 1089 1089
Partials 1187 1187 ☔ View full report in Codecov by Sentry. |
freeekanayaka
added a commit
to freeekanayaka/raft
that referenced
this pull request
Sep 14, 2023
…seCb
We actually don't track that there are pending segment writer close operations,
so this call to uvMaybeFireCloseCb() can't really unblock anything waiting on
it.
On the contrary, if it fires after the close callback has actually been fired,
it causes a double free:
=================================================================
==199067==ERROR: AddressSanitizer: attempting double-free on 0x602000000010 in thread T0:
#0 0x7fd5e44d7288 in __interceptor_free ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:52
cowsql#1 0x7fd5e4b5a259 in ioCloseCb src/raft.c:115
cowsql#2 0x7fd5e4b84861 in uvMaybeFireCloseCbReal src/uv.c:202
cowsql#3 0x7fd5e4afba70 in uv_run (/lib/x86_64-linux-gnu/libuv.so.1+0xfa70) (BuildId: 7f7f8c148150666c7b116bf98bf6e27f96c697a9)
cowsql#4 0x55b25438ebe5 in SubmitRun tools/benchmark/submit.c:314
cowsql#5 0x55b254388955 in main tools/benchmark/main.c:60
cowsql#6 0x7fd5e42456c9 (/lib/x86_64-linux-gnu/libc.so.6+0x276c9) (BuildId: 072feb34c63e054d60d94cbc68d92e4caad25d72)
cowsql#7 0x7fd5e4245784 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x27784) (BuildId: 072feb34c63e054d60d94cbc68d92e4caad25d72)
cowsql#8 0x55b254388bf0 in _start (/home/free/src/c/raft/tools/.libs/raft-benchmark+0x7bf0) (BuildId: 09105ce7d1a7fe71ca812de2af648a85fc490ebb)
0x602000000010 is located 0 bytes inside of 15-byte region [0x602000000010,0x60200000001f)
freed by thread T0 here:
#0 0x7fd5e44d7288 in __interceptor_free ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:52
cowsql#1 0x7fd5e4b5a259 in ioCloseCb src/raft.c:115
cowsql#2 0x7fd5e4b84861 in uvMaybeFireCloseCbReal src/uv.c:202
cowsql#3 0x7fd5e4afba70 in uv_run (/lib/x86_64-linux-gnu/libuv.so.1+0xfa70) (BuildId: 7f7f8c148150666c7b116bf98bf6e27f96c697a9)
previously allocated by thread T0 here:
#0 0x7fd5e44d85bf in __interceptor_malloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:69
cowsql#1 0x7fd5e4b59983 in raft_init src/raft.c:62
cowsql#2 0x55b25438e7d4 in serverInit tools/benchmark/submit.c:127
cowsql#3 0x55b25438ebbf in SubmitRun tools/benchmark/submit.c:302
cowsql#4 0x55b254388955 in main tools/benchmark/main.c:60
cowsql#5 0x7fd5e42456c9 (/lib/x86_64-linux-gnu/libc.so.6+0x276c9) (BuildId: 072feb34c63e054d60d94cbc68d92e4caad25d72)
Signed-off-by: Free Ekanayaka <free@ekanayaka.io>
freeekanayaka
added a commit
that referenced
this pull request
Sep 14, 2023
…Cb (#45) We actually don't track that there are pending segment writer close operations, so this call to uvMaybeFireCloseCb() can't really unblock anything waiting on it. On the contrary, if it fires after the close callback has actually been fired, it causes a double free: ==199067==ERROR: AddressSanitizer: attempting double-free on 0x602000000010 in thread T0: #0 0x7fd5e44d7288 in __interceptor_free ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:52 #1 0x7fd5e4b5a259 in ioCloseCb src/raft.c:115 #2 0x7fd5e4b84861 in uvMaybeFireCloseCbReal src/uv.c:202 #3 0x7fd5e4afba70 in uv_run (/lib/x86_64-linux-gnu/libuv.so.1+0xfa70) (BuildId: 7f7f8c148150666c7b116bf98bf6e27f96c697a9) #4 0x55b25438ebe5 in SubmitRun tools/benchmark/submit.c:314 #5 0x55b254388955 in main tools/benchmark/main.c:60 #6 0x7fd5e42456c9 (/lib/x86_64-linux-gnu/libc.so.6+0x276c9) (BuildId: 072feb34c63e054d60d94cbc68d92e4caad25d72) #7 0x7fd5e4245784 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x27784) (BuildId: 072feb34c63e054d60d94cbc68d92e4caad25d72) #8 0x55b254388bf0 in _start (/home/free/src/c/raft/tools/.libs/raft-benchmark+0x7bf0) (BuildId: 09105ce7d1a7fe71ca812de2af648a85fc490ebb) 0x602000000010 is located 0 bytes inside of 15-byte region [0x602000000010,0x60200000001f) freed by thread T0 here: #0 0x7fd5e44d7288 in __interceptor_free ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:52 #1 0x7fd5e4b5a259 in ioCloseCb src/raft.c:115 #2 0x7fd5e4b84861 in uvMaybeFireCloseCbReal src/uv.c:202 #3 0x7fd5e4afba70 in uv_run (/lib/x86_64-linux-gnu/libuv.so.1+0xfa70) (BuildId: 7f7f8c148150666c7b116bf98bf6e27f96c697a9) previously allocated by thread T0 here: #0 0x7fd5e44d85bf in __interceptor_malloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:69 #1 0x7fd5e4b59983 in raft_init src/raft.c:62 #2 0x55b25438e7d4 in serverInit tools/benchmark/submit.c:127 #3 0x55b25438ebbf in SubmitRun tools/benchmark/submit.c:302 #4 0x55b254388955 in main tools/benchmark/main.c:60 #5 0x7fd5e42456c9 (/lib/x86_64-linux-gnu/libc.so.6+0x276c9) (BuildId: 072feb34c63e054d60d94cbc68d92e4caad25d72)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The v2 version uses node12 which is deprecated.