Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(778)

Unified Diff: third_party/WebKit/Source/bindings/modules/v8/serialization/V8ScriptValueDeserializerForModules.cpp

Issue 2432493002: Handle RTCCertificate::fromPEM returning nullptr, with a unit test. (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/bindings/modules/v8/serialization/V8ScriptValueDeserializerForModules.cpp
diff --git a/third_party/WebKit/Source/bindings/modules/v8/serialization/V8ScriptValueDeserializerForModules.cpp b/third_party/WebKit/Source/bindings/modules/v8/serialization/V8ScriptValueDeserializerForModules.cpp
index 9cfade22feb22667501103c9988a352a83c47808..425d1efe1fbaff4c075b1cb73848f2ab38f97e83 100644
--- a/third_party/WebKit/Source/bindings/modules/v8/serialization/V8ScriptValueDeserializerForModules.cpp
+++ b/third_party/WebKit/Source/bindings/modules/v8/serialization/V8ScriptValueDeserializerForModules.cpp
@@ -34,6 +34,8 @@ ScriptWrappable* V8ScriptValueDeserializerForModules::readDOMObject(
Platform::current()->createRTCCertificateGenerator());
std::unique_ptr<WebRTCCertificate> certificate =
certificateGenerator->fromPEM(pemPrivateKey, pemCertificate);
+ if (!certificate)
+ return nullptr;
return new RTCCertificate(std::move(certificate));
}
default:

Powered by Google App Engine
This is Rietveld 408576698