[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: [PATCH] gpg-agent: search gpg-agent socket in $GNUPGHOME

From: Bert Huijben <bert_at_qqmail.nl>
Date: Thu, 27 Oct 2016 11:54:06 +0200

> -----Original Message-----
> From: Patrick Steinhardt [mailto:ps_at_pks.im]
> Sent: donderdag 27 oktober 2016 11:15
> To: Subversion <dev_at_subversion.apache.org>
> Subject: [PATCH] gpg-agent: search gpg-agent socket in $GNUPGHOME
>
> Hi,
>
> a recent thread on issues with gpg-agent lead me to take a look
> at how the gpg-agent socket is located in subversion. The current
> code was lacking support for $GNUPGHOME, which allows a user to
> relocate his gnupg configuration directory. As setting this
> environment variable would also cause S.gpg-agent to be created
> inside of $GNUPGHOME, we might fail to correctly locate the file
> in this case.
>
> Attached patch fixes the problem.

        Hi Patrick,

Last week (in r1766327) a slightly different patch was committed to resolve
this (or a similar problem).

This patch was nominated for backport to 1.9.x, so test input is very
welcome.

Do you know if we need both patches?

        Bert
>
> [[[
> gpg_agent: search in $GNUPGHOME for gpg-agent socket
>
> The socket used to connect to the gpg-agent resides in the GnuPG
> home directory, which is by default located at "$HOME/.gnupg".
> But in fact, the home directory can be relocated by the user by
> setting the environment variable GNUPGHOME, in which case the
> gpg-agent socket will live at "$GNUPGHOME/S.gpg-agent".
> Subversion does only search the standard home directory, though,
> without evaluating $GNUPGHOME.
>
> Fix the issue by using the socket located at
> "$GNUPGHOME/S.gpg-agent" instead of using "$HOME/.gnupg" when the
> environment variable is set.
>
> * subversion/libsvn_subr/gpg_agent.c
> (find_running_gpg_agent): evaluate $GNUPGHOME
> ]]]
Received on 2016-10-27 11:54:30 CEST

This is an archived mail posted to the Subversion Dev mailing list.