New issue
Advanced search Search tips

Issue 869495 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 1
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Deserializing cloned RSA key doesn't check failure from import

Project Member Reported by eroman@chromium.org, Jul 31

Issue description

This is the same code omission fixed by https://chromium-review.googlesource.com/c/chromium/src/+/1131837, but in RsaHashedAlgorithm::DeserializeKeyForClone().
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 1

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/51addae4ce0e312928f57c49ccac99c84b4dfd1e

commit 51addae4ce0e312928f57c49ccac99c84b4dfd1e
Author: Eric Roman <eroman@chromium.org>
Date: Wed Aug 01 20:45:26 2018

webcrypto: Check for errors when deserializing RSA keys.

Bug:  869495 
Change-Id: I96b3eb4e72b3c46b94bdafcbc4daaf0d9466b35b
Reviewed-on: https://chromium-review.googlesource.com/1159175
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579927}
[modify] https://crrev.com/51addae4ce0e312928f57c49ccac99c84b4dfd1e/components/webcrypto/algorithms/rsa.cc
[modify] https://crrev.com/51addae4ce0e312928f57c49ccac99c84b4dfd1e/third_party/blink/renderer/bindings/modules/v8/serialization/v8_script_value_serializer_for_modules_test.cc

Status: Fixed (was: Assigned)

Sign in to add a comment