Re: warnings as errors crypto/bn/bn_mul.c 1.15

[Richard Levitte - VMS Whacker]

From: Jeffrey Altman <[EMAIL PROTECTED]>

jaltman> > Does it get better if you change all occurences of "(-t-c)" to "(0-t-c)"?
jaltman> 
jaltman> That is what I did to make the warnings go away.  But I'm not 
jaltman> confident as to the lack of overflow or underflow or ....
jaltman> since we are computing a signed value and storing it back to
jaltman> an unsigned value

Not a problem, the values are supposed to wrap around, which they will
do correctly on any 2-complement machine.  The lines following are
there to detect the possible carry bit (which is also the overflow).

-- 
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]