From: Werner Koch <wk@gnupg.org>
Date: Fri, 24 Oct 2008 14:07:14 +0000 (+0000)
Subject: Fix last change.
X-Git-Tag: gpgme-1.1.8~10
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=6ce90fccd90e18115471957152ff98fc08e7eb2e;p=gpgme.git

Fix last change.
---

diff --git a/gpgme/ChangeLog b/gpgme/ChangeLog
index e9d8675..f848d18 100644
--- a/gpgme/ChangeLog
+++ b/gpgme/ChangeLog
@@ -1,3 +1,7 @@
+2008-10-24  Werner Koch  <wk@g10code.com>
+
+	* rungpg.c (gpg_keylist_preprocess): Escape backslashes too.
+
 2008-10-23  Marcus Brinkmann  <marcus@g10code.de>
 
 	* rungpg.c (gpg_keylist_preprocess): Convert percent escaped
diff --git a/gpgme/rungpg.c b/gpgme/rungpg.c
index 7524d68..fc0da11 100644
--- a/gpgme/rungpg.c
+++ b/gpgme/rungpg.c
@@ -1884,9 +1884,8 @@ gpg_keylist_preprocess (char *line, char **r_line)
       {
 	/* The user ID is percent escaped, but we want c-coded.
 	   Because we have to replace each '%HL' by '\xHL', we need at
-	   most 4/3 th the number of bytes.  But because this 
-	   security software, we err on the good side and allocate
-	   twice as much.  */
+	   most 4/3 th the number of bytes.  But because we also need
+	   to escape the backslashes we allocate twice as much.  */
 	char *uid = malloc (2 * strlen (field[1]) + 1);
 	char *src;
 	char *dst;
@@ -1902,14 +1901,17 @@ gpg_keylist_preprocess (char *line, char **r_line)
 		*(dst++) = '\\';
 		*(dst++) = 'x';
 		src++;
-		/* Copy the next two bytes unconditionally.  This is
-		   what reduces the maximum number of needed bytes
-		   from 2n+1 to (4/3)n+1, even for invalid strings.  */
+		/* Copy the next two bytes unconditionally.  */
 		if (*src)
 		  *(dst++) = *(src++);
 		if (*src)
 		  *(dst++) = *(src++);
 	      }
+	    else if (*src == '\\')
+              {
+                *dst++ = '\\';
+                *dst++ = '\\';
+              }
 	    else
 	      *(dst++) = *(src++);
 	  }