[jira] [Resolved] (IMPALA-6132) ASAN test fails when trying to move/copy string created by kudu::EnvPosix::GetExecutablePath into InitAuth()

["Sailesh Mukil (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/IMPALA-6132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sailesh Mukil resolved IMPALA-6132.
-----------------------------------
       Resolution: Fixed
         Assignee: Sailesh Mukil
    Fix Version/s: Impala 2.11.0

Commit in:
https://github.com/apache/incubator-impala/commit/1b0852c9d65c62b21df7bf7a9b1398052d5a1396

Explanation in: https://gerrit.cloudera.org/#/c/8412/

> ASAN test fails when trying to move/copy string created by kudu::EnvPosix::GetExecutablePath into InitAuth()
> ------------------------------------------------------------------------------------------------------------
>
>                 Key: IMPALA-6132
>                 URL: https://issues.apache.org/jira/browse/IMPALA-6132
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Backend
>            Reporter: Sailesh Mukil
>            Assignee: Sailesh Mukil
>              Labels: asan, test-failure
>             Fix For: Impala 2.11.0
>
>
> Following from the code review: https://gerrit.cloudera.org/#/c/8412/
> After moving the string into InitAuth() and storing it in AuthManager (which is a global singleton), the ASAN error still kept showing up. I spent many hours trying to debug this problem and I have no understanding of why it happens. My inclination is that ASAN gets confused by how kudu::PosixEnv::GetExecutablePath() creates a string. It creates a unique_ptr<char[]> and then assign()'s it to a string.
> For some reason this is seen as a heap-use-after free if it is called multiple times.
> The only thing that works for now is to leave the code as is in this patch set.
> I have 2 attempts to move/copy the string into GetExecutablePath() and their corresponding errors here:
> https://github.com/smukil/incubator-impala/commit/d3fedb0d665def4870b787a586e922e361cac2dc (copy)
> https://github.com/smukil/incubator-impala/commit/c2e6ca8987054539be70e5ef7e3244c49df2ee40 (move)
> If anyone else is able to make progress on this, that would be great.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)