openssh bugs - Sep 2007 - [Bug 778] sftp client globs entire path, directories enclosed in square brackets are unusable

http://bugzilla.mindrot.org/show_bug.cgi?id=778


Damien Miller <djm at mindrot.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|openssh-bugs at mindrot.org    |djm at mindrot.org
Attachment #1295 is|0                           |1
           obsolete|                            |
   Attachment #1357|                            |ok?
               Flag|                            |
   Attachment #1295|ok?                         |
               Flag|                            |




--- Comment #5 from Damien Miller <djm at mindrot.org>  2007-09-21
14:59:06 ---
Created an attachment (id=1357)
 --> (http://bugzilla.mindrot.org/attachment.cgi?id=1357)
Rewrite sftp command parsing and glob handling

I think this patch is correct: it rewrites sftp's command parsing code
to split the string into an argv using sh(1)-like rules, and should
handle glob sequences much more naturally.

Things like 'rm \*' and 'rm "*"' should work now (i.e.
they are not
equivalent to 'rm *'), as should more complicated sequences like
'put
"[some files] - blah"*'

It does need some regress tests though.


-- 
Configure bugmail: http://bugzilla.mindrot.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are watching the assignee of the bug.
You are watching someone on the CC list of the bug.

http://bugzilla.mindrot.org/show_bug.cgi?id=778


Damien Miller <djm at mindrot.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
Attachment #1357 is|0                           |1
           obsolete|                            |
   Attachment #1358|                            |ok?
               Flag|                            |
   Attachment #1357|ok?                         |
               Flag|                            |




--- Comment #6 from Damien Miller <djm at mindrot.org>  2007-09-21
18:11:14 ---
Created an attachment (id=1358)
 --> (http://bugzilla.mindrot.org/attachment.cgi?id=1358)
Tweaked diff with regress tests

The previous diff had a small bug related to glob's handling of
backslash characters. This version also adds some regression tests.


-- 
Configure bugmail: http://bugzilla.mindrot.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.
You are watching someone on the CC list of the bug.

http://bugzilla.mindrot.org/show_bug.cgi?id=778


Damien Miller <djm at mindrot.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |1353




--- Comment #7 from Damien Miller <djm at mindrot.org>  2007-09-21
18:11:47 ---
Put this on the radar for 4.8


-- 
Configure bugmail: http://bugzilla.mindrot.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.
You are watching someone on the CC list of the bug.