Skip to content

Commit cff22de

Browse files
Merge pull request #811 from AArnott/fixNullHasher
Fix `MessagePackSecurity.GetEqualityComparer<object>` to not return null
2 parents 68712aa + 643af49 commit cff22de

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

src/MessagePack.UnityClient/Assets/Scripts/MessagePack/MessagePackSecurity.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ private MessagePackSecurity()
4444
/// </summary>
4545
/// <param name="copyFrom">The template to copy from.</param>
4646
protected MessagePackSecurity(MessagePackSecurity copyFrom)
47+
: this()
4748
{
4849
if (copyFrom is null)
4950
{

src/MessagePack.UnityClient/Assets/Scripts/Tests/ShareTests/MessagePackSecurityTests.cs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,13 @@ public void EqualityComparer_ObjectFallback()
125125
Assert.NotEqual(eq.GetHashCode(o), eq.GetHashCode(new object()));
126126
}
127127

128+
[Fact]
129+
public void EqualityComparer_ObjectFallback_AfterCopyCtor()
130+
{
131+
var security = MessagePackSecurity.UntrustedData.WithMaximumObjectGraphDepth(15);
132+
Assert.NotNull(security.GetEqualityComparer<object>());
133+
}
134+
128135
/// <summary>
129136
/// Verifies that arbitrary other types not known to be hash safe will be rejected.
130137
/// </summary>

0 commit comments

Comments
 (0)