Skip to content

Commit

Permalink
Fix #6858: nwjs app crashes on addOriginAccessWhitelistEntry
Browse files Browse the repository at this point in the history
  • Loading branch information
rogerwang committed Nov 13, 2018
1 parent ec7b183 commit 64c8c20
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/nw_custom_bindings.cc
Original file line number Diff line number Diff line change
Expand Up @@ -341,13 +341,11 @@ void NWCustomBindings::GetOldCwd(

void NWCustomBindings::AddOriginAccessWhitelistEntry(const v8::FunctionCallbackInfo<v8::Value>& args) {
v8::Isolate* isolate = args.GetIsolate();
v8::Local<v8::Context> v8_context =
v8::Local<v8::Object>::Cast(args[0])->CreationContext();

std::string sourceOrigin = *v8::String::Utf8Value(isolate, args[0]);
std::string destinationProtocol = *v8::String::Utf8Value(isolate, args[1]);
std::string destinationHost = *v8::String::Utf8Value(isolate, args[2]);
bool allowDestinationSubdomains = args[3]->ToBoolean(v8_context).ToLocalChecked()->Value();
bool allowDestinationSubdomains = args[3].As<v8::Boolean>()->Value();

blink::WebSecurityPolicy::AddOriginAccessAllowListEntry(GURL(sourceOrigin),
blink::WebString::FromUTF8(destinationProtocol),
Expand Down

0 comments on commit 64c8c20

Please sign in to comment.