From: Marcus Brinkmann Date: Thu, 23 Oct 2008 10:51:31 +0000 (+0000) Subject: 2008-10-23 Marcus Brinkmann X-Git-Tag: gpgme-1.1.8~11 X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=69b1a65fb8138a46db6b006c54a10ceb942549df;p=gpgme.git 2008-10-23 Marcus Brinkmann * rungpg.c (gpg_keylist_preprocess): Convert percent escaped string to C coded string. --- diff --git a/gpgme/ChangeLog b/gpgme/ChangeLog index 2739952..e9d8675 100644 --- a/gpgme/ChangeLog +++ b/gpgme/ChangeLog @@ -1,3 +1,8 @@ +2008-10-23 Marcus Brinkmann + + * rungpg.c (gpg_keylist_preprocess): Convert percent escaped + string to C coded string. + 2008-10-20 Werner Koch * Makefile.am (EXTRA_DIST): Add gpgme.h.in. diff --git a/gpgme/rungpg.c b/gpgme/rungpg.c index 76bcaae..7524d68 100644 --- a/gpgme/rungpg.c +++ b/gpgme/rungpg.c @@ -1878,12 +1878,47 @@ gpg_keylist_preprocess (char *line, char **r_line) HTTP Keyserver Protocol (draft). We want: - uid:o::::::::: + uid:o::::::::: */ - if (asprintf (r_line, "uid:o%s::::%s:%s:::%s:", - field[4], field[2], field[3], field[1]) < 0) - return gpg_error_from_errno (errno); + { + /* 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. */ + char *uid = malloc (2 * strlen (field[1]) + 1); + char *src; + char *dst; + + if (! uid) + return gpg_error_from_errno (errno); + src = field[1]; + dst = uid; + while (*src) + { + if (*src == '%') + { + *(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. */ + if (*src) + *(dst++) = *(src++); + if (*src) + *(dst++) = *(src++); + } + else + *(dst++) = *(src++); + } + *dst = '\0'; + + if (asprintf (r_line, "uid:o%s::::%s:%s:::%s:", + field[4], field[2], field[3], uid) < 0) + return gpg_error_from_errno (errno); + } return 0; case RT_NONE: