Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GitHub runner ubuntu 24.04 and resolve some issues #4727

Merged
merged 9 commits into from
Oct 9, 2024

Conversation

matejk
Copy link
Contributor

@matejk matejk commented Oct 9, 2024

While changing some of the workflows to use Ubuntu 24.04 some issues have been discovered with memory sanitizer and data tests.

In addition to that new option POCO_SANITIZEFLAGS was added to CMake files that behaves the same way as SANITIZEFLAGS in Makefiles.

matejk added 8 commits October 7, 2024 17:14
==125909==ERROR: AddressSanitizer: stack-buffer-overflow on address 0x74737b8121d3 at pc 0x74737e0f95fe bp 0x7fffd0d49940 sp 0x7fffd0d490e8
READ of size 4 at 0x74737b8121d3 thread T0
    #0 0x74737e0f95fd in memcpy ../../../../src/libsanitizer/sanitizer_common/sanitizer_common_interceptors_memintrinsics.inc:115
    #1 0x5eb1aab242fd in memcpy /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29
    #2 0x5eb1aab242fd in Poco::BasicFIFOBuffer<char>::copy(char const*, unsigned long) /home/matejk/Documents/git/github/poco/Foundation/include/Poco/FIFOBuffer.h:326
    #3 0x5eb1aab04626 in CoreTest::testFIFOBufferChar() /home/matejk/Documents/git/github/poco/Foundation/testsuite/src/CoreTest.cpp:719
@matejk matejk added the CI Continuous Integration label Oct 9, 2024
@matejk matejk added this to the Release 1.14.0 milestone Oct 9, 2024
@matejk matejk requested a review from obiltschnig October 9, 2024 11:38
@matejk matejk self-assigned this Oct 9, 2024
@matejk matejk force-pushed the github-runner-ubuntu-24.04 branch from 7aa26c7 to 1c219eb Compare October 9, 2024 15:36
@matejk matejk force-pushed the github-runner-ubuntu-24.04 branch from 1c219eb to 92f67d3 Compare October 9, 2024 15:46
@matejk matejk merged commit a1efeaa into main Oct 9, 2024
43 checks passed
@matejk matejk deleted the github-runner-ubuntu-24.04 branch October 10, 2024 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant