src: skip revoke_data_object if uuid is not found

Fix: nodejs#42206 PR-URL: nodejs#42212 Fixes: nodejs#42206 Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Mestery <mestery@protonmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com>
bnb · bnb · Mar 2, 2022 · Mar 2, 2022 · Mar 2, 2022 · Mar 1, 2022
commit 3d4f3ca5eb598107eab8e4f4ee775e431e8e1bff
diff --git a/doc/api/url.md b/doc/api/url.md
@@ -659,7 +659,8 @@ added: v16.7.0
 * `id` {string} A `'blob:nodedata:...` URL string returned by a prior call to
   `URL.createObjectURL()`.
 
-Removes the stored {Blob} identified by the given ID.
+Removes the stored {Blob} identified by the given ID. Attempting to revoke a
+ID that isn’t registered will silently fail.
 
 ### Class: `URLSearchParams`
 

diff --git a/src/node_blob.cc b/src/node_blob.cc
@@ -444,7 +444,9 @@ void BlobBindingData::store_data_object(
 }
 
 void BlobBindingData::revoke_data_object(const std::string& uuid) {
-  CHECK_NE(data_objects_.find(uuid), data_objects_.end());
+  if (data_objects_.find(uuid) == data_objects_.end()) {
+    return;
+  }
   data_objects_.erase(uuid);
   CHECK_EQ(data_objects_.find(uuid), data_objects_.end());
 }

diff --git a/test/parallel/test-blob-createobjecturl.js b/test/parallel/test-blob-createobjecturl.js
@@ -29,6 +29,10 @@ const assert = require('assert');
     Buffer.from(await otherBlob.arrayBuffer()).toString(),
     'hello');
   URL.revokeObjectURL(id);
+
+  // should do nothing
+  URL.revokeObjectURL(id);
+
   assert.strictEqual(resolveObjectURL(id), undefined);
 
   // Leaving a Blob registered should not cause an assert