Home GnuPG
Diffusion GnuPG 90688b29f370

agent: Fix UPDATESTARTUPTTY for ssh.
90688b29f370Unpublished

Unpublished Commit · Learn More

Not On Permanent Ref: This commit is not an ancestor of any permanent ref.

Description

agent: Fix UPDATESTARTUPTTY for ssh.

* agent/command-ssh.c (setup_ssh_env): Fix env setting.

gniibe reported this to gnupg-devel on 2012-07-04:

[...]
(2) UPDATESTARTUPTTY doesn't work to switch TTY for pinentry for
    SSH.

[...]

Current implementation:

    In the function start_command_handler_ssh, the logic puts
    priority on ctrl->session_env which is initialized by
    agent_init_default_ctrl.  There are always GPG_TTY and TERM
    defined, because lines around 968 in gpg-agent.c, it says:

	/* Make sure that we have a default ttyname. */

    While UPDATESTARTUPTTY updates opt.startup_env, it doesn't
    affect at all.

Here is a patch to point the issue.  Tested and works for me.

(cherry picked from commit 9f5578c29adba6d4f7d3650121d07322c2f8d254)

  • Signed-off-by: Werner Koch <wk@gnupg.org>

Details

Provenance
wernerAuthored on Aug 19 2013, 11:44 AM
Parents
rG7c4bfa599fa6: gpgv: Init Libgcrypt to avoid syslog warning.
Branches
Unknown
Tags
Unknown

Event Timeline

Werner Koch <wk@gnupg.org> committed rG90688b29f370: agent: Fix UPDATESTARTUPTTY for ssh. (authored by Werner Koch <wk@gnupg.org>).Mar 7 2014, 9:59 AM