hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Haohui Mai (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HDFS-9265) InputStreamImpl should hold a shared_ptr of the BlockReader
Date Wed, 21 Oct 2015 18:49:27 GMT

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

Haohui Mai updated HDFS-9265:
-----------------------------
    Summary: InputStreamImpl should hold a shared_ptr of the BlockReader  (was: Use of undefined
behavior in remote_block_reader causing deterministic crashes.)

> InputStreamImpl should hold a shared_ptr of the BlockReader
> -----------------------------------------------------------
>
>                 Key: HDFS-9265
>                 URL: https://issues.apache.org/jira/browse/HDFS-9265
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: hdfs-client
>            Reporter: James Clampffer
>            Assignee: James Clampffer
>            Priority: Blocker
>         Attachments: HDFS-9265.HDFS-8707.000.patch
>
>
> The remote block reader relies on undefined behavior in how it uses enable_shared_from_this.
> http://en.cppreference.com/w/cpp/memory/enable_shared_from_this
> The spec states a shared_ptr to an object inheriting from enable_shared_from_this must
be live before calling make_shared_from_this.  
> Calling make_shared_from_this without an existing shared_ptr is undefined behavior and
causes deterministic crashes when the code is built with GCC.
> example:
> class foo : public enable_shared_from_this {/*bar*/};
> safe:
> auto ptr1 = std::make_shared<foo>();
> auto ptr2 = foo->make_shared_from_this();
> broken:
> foo *ptr = new foo();
> auto ptr2 = foo->make_shared_from_this(); //no existing live shared_ptr
> In order to fix the input stream should call std::make_shared and hang onto a shared_ptr
to the block reader.  The block reader will then be free to call make_shared_from this as
much as it wants without issue.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message