#2767: Prevent stack overflow in Assert.Equivalent with a max travers…

…al depth of 50 (v2)
xunit · Sep 1, 2023 · c018f02 · c018f02
1 parent 7513dbf
commit c018f02
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 1 deletion.
diff --git a/src/xunit.assert/Asserts b/src/xunit.assert/Asserts
diff --git a/test/test.xunit.assert/Asserts/EquivalenceAssertsTests.cs b/test/test.xunit.assert/Asserts/EquivalenceAssertsTests.cs
@@ -1497,6 +1497,29 @@ public void Actual_Shallow()
 		}
 	}
 
+	public class DepthLimit
+	{
+		[Fact]
+		public void PreventArbitrarilyLargeDepthObjectTree()
+		{
+			var expected = new InfiniteRecursionClass();
+			var actual = new InfiniteRecursionClass();
+
+			var ex = Record.Exception(() => Assert.Equivalent(expected, actual));
+
+			Assert.IsType<EquivalentException>(ex);
+			Assert.Equal(
+				"Assert.Equivalent() Failure: Exceeded the maximum depth 50 with 'Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Parent'; check for infinite recursion or circular references",
+				ex.Message
+			);
+		}
+
+		class InfiniteRecursionClass
+		{
+			public InfiniteRecursionClass Parent => new();
+		}
+	}
+
 	public class Indexers
 	{
 		[Fact]