simon-git: putty (pre-0.75): Simon Tatham

Commits to Tartarus hosted VCS tartarus-commits at lists.tartarus.org
Sun Apr 25 06:20:52 BST 2021


TL;DR:
  17371e0d Fix named_pipe_agent_exists(), which just didn't work.

Repository:     https://git.tartarus.org/simon/putty.git
On the web:     https://git.tartarus.org/?p=simon/putty.git
Branch updated: pre-0.75
Committer:      Simon Tatham <anakin at pobox.com>
Date:           2021-04-25 06:20:52

commit 17371e0df05ccf57f858b852b8bec632ac889a18
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=17371e0df05ccf57f858b852b8bec632ac889a18;hp=f69cf86a61a8109e3d6f3fe2168087c9819d4b2e
Author: Simon Tatham <anakin at pobox.com>
Date:   Sun Apr 25 06:07:31 2021 +0100

    Fix named_pipe_agent_exists(), which just didn't work.
    
    GetFileType() takes a HANDLE, not a pathname. So passing it the
    pathname of the agent named pipe would never have worked at all.
    
    I hadn't noticed, because the only call to that function logical-ORs
    its return value with that of wm_copydata_agent_exists(), and the
    latter _does_ work.
    
    So if you're running true Pageant, which presents both IPC interfaces,
    then there's no problem. But if a Pageant-emulating system wanted to
    present only the named-pipe version, then we wouldn't have detected
    it. Now we should do.

 windows/winpgntc.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)



More information about the tartarus-commits mailing list