Tim Armstrong has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/13743


Change subject: IMPALA-8713: fix stack overflow in unhex()
......................................................................

IMPALA-8713: fix stack overflow in unhex()

Write the results into the output heap buffer
instead of into a temporary stack buffer.

No additional memory is used because
AnyValUtil::FromBuffer() allocated a temporary
buffer anyway.

Testing:
Added a targeted test to expr-test that caused
a crash before this fix.

Change-Id: Ie0c1760511a04c0823fc465cf6e529e9681b2488
---
M be/src/exprs/expr-test.cc
M be/src/exprs/math-functions-ir.cc
2 files changed, 6 insertions(+), 3 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie0c1760511a04c0823fc465cf6e529e9681b2488
Gerrit-Change-Number: 13743
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Armstrong <[email protected]>

Reply via email to