Fix actor system usage after it was freed #7316

UgnineSirdis · 2024-07-31T13:22:40Z

Changelog entry

Problems affects test actor system

Changelog category

Not for changelog (changelog entry is not required)

Additional information

...

github-actions · 2024-07-31T13:23:52Z

⚪ 2024-07-31 13:23:51 UTC Pre-commit check for 8085aa9 has started.
⚪ 2024-07-31 13:26:37 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-07-31 13:31:18 UTC Build successful.

github-actions · 2024-07-31T13:25:07Z

⚪ 2024-07-31 13:25:07 UTC Pre-commit check for 8085aa9 has started.
⚪ 2024-07-31 13:29:44 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-07-31 14:24:14 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
11271	9571	0	377	1298	25

⚪ 2024-07-31 14:25:36 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟡 2024-07-31 14:37:03 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
393 (only retried tests)	53	0	323	0	17

⚪ 2024-07-31 14:37:49 UTC Failed tests rerun (try 3) linux-x86_64-relwithdebinfo is running...
🔴 2024-07-31 14:47:14 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
342 (only retried tests)	11	0	317	0	14

🟢 2024-07-31 14:48:03 UTC Build successful.
🟢 2024-07-31 14:48:47 UTC ydbd size 8.1 GiB changed* by -32 Bytes, which is <= 0 Bytes vs main: OK

ydbd size dash	main: `e464b73`	merge: `8085aa9`	diff	diff %
ydbd size	8 678 737 472 Bytes	8 678 737 440 Bytes	-32 Bytes	-0.000%
ydbd stripped size	471 916 032 Bytes	471 916 032 Bytes	0 Bytes	0.000%

^{*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation}

github-actions · 2024-07-31T13:39:30Z

⚪ 2024-07-31 13:39:30 UTC Pre-commit check for 8085aa9 has started.
⚪ 2024-07-31 13:42:39 UTC Check linux-x86_64-release-asan is running...
🔴 2024-07-31 14:50:35 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
8015	7487	0	494	1	33

🟢 2024-07-31 14:51:35 UTC Build successful.
🔴 2024-07-31 14:52:08 UTC ydbd size 5.5 GiB changed* by +275.7 MiB, which is >= 2.0 MiB vs main: Alert

ydbd size dash	main: `e464b73`	merge: `8085aa9`	diff	diff %
ydbd size	5 571 429 760 Bytes	5 860 490 776 Bytes	+275.7 MiB	+5.188%
ydbd stripped size	1 199 304 144 Bytes	1 467 594 736 Bytes	+255.9 MiB	+22.371%

^{*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation}

ydb/core/testlib/actors/test_runtime.cpp

Co-authored-by: Semyon Danilov <[email protected]>

github-actions · 2024-08-01T16:56:17Z

⚪ 2024-08-01 16:56:16 UTC Pre-commit check for 96aecc3 has started.
⚪ 2024-08-01 16:59:07 UTC Check linux-x86_64-relwithdebinfo is running...
🟢 2024-08-01 17:53:10 UTC Tests successful.

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
11283	9969	0	0	1300	14

🟢 2024-08-01 17:54:38 UTC Build successful.
🟢 2024-08-01 17:55:14 UTC ydbd size 8.1 GiB changed* by -32 Bytes, which is <= 0 Bytes vs main: OK

ydbd size dash	main: `4bf74af`	merge: `96aecc3`	diff	diff %
ydbd size	8 673 384 200 Bytes	8 673 384 168 Bytes	-32 Bytes	-0.000%
ydbd stripped size	472 106 984 Bytes	472 106 984 Bytes	0 Bytes	0.000%

^{*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation}

github-actions · 2024-08-01T16:56:21Z

⚪ 2024-08-01 16:56:21 UTC Pre-commit check for 96aecc3 has started.
⚪ 2024-08-01 17:00:52 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-01 18:12:42 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
8028	8015	0	3	0	10

🟢 2024-08-01 18:13:42 UTC Build successful.
🟢 2024-08-01 18:14:11 UTC ydbd size 5.4 GiB changed* by -96 Bytes, which is <= 0 Bytes vs main: OK

ydbd size dash	main: `4bf74af`	merge: `96aecc3`	diff	diff %
ydbd size	5 842 323 424 Bytes	5 842 323 328 Bytes	-96 Bytes	-0.000%
ydbd stripped size	1 464 110 544 Bytes	1 464 110 480 Bytes	-64 Bytes	-0.000%

^{*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation}

github-actions · 2024-08-01T16:56:30Z

⚪ 2024-08-01 16:56:30 UTC Pre-commit check for 96aecc3 has started.
⚪ 2024-08-01 16:59:45 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-01 17:05:16 UTC Build successful.

SammyVimes

LGTM!

Co-authored-by: Semyon Danilov <[email protected]> (cherry picked from commit 652be9a)

Fix actor system usage after it was freed

032c82b

github-actions bot added the not-for-changelog label Jul 31, 2024

