Hi! > Bobby Krupczak wrote: >> 1) In bn_rand.c add at line 141: memset(buf, 0, bytes); =20 >> >> buf = (unsigned char *)OPENSSL_malloc(bytes); >> if (buf == NULL) >> { >> BNerr(BN_F_BNRAND,ERR_R_MALLOC_FAILURE); >> goto err; >> } >> memset(buf, 0, bytes); > > >> 2) bn_mont.c: Initialize tmod variable declared at line 392 >> memset(&tmod, 0, sizeof(tmod)); > > > Maybe these two items are genuine bugs ? If some code should be using > calloc() instead of malloc() in the first case ? > > > You have have any simple code that exposes those bugs, simply meaning just > a page of code that when run under valgrind exposes uninit'ed data is being > used (even after a -DPURIFY version of OpenSSL is used).
The example client/server code from the book NSwO demonstrated the problem even after -DPURIFY was used to build openssl. I'm not sure it is a bug or not. I guess it would be up to the openssl maintainers and developers to decide. Thanks, Bobby ______________________________________________________________________ OpenSSL Project http://www.openssl.org User Support Mailing List openssl-users@openssl.org Automated List Manager [EMAIL PROTECTED]