.
[gnupg.git] / doc / vuln-announce-2010-kbx-realloc.txt
1              Realloc Bug with X.509 certificates in GnuPG
2             ==============================================
3                               2010-07-23
4
5
6 Summary
7 =======
8
9 While trying to import a server certificate for a CDN service, a segv
10 bug was found in GnuPG's GPGSM tool.  It is likely that this bug is
11 exploitable by sending a special crafted signed message and having a
12 user verify the signature.
13
14 [ Please do not send private mail in response to this message.  The
15   mailing list gnupg-devel is the best place to discuss this problem
16   (please subscribe first so you don't need moderator approval [1]). ]
17
18
19 Impact
20 ======
21
22 All applications using GnuPG's GPGSM tool to process S/MIME messages
23 or manage X.509 certificates are affected.  The bug exists in all
24 versions of GnuPG including the recently released GnuPG 2.0.16.
25
26 GnuPG 1.x is not affected because it does not come with the GPGSM
27 tool.
28
29 An exploit is not yet known but it can't be ruled out for sure that
30 the problem has not already been identified by some dark forces.
31
32
33 Description
34 ===========
35
36 Importing a certificate with more than 98 Subject Alternate Names [2]
37 via GPGSM's import command or implicitly while verifying a signature
38 causes GPGSM to reallocate an array with the names.  The bug is that
39 the reallocation code misses assigning the reallocated array to the
40 old array variable and thus the old and freed array will be used.
41 Usually this leads to a segv.
42
43 It might be possible to use one of the techniques to exploit
44 assignments to malloced and freed memory.  Such an exploit won't be
45 easy to write because the attack vector must fit into a valid ASN.1
46 DER encoded DN.  To further complicate the task, that DN is not used
47 directly but after a transformation to RFC-2253 format.
48
49
50
51 Solution
52 ========
53
54 Apply the following patch.  The patch is required for all GnuPG
55 versions < 2.0.17.  It applies to 2.0.16 but should apply to many
56 older versions as well.
57
58
59 --- kbx/keybox-blob.c   (revision 5367)
60 +++ kbx/keybox-blob.c   (working copy)
61 @@ -898,6 +898,7 @@
62                rc = gpg_error_from_syserror ();
63                goto leave;
64              }
65 +          names = tmp;
66          }
67        names[blob->nuids++] = p;
68        if (!i && (p=x509_email_kludge (p)))
69
70
71
72
73 Support 
74 =======
75
76 g10 Code GmbH [3], a Duesseldorf based company owned and headed by
77 GnuPG's principal author, is currently funding GnuPG development.
78 Support contracts or other financial backing will greatly help us to
79 improve the quality of GnuPG.
80
81
82 Thanks
83 ======
84
85 Peter Gutmann for his "A mighty fortress is our PKI" mail to the
86 cryptography ML which contained a pointer to a certificate to exhibit
87 the problem.  This bug was created, found and fixed by Werner Koch.
88
89
90 [1] See http://lists.gnupg.org/mailman/listinfo/gnupg-devel
91 [2] <http://cvs.gnupg.org/cgi-bin/viewcvs.cgi/*checkout*/trunk/tests/samplekeys/cert-with-117-akas.pem>
92 [3] See http://www.gnupg.org/service.html