Skip to content

Commit 2fb7a51

Browse files
stephentoubBrent Schmaltz
authored andcommitted
Fix erroneous double list allocation in JsonWebToken.Audiences (#2172)
1 parent f63ffa0 commit 2fb7a51

File tree

1 file changed

+9
-11
lines changed

1 file changed

+9
-11
lines changed

src/Microsoft.IdentityModel.JsonWebTokens/JsonWebToken.cs

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -604,19 +604,17 @@ public IEnumerable<string> Audiences
604604
lock (_audiencesLock)
605605
{
606606
if (_audiences == null)
607-
608-
_audiences = new List<string>();
609-
610607
if (Payload.TryGetValue(JwtRegisteredClaimNames.Aud, out JsonElement audiences))
611608
{
612-
if (audiences.ValueKind == JsonValueKind.String)
613-
_audiences = new List<string> { audiences.GetString() };
614-
615-
if (audiences.ValueKind == JsonValueKind.Array)
616-
{
617-
foreach (JsonElement jsonElement in audiences.EnumerateArray())
618-
_audiences.Add(jsonElement.ToString());
619-
}
609+
if (audiences.ValueKind == JsonValueKind.String)
610+
{
611+
_audiences = new List<string> { audiences.GetString() };
612+
}
613+
else if (audiences.ValueKind == JsonValueKind.Array)
614+
{
615+
foreach (JsonElement jsonElement in audiences.EnumerateArray())
616+
_audiences.Add(jsonElement.ToString());
617+
}
620618
}
621619
}
622620
}

0 commit comments

Comments
 (0)