[ 
https://issues.apache.org/jira/browse/EXEC-26?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12614875#action_12614875
 ] 

sgoeschl edited comment on EXEC-26 at 7/18/08 2:17 PM:
----------------------------------------------------------------

I applied the patch and it works on my box (Mac OS X) but I'm not sure if we 
can create the directory and files on all OS we support, e.g OpenVMS, z/OS, 
OS/2. And not not being able to create those directory and files would 
effectively break the regression test suite for those platforms. So I will not 
apply the patch but I added testExecuteWithFancyArg to the DefaultExecutorTest.

So if nobody objects with any better reasoning I will reject the patch ...

      was (Author: sgoeschl):
    I applied the patch and it works on my box (Mac OS X) but I'm not sure if 
we can create the directory and files on all OS we support, e.g OpenVMS, z/OS, 
OS/2. And not not being able to create those directory and files would 
effectively break the regression test suite for those platforms. So I will not 
apply the patch but I added testExecuteWithFancyArg to the DefaultExecutorTest.
  
> Extend test suite to check non-trivial characters
> -------------------------------------------------
>
>                 Key: EXEC-26
>                 URL: https://issues.apache.org/jira/browse/EXEC-26
>             Project: Commons Exec
>          Issue Type: Test
>            Reporter: Benjamin Bentmann
>            Priority: Minor
>         Attachments: EXEC-26.patch
>
>
> Might be interesting to check whether spaces and other dangerous characters 
> are properly handled in paths/args.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to