[
https://issues.apache.org/jira/browse/HADOOP-9511?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13648628#comment-13648628
]
Omkar Vinit Joshi commented on HADOOP-9511:
-------------------------------------------
bq. May be we should just pass in a File to openFSDataInputStream?
Fixed.
bq. The isExists() check is unnecesary? As the subsequent stat will anyways
validate that, right?
Yes this check is also there in FSDataInputStream.open call.. removing it.
bq. Similarly for forceSecureOpenFSDataInputStream.
No additional exists checks here..
Updating tests
> Adding support for additional input streams (FSDataInputStream and
> RandomAccessFile) in SecureIOUtils.
> ------------------------------------------------------------------------------------------------------
>
> Key: HADOOP-9511
> URL: https://issues.apache.org/jira/browse/HADOOP-9511
> Project: Hadoop Common
> Issue Type: Improvement
> Reporter: Omkar Vinit Joshi
> Assignee: Omkar Vinit Joshi
> Attachments: HADOOP-9511.20130426.patch, HADOOP-9511.20130429.patch,
> HADOOP-9511.20130430.1.patch, HADOOP-9511.20130430.patch
>
>
> At present we can only get secured InputStream. However we want other types
> of input streams to avoid possible security attacks.
> YARN-578 needs this fix.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
