Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/22453 )

Change subject: IMPALA-13736: Fix Use-After-Free in ExecutorGroup.RemoveExecutor
......................................................................

IMPALA-13736: Fix Use-After-Free in ExecutorGroup.RemoveExecutor

The RemoveExecutor() function within the ExecutorGroup class has a
potential use-after-free bug. Since the function takes an object
reference as input, the iterator that erases the backend could erase
the object references by the function input.

This change fixes the issue by storing the necessary data from the
provided input object and then referencing that stored data after the
erase has occurred.

Change-Id: If14a3c89ee631ebb05efc9a47745f7e63ab98690
Reviewed-on: http://gerrit.cloudera.org:8080/22453
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M be/src/scheduling/executor-group-test.cc
M be/src/scheduling/executor-group.cc
M be/src/scheduling/executor-group.h
3 files changed, 20 insertions(+), 4 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/22453
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: If14a3c89ee631ebb05efc9a47745f7e63ab98690
Gerrit-Change-Number: 22453
Gerrit-PatchSet: 5
Gerrit-Owner: Jason Fehr <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>

Reply via email to