simon-git: putty (main): Simon Tatham

Commits to Tartarus hosted VCS tartarus-commits at lists.tartarus.org
Thu May 13 18:43:16 BST 2021


TL;DR:
  155d8121 Fix confusion between invalid Windows HANDLE values.
  6e69223d Close agent named-pipe handles when queries complete.
  6791bdc9 Don't #include <utmp.h> if it doesn't exist.

Repository:     https://git.tartarus.org/simon/putty.git
On the web:     https://git.tartarus.org/?p=simon/putty.git
Branch updated: main
Committer:      Simon Tatham <anakin at pobox.com>
Date:           2021-05-13 18:43:16

commit 155d8121e6584f842fa06f5fbad75b2555f60269
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=155d8121e6584f842fa06f5fbad75b2555f60269;hp=0e83e72b09744e89c459c8156ac88cb07f35065e
Author: Simon Tatham <anakin at pobox.com>
Date:   Thu May 13 18:20:41 2021 +0100

    Fix confusion between invalid Windows HANDLE values.
    
    I was checking a HANDLE against INVALID_HANDLE_VALUE to decide whether
    it should be closed. But ten lines further up, I was setting it
    manually to NULL to suppress the close. Oops.

 windows/agent-client.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

commit 6e69223dc262755c2cd2a3bba5c188d8fc91943a
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=6e69223dc262755c2cd2a3bba5c188d8fc91943a;hp=155d8121e6584f842fa06f5fbad75b2555f60269
Author: Simon Tatham <anakin at pobox.com>
Date:   Thu May 13 18:22:05 2021 +0100

    Close agent named-pipe handles when queries complete.
    
    I was cleaning up the 'struct handle', but not the underlying HANDLE.
    As a result, any PuTTY process that makes a request to Pageant keeps
    the named pipe connection open until the end of the process's
    lifetime.

 windows/agent-client.c | 3 +++
 1 file changed, 3 insertions(+)

commit 6791bdc9b6edcf67a5da718b50c0444ac512c935
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=6791bdc9b6edcf67a5da718b50c0444ac512c935;hp=6e69223dc262755c2cd2a3bba5c188d8fc91943a
Author: Simon Tatham <anakin at pobox.com>
Date:   Thu May 13 18:40:05 2021 +0100

    Don't #include <utmp.h> if it doesn't exist.
    
    A FreeBSD user reports that it doesn't exist there.

 cmake/cmake.h.in           | 1 +
 cmake/platforms/unix.cmake | 1 +
 unix/pty.c                 | 2 ++
 3 files changed, 4 insertions(+)



More information about the tartarus-commits mailing list