summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDr. Stephen Henson <steve@openssl.org>2016-08-19 23:28:29 +0100
committerMatt Caswell <matt@openssl.org>2016-08-24 14:12:51 +0100
commit55d83bf7c10c7b205fffa23fa7c3977491e56c07 (patch)
tree1829db8e3fe3ec8542309a5239ce66dde10b77cd
parentef28891bab7054667f2f6739f6d376c38b3ca1cc (diff)
Avoid overflow in MDC2_Update()
Thanks to Shi Lei for reporting this issue. CVE-2016-6303 Reviewed-by: Matt Caswell <matt@openssl.org>
-rw-r--r--crypto/mdc2/mdc2dgst.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/mdc2/mdc2dgst.c b/crypto/mdc2/mdc2dgst.c
index 6397a47e92..37d99f48a5 100644
--- a/crypto/mdc2/mdc2dgst.c
+++ b/crypto/mdc2/mdc2dgst.c
@@ -42,7 +42,7 @@ int MDC2_Update(MDC2_CTX *c, const unsigned char *in, size_t len)
i = c->num;
if (i != 0) {
- if (i + len < MDC2_BLOCK) {
+ if (len < MDC2_BLOCK - i) {
/* partial block */
memcpy(&(c->data[i]), in, len);
c->num += (int)len;