Bug 2414

Summary: ControlMaster directive should be ignored on Cygwin
Product: Portable OpenSSH Reporter: Ismail Donmez <ismail>
Component: sshAssignee: Damien Miller <djm>
Status: ASSIGNED ---    
Severity: enhancement CC: djm, dtucker, jjelen
Priority: P5    
Version: -current   
Hardware: Other   
OS: Cygwin on NT/2k/Win7-11   
Attachments:
Description Flags
Make ssh_config IgnoreUnknown silence oUnsupported options too djm: ok? (dtucker)

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.