diff options
| author | Chris Loer <chris.loer@gmail.com> | 2018-02-23 16:24:29 -0800 |
|---|---|---|
| committer | Chris Loer <chris.loer@gmail.com> | 2018-02-26 13:51:08 -0800 |
| commit | 5d922cce8adc7795efd0eadfcb21b6b0b70538a8 (patch) | |
| tree | 6235871ee38efba9e805833165221b0b45750858 | |
| parent | ab8f6429f097a2f41742ed9c96b3e607d41c5f30 (diff) | |
| download | qtlocation-mapboxgl-upstream/cp-ci-fixes.tar.gz | |
[core, node] Hold on to map handle during NodeMap::request.upstream/cp-ci-fixes
Avoids a potential crash if garbage collection happens in the middle of a call to NodeMap::request from a map that's eligible for GC.
Fixes issue #11281
| -rw-r--r-- | platform/node/src/node_map.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/platform/node/src/node_map.cpp b/platform/node/src/node_map.cpp index ac14df0228..0fe69e8ac9 100644 --- a/platform/node/src/node_map.cpp +++ b/platform/node/src/node_map.cpp @@ -1154,6 +1154,10 @@ NodeMap::~NodeMap() { std::unique_ptr<mbgl::AsyncRequest> NodeMap::request(const mbgl::Resource& resource, mbgl::FileSource::Callback callback_) { Nan::HandleScope scope; + // Because this method may be called while this NodeMap is already eligible for garbage collection, + // we need to explicitly hold onto our own handle here so that GC during a v8 call doesn't destroy + // *this while we're still executing code. + handle(); v8::Local<v8::Value> argv[] = { Nan::New<v8::External>(this), |