Dr. Stephen Henson wrote:
> OpenSSL CVS Repository
> http://cvs.openssl.org/
> ____________________________________________________________________________
>
> Server: cvs.openssl.org Name: Dr. Stephen Henson
> Root: /v/openssl/cvs Email: [EMAIL PROTECTED]
> Module: openssl Date: 02-Nov-2008 19:29:27
> Branch: HEAD Handle: 2008110218292700
>
> Modified files:
> openssl/crypto/rand md_rand.c
>
> Log:
> Not sure about this one... seems to be needed to make 64 bit release
> builds work properly...
k can be < 0, but not j. In both cases, it turns out. But interesting
that it only shows up in 64 bit?
Also, getting the types "right" is all very well, but this is a great
example where it doesn't really work unless you also do range checking
(e.g. a size_t with top bit set is not going to work properly when
converted to a ssize_t).
>
> Summary:
> Revision Changes Path
> 1.81 +2 -2 openssl/crypto/rand/md_rand.c
> ____________________________________________________________________________
>
> patch -p0 <<'@@ .'
> Index: openssl/crypto/rand/md_rand.c
> ============================================================================
> $ cvs diff -u -r1.80 -r1.81 md_rand.c
> --- openssl/crypto/rand/md_rand.c 1 Nov 2008 16:40:34 -0000 1.80
> +++ openssl/crypto/rand/md_rand.c 2 Nov 2008 18:29:27 -0000 1.81
> @@ -190,7 +190,7 @@
> static void ssleay_rand_add(const void *buf, size_t num, double add)
> {
> int i,st_idx;
> - size_t j,k;
> + ssize_t j,k;
> long md_c[2];
> unsigned char local_md[MD_DIGEST_LENGTH];
> EVP_MD_CTX m;
> @@ -325,7 +325,7 @@
> {
> static volatile int stirred_pool = 0;
> int i,st_num,st_idx;
> - size_t j,k;
> + ssize_t j,k;
> int num_ceil;
> int ok;
> long md_c[2];
> @@ .
> ______________________________________________________________________
> OpenSSL Project http://www.openssl.org
> CVS Repository Commit List [EMAIL PROTECTED]
> Automated List Manager [EMAIL PROTECTED]
>
>
--
http://www.apache-ssl.org/ben.html http://www.links.org/
"There is no limit to what a man can do or how far he can go if he
doesn't mind who gets the credit." - Robert Woodruff
______________________________________________________________________
OpenSSL Project http://www.openssl.org
Development Mailing List [email protected]
Automated List Manager [EMAIL PROTECTED]
