Thomas Tauber-Marshall has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/3242

Change subject: IMPALA-3639: expr-test fails on ASAN
......................................................................

IMPALA-3639: expr-test fails on ASAN

In ExprTest::GetValue, we create a local string and then end up returning
a reference to that string, resulting in a memory error. This patch moves
the local string to be a member variable of ExprTest, ensuring that the
memory is still valid after GetValue returns.

Change-Id: If6fd24a59248fbf27a697ff25412c768ac0c472a
---
M be/src/exprs/expr-test.cc
1 file changed, 4 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/42/3242/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3242
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If6fd24a59248fbf27a697ff25412c768ac0c472a
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Thomas Tauber-Marshall <[email protected]>

Reply via email to