Update the NSS bug number for the invalid read when AES-CBC decrypting.

Update the NSS bug number for the invalid read when AES-CBC decrypting.

R=eroman@chromium.org
BUG=300681
TEST=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=226530

[eroman, 2013-10-02 02:13:17]

LGTM


On Tue, Oct 1, 2013 at 5:49 PM, <wtc@chromium.org> wrote:

> Reviewers: eroman,
>
> Description:
> Update the NSS bug number for the invalid read when AES-CBC decrypting.
>
> R=eroman@chromium.org
> BUG=300681
> TEST=none
>
> Please review this at
https://codereview.chromium.**org/25637004/<https://codereview.chromium.org/2...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src/<http://svn.chromium.org/chrome/trunk/src/>
>
> Affected files (+1, -1 lines):
>   M     crypto/encryptor_nss.cc
>
>
> Index: crypto/encryptor_nss.cc
> ==============================**==============================**=======
> --- crypto/encryptor_nss.cc     (revision 226247)
> +++ crypto/encryptor_nss.cc     (working copy)
> @@ -101,7 +101,7 @@
>    if (ciphertext.size() % AES_BLOCK_SIZE != 0) {
>      // Decryption will fail if the input is not a multiple of the block
> size.
>      // PK11_CipherOp has a bug where it will do an invalid memory access
> before
> -    // the start of the input, so avoid calling it. (Possibly NSS bug
> 921687).
> +    // the start of the input, so avoid calling it. (NSS bug 922780).
>      plaintext->clear();
>      return false;
>    }
>
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2013-10-02 14:23:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wtc@chromium.org/25637004/1

[commit-bot: I haz the power, 2013-10-02 19:42:55]

Change committed as 226530