commit 756ad173371b9862a78bdd69549e739fedd5636b
author David Benjamin <davidben@chromium.org> Tue Dec 15 14:44:19 2015 -0500
committer Adam Langley <agl@google.com> Tue Dec 15 19:46:39 2015 +0000
tree d80b2dc2b6a60d8ed4d8f4dee40034755901097e
parent 1634a334955ac07d4ee2c13a2efb420d4a406961

Initialize |one_index| in OAEP padding check.

This was a mistake in https://boringssl-review.googlesource.com/6611.

Change-Id: Ifb5c52cc7571b6f1ada4af9b46eab1f9b080b4f6
Reviewed-on: https://boringssl-review.googlesource.com/6730
Reviewed-by: Adam Langley <agl@google.com>

crypto/rsa/padding.c

diff --git a/crypto/rsa/padding.c b/crypto/rsa/padding.c
index 4054292..274c113 100644
--- a/crypto/rsa/padding.c
+++ b/crypto/rsa/padding.c
@@ -402,7 +402,7 @@
                                       const uint8_t *from, unsigned flen,
                                       const uint8_t *param, unsigned plen,
                                       const EVP_MD *md, const EVP_MD *mgf1md) {
-  unsigned i, dblen, mlen = -1, mdlen, bad, looking_for_one_byte, one_index;
+  unsigned i, dblen, mlen = -1, mdlen, bad, looking_for_one_byte, one_index = 0;
   const uint8_t *maskeddb, *maskedseed;
   uint8_t *db = NULL, seed[EVP_MAX_MD_SIZE], phash[EVP_MAX_MD_SIZE];