fix: crash on objects with keys shadowing hasOwnProperty

angus-c · Jan 26, 2024 · c2c27f5 · c2c27f5
1 parent d8c5dd1
commit c2c27f5
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/packages/collection-compare/index.cjs b/packages/collection-compare/index.cjs
@@ -88,7 +88,7 @@ function compareObjects(value1, value2) {
   for (var i = 0; i < len; i++) {
     var key1 = keys1[i];
 
-    if (!(value2.hasOwnProperty(key1) && compare(value1[key1], value2[key1]))) {
+    if (!({}.hasOwnProperty.call(value2, key1) && compare(value1[key1], value2[key1]))) {
       return false;
     }
   }

diff --git a/packages/collection-compare/index.mjs b/packages/collection-compare/index.mjs
@@ -88,7 +88,7 @@ function compareObjects(value1, value2) {
   for (var i = 0; i < len; i++) {
     var key1 = keys1[i];
 
-    if (!(value2.hasOwnProperty(key1) && compare(value1[key1], value2[key1]))) {
+    if (!({}.hasOwnProperty.call(value2, key1) && compare(value1[key1], value2[key1]))) {
       return false;
     }
   }

diff --git a/test/collection-compare/index.cjs b/test/collection-compare/index.cjs
@@ -204,3 +204,14 @@ test('primitives compared with primitive wrappers return false', function(t) {
   t.notOk(compare(value2, value1));
   t.end();
 });
+
+test('objects with keys shadowing hasOwnProperty do not crash', function(t) {
+  t.plan(2);
+  var value1 = {hasOwnProperty: 1};
+  var value2 = {hasOwnProperty: 1};
+  t.ok(compare(value1, value2));
+  var value3 = {hasOwnProperty: 1};
+  var value4 = {hasOwnProperty: 2};
+  t.notOk(compare(value3, value4));
+  t.end();
+});