[ 
https://issues.apache.org/jira/browse/YARN-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15477967#comment-15477967
 ] 

Allen Wittenauer commented on YARN-5621:
----------------------------------------

Actually, that reminds me.... where *is* the input validation here? Every time 
we use bash, we're opening the door to all sorts of fun metacharacter issues.

Is it possible for a user to try to get a symlink for:

{code}
1\\;rm -rf /
1"\\;rm -rf /
1\\\;;;;;;rm -rf /
1\\;;*;rm -rf /
{code}

...
etc.

FWIW, I'd love to see us drop the container launch script.  I haven't tried it, 
but I suspect we can do lots of fun things with the env vars.  Especially if it 
is using execlp instead of execl.  Just because we wrap stuff in quotes doesn't 
mean that code is magically safe. (and because set -e, pipefail, etc aren't set 
in that launch script, it just makes it a bigger/easier target.)

> Support LinuxContainerExecutor to create symlinks for continuously localized 
> resources
> --------------------------------------------------------------------------------------
>
>                 Key: YARN-5621
>                 URL: https://issues.apache.org/jira/browse/YARN-5621
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Jian He
>            Assignee: Jian He
>         Attachments: YARN-5621.1.patch, YARN-5621.2.patch, YARN-5621.3.patch
>
>
> When new resources are localized, new symlink needs to be created for the 
> localized resource. This is the change for the LinuxContainerExecutor to 
> create the symlinks.



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

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

Reply via email to