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

Double free when accessing .AsBoxed() on Pod with 0 refcount #7659

Merged
merged 2 commits into from
Aug 12, 2024

Conversation

MrLolthe1st
Copy link
Collaborator

No description provided.

@MrLolthe1st MrLolthe1st requested a review from a team as a code owner August 12, 2024 12:43
@MrLolthe1st MrLolthe1st self-assigned this Aug 12, 2024
vitstn
vitstn previously approved these changes Aug 12, 2024
Copy link

github-actions bot commented Aug 12, 2024

2024-08-12 12:46:27 UTC Pre-commit check for 2375cdb has started.
2024-08-12 12:50:23 UTC Check linux-x86_64-release-clang14 is running...
🔴 2024-08-12 12:54:21 UTC Build failed. see the logs.

Copy link

github-actions bot commented Aug 12, 2024

2024-08-12 12:46:51 UTC Pre-commit check for 2375cdb has started.
2024-08-12 12:50:58 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-12 12:56:19 UTC Build failed. see the logs.

Copy link

github-actions bot commented Aug 12, 2024

2024-08-12 12:48:05 UTC Pre-commit check for 2375cdb has started.
2024-08-12 12:51:20 UTC Check linux-x86_64-relwithdebinfo is running...
🔴 2024-08-12 12:57:11 UTC Build failed. see the logs.

Copy link

github-actions bot commented Aug 12, 2024

2024-08-12 14:15:28 UTC Pre-commit check for a7b8e27 has started.
2024-08-12 14:19:29 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-12 14:58:45 UTC Build successful.

Copy link

github-actions bot commented Aug 12, 2024

2024-08-12 14:17:06 UTC Pre-commit check for a7b8e27 has started.
2024-08-12 14:20:20 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-08-12 16:32:25 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
76901 63283 0 9 13594 15

2024-08-12 16:42:17 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟢 2024-08-12 16:51:22 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
27 (only retried tests) 18 0 0 0 9

🟢 2024-08-12 16:51:29 UTC Build successful.
🟢 2024-08-12 16:52:02 UTC ydbd size 8.1 GiB changed* by +46.2 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 976e2c7 merge: a7b8e27 diff diff %
ydbd size 8 694 543 584 Bytes 8 694 590 880 Bytes +46.2 KiB +0.001%
ydbd stripped size 472 914 216 Bytes 472 920 680 Bytes +6.3 KiB +0.001%

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

Copy link

github-actions bot commented Aug 12, 2024

2024-08-12 14:18:36 UTC Pre-commit check for a7b8e27 has started.
2024-08-12 14:22:24 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-12 16:52:10 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15314 15257 0 8 33 16

🟢 2024-08-12 16:53:56 UTC Build successful.
🟢 2024-08-12 16:54:25 UTC ydbd size 5.4 GiB changed* by -51.4 KiB, which is <= 0 Bytes vs main: OK

ydbd size dash main: 976e2c7 merge: a7b8e27 diff diff %
ydbd size 5 847 171 512 Bytes 5 847 118 904 Bytes -51.4 KiB -0.001%
ydbd stripped size 1 468 657 904 Bytes 1 468 648 688 Bytes -9.0 KiB -0.001%

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

@MrLolthe1st MrLolthe1st merged commit c66a55e into ydb-platform:main Aug 12, 2024
10 of 12 checks passed
@rvu1024 rvu1024 added the area/yql YQL query language issues label Aug 13, 2024
spuchin added a commit that referenced this pull request Aug 15, 2024
…7721)

Co-authored-by: Александр Новожилов <mrlolthe1st@ydb.tech>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/yql YQL query language issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants