summaryrefslogtreecommitdiffstats
path: root/lib/digsig.c
diff options
context:
space:
mode:
authorDmitry Kasatkin <dmitry.kasatkin@intel.com>2012-09-12 13:26:55 +0300
committerLinus Torvalds <torvalds@linux-foundation.org>2012-09-13 09:13:02 +0800
commitbc01637a80f5b670bd70a0279d3f93fa8de1c96d (patch)
tree55913071b502e2adc7a2dd4daf3b5b92c5e06b65 /lib/digsig.c
parent8507876aaada8a2cf68ebccdf1d056465dd1fc11 (diff)
downloadop-kernel-dev-bc01637a80f5b670bd70a0279d3f93fa8de1c96d.zip
op-kernel-dev-bc01637a80f5b670bd70a0279d3f93fa8de1c96d.tar.gz
digsig: add hash size comparision on signature verification
When pkcs_1_v1_5_decode_emsa() returns without error and hash sizes do not match, hash comparision is not done and digsig_verify_rsa() returns no error. This is a bug and this patch fixes it. The bug was introduced in v3.3 by commit b35e286a640f ("lib/digsig: pkcs_1_v1_5_decode_emsa cleanup"). Cc: stable@vger.kernel.org Signed-off-by: Dmitry Kasatkin <dmitry.kasatkin@intel.com> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'lib/digsig.c')
-rw-r--r--lib/digsig.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/lib/digsig.c b/lib/digsig.c
index 286d558..8c0e629 100644
--- a/lib/digsig.c
+++ b/lib/digsig.c
@@ -163,9 +163,11 @@ static int digsig_verify_rsa(struct key *key,
memcpy(out1 + head, p, l);
err = pkcs_1_v1_5_decode_emsa(out1, len, mblen, out2, &len);
+ if (err)
+ goto err;
- if (!err && len == hlen)
- err = memcmp(out2, h, hlen);
+ if (len != hlen || memcmp(out2, h, hlen))
+ err = -EINVAL;
err:
mpi_free(in);
OpenPOWER on IntegriCloud