Joe McDonnell has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16004


Change subject: IMPALA-9800: Fix undefined behavior in 
DecimalUtil::DecodeFromFixedLenByteArray()
......................................................................

IMPALA-9800: Fix undefined behavior in 
DecimalUtil::DecodeFromFixedLenByteArray()

A recent fix (IMPALA-9781) switched DecodeFromFixedLenByteArray() to
use memset() to eliminate an unaligned store under GCC7. However,
this caused crashes in the UBSAN tests.

Added a cast to void* for the first argument for memset().

Testing:
 - Ran parquet-plain-test in UBSAN mode

Change-Id: I7c7dd8a813c5ff8c77b57e2f8e42cbed39eff38f
---
M be/src/util/decimal-util.h
1 file changed, 1 insertion(+), 1 deletion(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7c7dd8a813c5ff8c77b57e2f8e42cbed39eff38f
Gerrit-Change-Number: 16004
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com>

Reply via email to