diff options
| author | David Benjamin <davidben@chromium.org> | 2014-09-30 21:00:38 -0400 |
|---|---|---|
| committer | Adam Langley <agl@google.com> | 2014-10-01 02:17:38 +0000 |
| commit | fb3ff2c66cc873150022a4ab87ac72dd9d78b549 (patch) | |
| tree | 1a1c9e5afda5354c6e3a4f0442699703c9141301 /crypto/rsa | |
| parent | 5e77bd449e9d7050d59271db157e3fc8957e23e9 (diff) | |
| download | boringssl-fb3ff2c66cc873150022a4ab87ac72dd9d78b549.zip boringssl-fb3ff2c66cc873150022a4ab87ac72dd9d78b549.tar.gz boringssl-fb3ff2c66cc873150022a4ab87ac72dd9d78b549.tar.bz2 | |
Don't compare signed vs. unsigned.
This resolves a pile of MSVC warnings in Chromium.
Change-Id: Ib9a29cb88d8ed8ec4118d153260f775be059a803
Reviewed-on: https://boringssl-review.googlesource.com/1865
Reviewed-by: Adam Langley <agl@google.com>
Diffstat (limited to 'crypto/rsa')
| -rw-r--r-- | crypto/rsa/padding.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/crypto/rsa/padding.c b/crypto/rsa/padding.c index 3631424..4d29b07 100644 --- a/crypto/rsa/padding.c +++ b/crypto/rsa/padding.c @@ -55,6 +55,8 @@ #include <openssl/rsa.h> +#include <assert.h> + #include <openssl/digest.h> #include <openssl/err.h> #include <openssl/mem.h> @@ -659,7 +661,7 @@ int RSA_padding_add_PKCS1_PSS_mgf1(RSA *rsa, unsigned char *EM, int sLen) { int i; int ret = 0; - int maskedDBLen, MSBits, emLen; + size_t maskedDBLen, MSBits, emLen; size_t hLen; unsigned char *H, *salt = NULL, *p; EVP_MD_CTX ctx; @@ -693,6 +695,7 @@ int RSA_padding_add_PKCS1_PSS_mgf1(RSA *rsa, unsigned char *EM, MSBits = (BN_num_bits(rsa->n) - 1) & 0x7; emLen = RSA_size(rsa); if (MSBits == 0) { + assert(emLen >= 1); *EM++ = 0; emLen--; } |