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

James Clampffer updated HDFS-10527:
-----------------------------------
    Attachment: HDFS-10527.HDFS-8707.000.patch

Attached fix.  Not much to it.

> libhdfs++: hdfsGetBlockLocations doesn't null terminate ip address strings
> --------------------------------------------------------------------------
>
>                 Key: HDFS-10527
>                 URL: https://issues.apache.org/jira/browse/HDFS-10527
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: hdfs-client
>            Reporter: James Clampffer
>            Assignee: James Clampffer
>         Attachments: HDFS-10527.HDFS-8707.000.patch
>
>
> Code looks like this
> {code}
> strncpy(buf, ipaddr.c_str(),ipaddr.size());
> {code}
> But should be
> {code}
> strncpy(buf, ipaddr.c_str(),ipaddr.size()+1);
> {code}
> In order to make sure there is at least 1 null terminating byte.  If we could 
> run the minidfscluster an another process and run valgrind on the libhdfs++ 
> tests this would show up really quickly as a sequence of invalid reads when 
> that const char* was passed to std::string::string(const char*), strlen, or 
> strcpy.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to