Fix bug in copy_email() which would not
find emailAddress at start of subject name.
This commit is contained in:
parent
3d2e469cfa
commit
f23478c314
2 changed files with 5 additions and 1 deletions
4
CHANGES
4
CHANGES
|
@ -3,6 +3,10 @@
|
|||
|
||||
Changes between 0.9.6 and 0.9.7 [xx XXX 2000]
|
||||
|
||||
*) In copy_email() check for >= 0 as a return value for
|
||||
X509_NAME_get_index_by_NID() since 0 is a valid index.
|
||||
[Steve Henson reported by Massimiliano Pala <madwolf@opensca.org>]
|
||||
|
||||
*) In BN_div() keep a copy of the sign of 'num' before writing the
|
||||
result to 'rm' because if rm==num the value will be overwritten
|
||||
and produce the wrong result if 'num' is negative: this caused
|
||||
|
|
|
@ -318,7 +318,7 @@ static int copy_email(X509V3_CTX *ctx, GENERAL_NAMES *gens)
|
|||
/* Now add any email address(es) to STACK */
|
||||
i = -1;
|
||||
while((i = X509_NAME_get_index_by_NID(nm,
|
||||
NID_pkcs9_emailAddress, i)) > 0) {
|
||||
NID_pkcs9_emailAddress, i)) >= 0) {
|
||||
ne = X509_NAME_get_entry(nm, i);
|
||||
email = M_ASN1_IA5STRING_dup(X509_NAME_ENTRY_get_data(ne));
|
||||
if(!email || !(gen = GENERAL_NAME_new())) {
|
||||
|
|
Loading…
Reference in a new issue