SammyVimes requested changes Aug 1, 2024

View reviewed changes

ydb/core/testlib/actors/test_runtime.cpp Show resolved Hide resolved

Update ydb/core/testlib/actors/test_runtime.cpp

0ec45a2

Co-authored-by: Semyon Danilov <[email protected]>

SammyVimes approved these changes Aug 2, 2024

View reviewed changes

UgnineSirdis merged commit 652be9a into ydb-platform:main Aug 2, 2024
10 of 12 checks passed

UgnineSirdis mentioned this pull request Aug 2, 2024

[Viewer] AddressSanitizer: heap-use-after-free NActors::TAsyncHttpMon::Stop() ydb/core/mon/async_http_mon.cpp:809:26 #7282

Closed

UgnineSirdis added a commit to UgnineSirdis/ydb that referenced this pull request Aug 2, 2024

Fix actor system usage after it was freed (ydb-platform#7316)

af026b8

Co-authored-by: Semyon Danilov <[email protected]> (cherry picked from commit 652be9a)

UgnineSirdis added a commit that referenced this pull request Aug 5, 2024

Fix actor system usage after it was freed (#7316) (#7404)

4876ec0

UgnineSirdis linked an issue Aug 7, 2024 that may be closed by this pull request

[Viewer] AddressSanitizer: heap-use-after-free NActors::TAsyncHttpMon::Stop() ydb/core/mon/async_http_mon.cpp:809:26 #7282

Closed

StekPerepolnen mentioned this pull request Aug 8, 2024

tenantinfo handler added DatabaseStorage #7554

Merged

This was referenced Aug 8, 2024

YQ revert local grpc peer value #7555

Merged

YQ kqprun pass actor system config #7693

Merged

niksaveliev mentioned this pull request Aug 15, 2024

New kafka metering fix #7856

Closed

This was referenced Aug 16, 2024

Fix use after move in http_proxy #7884

Closed

fix use after move in http_proxy #7885

Merged

This was referenced Aug 23, 2024

merge/stable 24 3/query float test asan #8192

Closed

asan viewer tests #8247

Open

StekPerepolnen added a commit to StekPerepolnen/ydb that referenced this pull request Aug 26, 2024

merge ydb-platform#7316

96e1998

This was referenced Aug 26, 2024

query transaction mode and asan tests fixed - merge stable-24-2 #8197

Merged

query transaction mode and asan tests fixed - merge stable-24-3 #8195

Merged

siarheivesialou mentioned this pull request Aug 28, 2024

stable 24 3 6 logbroker add 5 last requests #8389

Closed

StekPerepolnen added a commit that referenced this pull request Aug 28, 2024

merge #7316

bdb5777

swalrus1 mentioned this pull request Sep 3, 2024

Update stable-24-3: add metrics to describe table #8688

Closed

UgnineSirdis deleted the fix-use-after-free-in-mon-page branch October 7, 2024 07:35

This was referenced Oct 29, 2024

viewer healthcheck pathmerge stable 24 3 11 hotfix #11042

Closed

viewer healthcheck pathmerge stable 24 #11044

Closed

serbel324 mentioned this pull request Nov 19, 2024

Merge double accelerations into 24-3 #11740

Merged

GrigoriyPA mentioned this pull request Feb 21, 2025

merge to ydb stable YQ-4070 supported insert values for s3 #14914

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix actor system usage after it was freed #7316

Fix actor system usage after it was freed #7316

UgnineSirdis commented Jul 31, 2024

github-actions bot commented Jul 31, 2024 •

edited

Loading

github-actions bot commented Jul 31, 2024 •

edited

Loading

github-actions bot commented Jul 31, 2024 •

edited

Loading

github-actions bot commented Aug 1, 2024 •

edited

Loading

github-actions bot commented Aug 1, 2024 •

edited

Loading

github-actions bot commented Aug 1, 2024 •

edited

Loading

SammyVimes left a comment

Fix actor system usage after it was freed #7316

Fix actor system usage after it was freed #7316

Conversation

UgnineSirdis commented Jul 31, 2024

Changelog entry

Changelog category

Additional information

github-actions bot commented Jul 31, 2024 • edited Loading

github-actions bot commented Jul 31, 2024 • edited Loading

github-actions bot commented Jul 31, 2024 • edited Loading

github-actions bot commented Aug 1, 2024 • edited Loading

github-actions bot commented Aug 1, 2024 • edited Loading

github-actions bot commented Aug 1, 2024 • edited Loading

SammyVimes left a comment

Choose a reason for hiding this comment

github-actions bot commented Jul 31, 2024 •

edited

Loading

github-actions bot commented Jul 31, 2024 •

edited

Loading

github-actions bot commented Jul 31, 2024 •

edited

Loading

github-actions bot commented Aug 1, 2024 •

edited

Loading

github-actions bot commented Aug 1, 2024 •

edited

Loading

github-actions bot commented Aug 1, 2024 •

edited

Loading