"Richard Levitte - VMS Whacker" <[EMAIL PROTECTED]> said:

> OK, this is very odd.  Can you check against the two attached
> archives?  In those files, it looks like the result of your proposed
> patch.

No, the files in you attachment are wrong. It's quite simple; 
'DSA_generate_parameters()' should have 7 parameters (not 6 as now). 
Look at the protype in <crypto/dsa/dsa.h> and implementation in
crypto/dsa/dsa_depr.c

Similarily for 'X509V3_EXT_print_fp()' should have 4 args (not 3).
Since one of us is confused, here is the patch again.

-------------------------------------------------------------
--- ./crypto/dsa/dsagen.c.orig
+++ ./crypto/dsa/dsagen.c
@@ -103,7 +103,7 @@
                bio_err=BIO_new_fp(stderr,BIO_NOCLOSE);

        memcpy(seed_buf,seed,20);
+       dsa=DSA_generate_parameters(1024,seed,20,&counter,&h,cb,bio_err);
-       dsa=DSA_generate_parameters(1024,seed,20,&counter,&h,cb);

        if (dsa == NULL)
                DSA_print(bio_err,dsa,0);

--- ./crypto/x509v3/v3conf.c.orig
+++ ./crypto/x509v3/v3conf.c
@@ -118,7 +118,7 @@
                printf("%s", OBJ_nid2ln(OBJ_obj2nid(ext->object)));
                if(ext->critical) printf(",critical:\n");
                else printf(":\n");
+               X509V3_EXT_print_fp(stdout, ext, 0, 0);
-               X509V3_EXT_print_fp(stdout, ext, 0);
                printf("\n");

        }
-------------------------------------------------------------

Gisle V.



______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [EMAIL PROTECTED]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to