Re: cvs commit: openssl/crypto/bnbn.hbn_exp.cbn_exp2.cbn_gcd.cbn_lib.c bn_mont.c bn_mul.cbn_prime.cbn_recp.cbn_shift.cbn_sqr.cbntest.c

[Richard Levitte - VMS Whacker]

From: Ben Laurie <[EMAIL PROTECTED]>

ben> BTW, Bodo mentioned that dup_expand was dupping regardless of whether it
ben> expands - if so, that'll slow it down a lot, I'd guess. Did you fix
ben> that?

Depends on what you mean with "fix".  As it is in the repository right
now, no.  That's exactly what is required if constness shall be
perfectly clean, since BN_mul needs to make absolutely sure the extra
data is 0.  Since the BIGNUM may already be of the correct size
(remember, top and dmax are not the same thing), there may as well
some random data in that last word.  The only way to solve that so
constness wasn't broken was to dup it "needlessly".

In my work directory, I currently have the variant that I mentioned to
Rich Saltz a few messages ago, which basically breaks constness.

-- 
Richard Levitte   \ Spannv�gen 38, II \ [EMAIL PROTECTED]
Chairman@Stacken   \ S-168 35  BROMMA  \ T: +46-8-26 52 47
Redakteur@Stacken   \      SWEDEN       \ or +46-709-50 36 10
Procurator Odiosus Ex Infernis                -- [EMAIL PROTECTED]
Member of the OpenSSL development team: http://www.openssl.org/
Software Engineer, Celo Communications: http://www.celocom.com/

Unsolicited commercial email is subject to an archival fee of $400.
See <http://www.stacken.kth.se/~levitte/mail/> for more info.
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [EMAIL PROTECTED]
Automated List Manager                           [EMAIL PROTECTED]