]> asedeno.scripts.mit.edu Git - PuTTY.git/commit
Turn off Windows process ACL restriction by default.
authorSimon Tatham <anakin@pobox.com>
Sat, 28 Jan 2017 21:56:28 +0000 (21:56 +0000)
committerSimon Tatham <anakin@pobox.com>
Sun, 29 Jan 2017 23:08:19 +0000 (23:08 +0000)
commite22120fea8d39e6a2ef6b2f4ab3ee5502f56169a
tree77ac856c87650dc9a00d06d516f7c3239430eac4
parent54cc0c5b296ee7c27b48a3c8e7aead6e74f2abf1
Turn off Windows process ACL restriction by default.

As documented in bug 'win-process-acl-finesse', we've had enough
assorted complaints about it breaking various non-malicious pieces of
Windows process interaction (ranging from git->plink integration to
screen readers for the vision-impaired) that I think it's more
sensible to set the process back to its default level of protection.

This precaution was never a fully effective protection anyway, due to
the race condition at process startup; the only properly effective
defence would have been to prevent malware running under the same user
ID as PuTTY in the first place, so in that sense, nothing has changed.
But people who want the arguable defence-in-depth advantage of the ACL
restriction can now turn it on with the '-restrict-acl' command-line
option, and it's up to them whether they can live with the assorted
inconveniences that come with it.

In the course of this change, I've centralised a bit more of the
restriction code into winsecur.c, to avoid repeating the error
handling in multiple places.
Recipe
cmdline.c
doc/using.but
windows/window.c
windows/winpgen.c
windows/winpgnt.c
windows/winplink.c
windows/winsecur.c
windows/winsecur.h
windows/winsftp.c
windows/winstuff.h