aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWerner Koch <[email protected]>2024-01-09 09:09:36 +0000
committerWerner Koch <[email protected]>2024-01-09 09:13:54 +0000
commit45f6357881459dcb6b2b78e475d1c136bcb6f606 (patch)
tree33d18825ff3df0852307b4022a71640243f6d7e7
parentgpg: Improve error return for --quick-add-subkey and -add-adsk. (diff)
downloadgnupg-45f6357881459dcb6b2b78e475d1c136bcb6f606.tar.gz
gnupg-45f6357881459dcb6b2b78e475d1c136bcb6f606.zip
common,w32: Remove duplicated backslashes when setting the homedir.
* common/homedir.c (copy_dir_with_fixup) [W32]: Fold double backslashes. -- This is in general no problem but when we hash or compare the directory to test whether tit is the standard home directory, we may use a different socket file and thus a second instance of a daemon. GnuPG-bug-id: 6833
-rw-r--r--NEWS3
-rw-r--r--common/homedir.c27
2 files changed, 30 insertions, 0 deletions
diff --git a/NEWS b/NEWS
index 889aff8e1..7b99deeab 100644
--- a/NEWS
+++ b/NEWS
@@ -3,6 +3,9 @@ Noteworthy changes in version 2.4.4 (unreleased)
* gpgsm: Support ECDSA in de-vs compliance mode. [T6802]
+ * dirmngr: Avoid starting a second instance on Windows via GPGME
+ based launching. [T6833]
+
* Fix garbled time output in non-English Windows. [T6741]
Release-info: https://dev.gnupg.org/T6578
diff --git a/common/homedir.c b/common/homedir.c
index 2b99c9bdc..6f99f3eab 100644
--- a/common/homedir.c
+++ b/common/homedir.c
@@ -222,6 +222,10 @@ copy_dir_with_fixup (const char *newdir)
{
char *result = NULL;
char *p;
+#ifdef HAVE_W32_SYSTEM
+ char *p0;
+ const char *s;
+#endif
if (!*newdir)
return NULL;
@@ -253,6 +257,29 @@ copy_dir_with_fixup (const char *newdir)
*p-- = 0;
}
+ /* Hack to mitigate badly doubled backslashes. */
+ s = result? result : newdir;
+ if (s[0] == '\\' && s[1] == '\\' && s[2] != '\\')
+ {
+ /* UNC (\\Servername\file) or Long UNC (\\?\Servername\file)
+ * Does not seem to be double quoted. */
+ }
+ else if (strstr (s, "\\\\"))
+ {
+ /* Double quotes detected. Fold them into one because that is
+ * what what Windows does. This way we get a unique hash
+ * regardless of the number of doubled backslashes. */
+ if (!result)
+ result = xstrdup (newdir);
+ for (p0=p=result; *p; p++)
+ {
+ *p0++ = *p;
+ while (*p == '\\' && p[1] == '\\')
+ p++;
+ }
+ *p0 = 0;
+ }
+
#else /*!HAVE_W32_SYSTEM*/
if (newdir[strlen (newdir)-1] == '/')