fix: extra child traversal in collectDirtyChildren to cause app to ha…

…ng when saving object with many pointers
parse-community · Oct 23, 2024 · 717081a · 717081a
1 parent 49b771e
commit 717081a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/Parse/Parse/Source/PFObject.m b/Parse/Parse/Source/PFObject.m
@@ -211,7 +211,7 @@ + (BFTask *)_enqueue:(BFTask *(^)(BFTask *toAwait))taskStart forObjects:(NSArray
 + (BOOL)collectDirtyChildren:(id)node
                     children:(NSMutableSet *)dirtyChildren
                        files:(NSMutableSet *)dirtyFiles
-                        seen:(NSSet *)seen
+                        seen:(NSMutableSet *)seen
                      seenNew:(NSSet *)seenNew
                  currentUser:(PFUser *)currentUser
                        error:(NSError * __autoreleasing *)error {
@@ -292,7 +292,7 @@ + (BOOL)collectDirtyChildren:(id)node
             if ([seen containsObject:object]) {
                 return YES;
             }
-            seen = [seen setByAddingObject:object];
+            [seen addObject:object];
 
             // Recurse into this object's children looking for dirty children.
             // We only need to look at the child object's current estimated data,
@@ -334,7 +334,7 @@ + (BOOL)collectDirtyChildren:(id)child
     return [self collectDirtyChildren:child
                              children:dirtyChildren
                                 files:dirtyFiles
-                                 seen:[NSSet set]
+                                 seen:[NSMutableSet set]
                               seenNew:[NSSet set]
                           currentUser:currentUser
                                 error:error];