[Bug 3474] ssh_config can escape double quotes with a backslash

2023-11-13 Thread bugzilla-daemon
https://bugzilla.mindrot.org/show_bug.cgi?id=3474

--- Comment #7 from Damien Miller  ---
Created attachment 3758
  --> https://bugzilla.mindrot.org/attachment.cgi?id=3758=edit
Make Match keywords in ssh_config and sshd_config use argv rather than
strdelim

This changes the argument splitting of Match to use the argv tokeniser
that is used for the rest of config parsing. Please test.

-- 
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.
___
openssh-bugs mailing list
openssh-bugs@mindrot.org
https://lists.mindrot.org/mailman/listinfo/openssh-bugs


[Bug 3474] ssh_config can escape double quotes with a backslash

2023-11-13 Thread bugzilla-daemon
https://bugzilla.mindrot.org/show_bug.cgi?id=3474

--- Comment #6 from NIWA Naoya  ---
Excuse me, I forgot to write an important explanation. I'm encountering
this problem with "exec" parameter in "Match" statement. This
implementation uses "strdelim" and cannot escape double quotes.

-- 
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.
___
openssh-bugs mailing list
openssh-bugs@mindrot.org
https://lists.mindrot.org/mailman/listinfo/openssh-bugs


[Bug 3474] ssh_config can escape double quotes with a backslash

2023-11-13 Thread bugzilla-daemon
https://bugzilla.mindrot.org/show_bug.cgi?id=3474

NIWA Naoya  changed:

   What|Removed |Added

 CC||mine@mine02c4.nagoya

--- Comment #5 from NIWA Naoya  ---
I am facing this problem. Specifically, I want to pass the result of
the command execution including spaces to "test" command as a single
string.
This requires passing a string containing double quotes to the shell,
which is not possible with the current version. Miller's patch does
this, but it is not currently implemented. Is there anything I can do
to get this into the main stream?

-- 
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.
___
openssh-bugs mailing list
openssh-bugs@mindrot.org
https://lists.mindrot.org/mailman/listinfo/openssh-bugs


[Bug 3474] ssh_config can escape double quotes with a backslash

2022-10-12 Thread bugzilla-daemon
https://bugzilla.mindrot.org/show_bug.cgi?id=3474

--- Comment #4 from unknowndev...@pm.me ---
Sorry, I can't do it.

-- 
You are receiving this mail because:
You are watching someone on the CC list of the bug.
You are watching the assignee of the bug.
___
openssh-bugs mailing list
openssh-bugs@mindrot.org
https://lists.mindrot.org/mailman/listinfo/openssh-bugs


[Bug 3474] ssh_config can escape double quotes with a backslash

2022-10-10 Thread bugzilla-daemon
https://bugzilla.mindrot.org/show_bug.cgi?id=3474

--- Comment #3 from Damien Miller  ---
No unfortunately - it's a bit more of a substantial rewrite than that.
strdelim() is an incremental tokeniser whereas argv_split() turns a
string into a vector of arguments in one go.

https://github.com/openssh/openssh-portable/commit/ea9e45c89a is the
commit where we started using it for config parsing if you want to have
a go at this yourself.

-- 
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.
___
openssh-bugs mailing list
openssh-bugs@mindrot.org
https://lists.mindrot.org/mailman/listinfo/openssh-bugs


[Bug 3474] ssh_config can escape double quotes with a backslash

2022-10-10 Thread bugzilla-daemon
https://bugzilla.mindrot.org/show_bug.cgi?id=3474

--- Comment #2 from unknowndev...@pm.me ---
(In reply to Damien Miller from comment #1)
> readconf.c:match_cfg_line() probably needs to be converted from
> strdelim() to argv_split() for this to happen.

s/strdelim()/argv_split()/
Like this?

-- 
You are receiving this mail because:
You are watching someone on the CC list of the bug.
You are watching the assignee of the bug.
___
openssh-bugs mailing list
openssh-bugs@mindrot.org
https://lists.mindrot.org/mailman/listinfo/openssh-bugs


[Bug 3474] ssh_config can escape double quotes with a backslash

2022-09-23 Thread bugzilla-daemon
https://bugzilla.mindrot.org/show_bug.cgi?id=3474

Damien Miller  changed:

   What|Removed |Added

 CC||d...@mindrot.org

--- Comment #1 from Damien Miller  ---
readconf.c:match_cfg_line() probably needs to be converted from
strdelim() to argv_split() for this to happen.

-- 
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.
___
openssh-bugs mailing list
openssh-bugs@mindrot.org
https://lists.mindrot.org/mailman/listinfo/openssh-bugs