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.

Sign up for GitHub

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

Jump to bottom

[YCQL] heap-use-after-free in org.yb.cql.TestReturnsClause#testBatchDMLWithUniqueIndex #1042

Closed
mbautin opened this issue Mar 20, 2019 · 1 comment
Closed

[YCQL] heap-use-after-free in org.yb.cql.TestReturnsClause#testBatchDMLWithUniqueIndex #1042

mbautin opened this issue Mar 20, 2019 · 1 comment
Assignees
@robertpang
Labels
kind/bug This issue is a bug

Comments

@mbautin
Copy link
Contributor

mbautin commented Mar 20, 2019

ybd asan --java-test org.yb.cql.TestReturnsClause#testBatchDMLWithUniqueIndex

https://gist.githubusercontent.com/mbautin/b8879d1af42671a7ad67d89d76097163/raw
@robertpang robertpang added the kind/bug This issue is a bug label Mar 20, 2019
yugabyte-ci pushed a commit that referenced this issue Mar 22, 2019
@robertpang
[YCQL] #1042: Fix ASAN issue in YCQL executor restart
14279cc 
Summary:
When a restart is required inside Executor::ProcessTnodeResults(), the current ExecContext and
the children TnodeContext have been reset. It should return immeidately and avoid accessing those
contexts to avoid ASAN issue.

Test Plan: Run org.yb.cql.TestReturnsClause#testBatchDMLWithUniqueIndex Java test in ASAN mode for 100 times

Reviewers: mihnea, mikhail, sergei

Reviewed By: mikhail, sergei

Subscribers: yql

Differential Revision: https://phabricator.dev.yugabyte.com/D6365
@robertpang
Copy link
Contributor

Fixed in commit 14279cc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@robertpang robertpang

Labels
kind/bug This issue is a bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mbautin @robertpang