Honeywell Internal
I think I have an answer for why this commit needed to be reverted:
Author: Dr. Stephen Henson <st...@openssl.org>
Author date: 2 years ago (12/20/2015 10:18:43 AM)
Commit date: 2 years ago (12/20/2015 11:27:03 AM)
Commit hash: 6656ba7152dfe4bba865e327dd362ea08544aa80
Children: 1c7de36f62
Parent(s): 17592f323a
Don't check RSA_FLAG_SIGN_VER.
Reviewed-by: Richard Levitte <levi...@openssl.org>
The change made in that commit was to simply remove the attempt to check for
the RSA_FLAG_SIGN_VER flag. But that’s not what is wrong with this code that
required changing.
The change should be to add “meth” prior to flags:
@@ -84,7 +89,7 @@ int RSA_sign(int type, const unsigned char *m, unsigned int
m_len,
return 0;
}
#endif
- if ((rsa->flags & RSA_FLAG_SIGN_VER) && rsa->meth->rsa_sign) {
+ if ((rsa->meth->flags & RSA_FLAG_SIGN_VER) && rsa->meth->rsa_sign) {
return rsa->meth->rsa_sign(type, m, m_len, sigret, siglen, rsa);
}
/* Special case: SSL signature, just check the length */
@@ -293,7 +298,7 @@ int RSA_verify(int dtype, const unsigned char *m, unsigned
int m_len,
const unsigned char *sigbuf, unsigned int siglen, RSA *rsa)
{
- if ((rsa->flags & RSA_FLAG_SIGN_VER) && rsa->meth->rsa_verify) {
+ if ((rsa->meth->flags & RSA_FLAG_SIGN_VER) && rsa->meth->rsa_verify) {
return rsa->meth->rsa_verify(dtype, m, m_len, sigbuf, siglen, rsa);
}
--
-Ike-
John Eichenberger
Intermec by Honeywell
Principal Engineer: Sustaining Engineering
425.921.4507
--
openssl-users mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-users
