Bug 2414 - ControlMaster directive should be ignored on Cygwin
Summary: ControlMaster directive should be ignored on Cygwin
Status: ASSIGNED
Alias: None
Product: Portable OpenSSH
Classification: Unclassified
Component: ssh (show other bugs)
Version: -current
Hardware: Other Cygwin on NT/2k/Win7-11
: P5 enhancement
Assignee: Damien Miller
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-06-17 20:34 AEST by Ismail Donmez
Modified: 2015-08-11 16:26 AEST (History)
3 users (show)

See Also:

Attachments
Make ssh_config IgnoreUnknown silence oUnsupported options too (801 bytes, patch)
2015-07-17 13:23 AEST, Damien Miller 		djm: ok? (dtucker)
 Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ismail Donmez 2015-06-17 20:34:32 AEST 
I am sharing my ssh configuration between Linux & Cygwin and having

ControlMaster auto

in my configuration breaks havoc on Cygwin since its actually not supported there and probably won't be supported anytime soon. Hence, I believe it would be very nice to ignore ControlMaster directive on Cygwin, or assume it to be "no" all the time.
Comment 1 Damien Miller 2015-07-17 13:23:35 AEST 
Created attachment 2673 [details]
Make ssh_config IgnoreUnknown silence oUnsupported options too

We could mark the option as oUnsupported, but AFAIK that would cause stderr spam every time you ran ssh. I guess we could make IgnoreUnknown apply to unsupported options too?

Here's a patch that attempts this.
Comment 2 Damien Miller 2015-07-17 13:28:01 AEST 
oh wait, what error do you receive when you attempt "ControlMaster auto"?
Comment 3 Ismail Donmez 2015-07-18 18:08:58 AEST 
(In reply to Damien Miller from comment #2)
> oh wait, what error do you receive when you attempt "ControlMaster
> auto"?

λ ssh -o ControlMaster=auto i10z.com
process_mux_new_session: tcgetattr: Inappropriate ioctl for device
Last login: Thu Jul 16 20:08:03 2015 from 95.14.75.132
havana β ~
λ select: Interrupted system call
Shared connection to i10z.com closed.
Comment 4 Ismail Donmez 2015-08-08 23:56:01 AEST 
Any chance of having this for 7.0 release?
Comment 5 Jakub Jelen 2015-08-11 16:26:12 AEST 
I attached a patch with different solution to the other bug #1278 related to this issue.

Basically I don't disable this feature, since it is quite useful in remote scripting, instead I did workaround which is now successfully used in scap-workbench.