diff options
Diffstat (limited to 'src/lib/libcrypto/evp')
-rw-r--r-- | src/lib/libcrypto/evp/e_aes_cbc_hmac_sha1.c | 5 | ||||
-rw-r--r-- | src/lib/libcrypto/evp/encode.c | 13 | ||||
-rw-r--r-- | src/lib/libcrypto/evp/evp_enc.c | 4 |
3 files changed, 7 insertions, 15 deletions
diff --git a/src/lib/libcrypto/evp/e_aes_cbc_hmac_sha1.c b/src/lib/libcrypto/evp/e_aes_cbc_hmac_sha1.c index 8d33896e1c..42aa20701b 100644 --- a/src/lib/libcrypto/evp/e_aes_cbc_hmac_sha1.c +++ b/src/lib/libcrypto/evp/e_aes_cbc_hmac_sha1.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: e_aes_cbc_hmac_sha1.c,v 1.10 2016/05/03 12:38:53 tedu Exp $ */ | 1 | /* $OpenBSD: e_aes_cbc_hmac_sha1.c,v 1.11 2016/05/04 14:53:29 tedu Exp $ */ |
2 | /* ==================================================================== | 2 | /* ==================================================================== |
3 | * Copyright (c) 2011-2013 The OpenSSL Project. All rights reserved. | 3 | * Copyright (c) 2011-2013 The OpenSSL Project. All rights reserved. |
4 | * | 4 | * |
@@ -60,7 +60,6 @@ | |||
60 | #include <openssl/aes.h> | 60 | #include <openssl/aes.h> |
61 | #include <openssl/sha.h> | 61 | #include <openssl/sha.h> |
62 | #include "evp_locl.h" | 62 | #include "evp_locl.h" |
63 | #include "constant_time_locl.h" | ||
64 | 63 | ||
65 | #ifndef EVP_CIPH_FLAG_AEAD_CIPHER | 64 | #ifndef EVP_CIPH_FLAG_AEAD_CIPHER |
66 | #define EVP_CIPH_FLAG_AEAD_CIPHER 0x200000 | 65 | #define EVP_CIPH_FLAG_AEAD_CIPHER 0x200000 |
@@ -283,8 +282,6 @@ aesni_cbc_hmac_sha1_cipher(EVP_CIPHER_CTX *ctx, unsigned char *out, | |||
283 | maxpad |= (255 - maxpad) >> (sizeof(maxpad) * 8 - 8); | 282 | maxpad |= (255 - maxpad) >> (sizeof(maxpad) * 8 - 8); |
284 | maxpad &= 255; | 283 | maxpad &= 255; |
285 | 284 | ||
286 | ret &= constant_time_ge(maxpad, pad); | ||
287 | |||
288 | inp_len = len - (SHA_DIGEST_LENGTH + pad + 1); | 285 | inp_len = len - (SHA_DIGEST_LENGTH + pad + 1); |
289 | mask = (0 - ((inp_len - len) >> | 286 | mask = (0 - ((inp_len - len) >> |
290 | (sizeof(inp_len) * 8 - 1))); | 287 | (sizeof(inp_len) * 8 - 1))); |
diff --git a/src/lib/libcrypto/evp/encode.c b/src/lib/libcrypto/evp/encode.c index e4104173d4..637870cef6 100644 --- a/src/lib/libcrypto/evp/encode.c +++ b/src/lib/libcrypto/evp/encode.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: encode.c,v 1.22 2016/05/03 14:05:41 bcook Exp $ */ | 1 | /* $OpenBSD: encode.c,v 1.23 2016/05/04 14:53:29 tedu Exp $ */ |
2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) | 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) |
3 | * All rights reserved. | 3 | * All rights reserved. |
4 | * | 4 | * |
@@ -125,13 +125,13 @@ EVP_EncodeUpdate(EVP_ENCODE_CTX *ctx, unsigned char *out, int *outl, | |||
125 | const unsigned char *in, int inl) | 125 | const unsigned char *in, int inl) |
126 | { | 126 | { |
127 | int i, j; | 127 | int i, j; |
128 | size_t total = 0; | 128 | unsigned int total = 0; |
129 | 129 | ||
130 | *outl = 0; | 130 | *outl = 0; |
131 | if (inl == 0) | 131 | if (inl == 0) |
132 | return; | 132 | return; |
133 | OPENSSL_assert(ctx->length <= (int)sizeof(ctx->enc_data)); | 133 | OPENSSL_assert(ctx->length <= (int)sizeof(ctx->enc_data)); |
134 | if (ctx->length - ctx->num > inl) { | 134 | if ((ctx->num + inl) < ctx->length) { |
135 | memcpy(&(ctx->enc_data[ctx->num]), in, inl); | 135 | memcpy(&(ctx->enc_data[ctx->num]), in, inl); |
136 | ctx->num += inl; | 136 | ctx->num += inl; |
137 | return; | 137 | return; |
@@ -148,7 +148,7 @@ EVP_EncodeUpdate(EVP_ENCODE_CTX *ctx, unsigned char *out, int *outl, | |||
148 | *out = '\0'; | 148 | *out = '\0'; |
149 | total = j + 1; | 149 | total = j + 1; |
150 | } | 150 | } |
151 | while (inl >= ctx->length && total <= INT_MAX) { | 151 | while (inl >= ctx->length) { |
152 | j = EVP_EncodeBlock(out, in, ctx->length); | 152 | j = EVP_EncodeBlock(out, in, ctx->length); |
153 | in += ctx->length; | 153 | in += ctx->length; |
154 | inl -= ctx->length; | 154 | inl -= ctx->length; |
@@ -157,11 +157,6 @@ EVP_EncodeUpdate(EVP_ENCODE_CTX *ctx, unsigned char *out, int *outl, | |||
157 | *out = '\0'; | 157 | *out = '\0'; |
158 | total += j + 1; | 158 | total += j + 1; |
159 | } | 159 | } |
160 | if (total > INT_MAX) { | ||
161 | /* Too much output data! */ | ||
162 | *outl = 0; | ||
163 | return; | ||
164 | } | ||
165 | if (inl != 0) | 160 | if (inl != 0) |
166 | memcpy(&(ctx->enc_data[0]), in, inl); | 161 | memcpy(&(ctx->enc_data[0]), in, inl); |
167 | ctx->num = inl; | 162 | ctx->num = inl; |
diff --git a/src/lib/libcrypto/evp/evp_enc.c b/src/lib/libcrypto/evp/evp_enc.c index 30941ed83d..a4e369f622 100644 --- a/src/lib/libcrypto/evp/evp_enc.c +++ b/src/lib/libcrypto/evp/evp_enc.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: evp_enc.c,v 1.28 2016/05/03 12:38:53 tedu Exp $ */ | 1 | /* $OpenBSD: evp_enc.c,v 1.29 2016/05/04 14:53:29 tedu Exp $ */ |
2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) | 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) |
3 | * All rights reserved. | 3 | * All rights reserved. |
4 | * | 4 | * |
@@ -334,7 +334,7 @@ EVP_EncryptUpdate(EVP_CIPHER_CTX *ctx, unsigned char *out, int *outl, | |||
334 | return 0; | 334 | return 0; |
335 | } | 335 | } |
336 | if (i != 0) { | 336 | if (i != 0) { |
337 | if (bl - i > inl) { | 337 | if (i + inl < bl) { |
338 | memcpy(&(ctx->buf[i]), in, inl); | 338 | memcpy(&(ctx->buf[i]), in, inl); |
339 | ctx->buf_len += inl; | 339 | ctx->buf_len += inl; |
340 | *outl = 0; | 340 | *outl = 0; |