[EMAIL PROTECTED] - Thu Jul 20 17:27:31 2006]:

> 
> Yeah. And I even managed to reproduce it on solaris64-sparcv9-cc. So 
> it's our bug, not compiler. Verify below patch. Why I commented on gcc 
> and being root thing? Well, for future reference. If you run into 
> compiler bug, you can't expect us to test several compilers. You should 
> be prepared to do that without false excuses:-) A.
> 
> --- crypto/pkcs7/pk7_doit.c     10 Jul 2006 18:36:53 -0000      1.72
> +++ crypto/pkcs7/pk7_doit.c     20 Jul 2006 15:19:58 -0000
> @@ -829,7 +829,7 @@
>          EVP_PKEY_CTX *pctx;
>          unsigned char *abuf = NULL;
>          int alen;
> -       unsigned int siglen;
> +       size_t siglen;
>          const EVP_MD *md = NULL;
> 
>          md = EVP_get_digestbyobj(si->digest_alg->algorithm);
> 

Oops, mea culpa on that. I changed it from the old EVP_Sign*() interface
to EVP_DigestSign(). The old one was unsigned int * for the sig length
the new one (in line with other things) uses size_t *.

Which causes problems is sizeof(size_t) == sizeof(unsigned int).

Is there a compiler warning when compiling that file?
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [email protected]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to