Skip to content

Commit 85747c8

Browse files
iamcarbonabergs
authored andcommitted
Eliminate redundant dictionary lookups
1 parent a948166 commit 85747c8

File tree

3 files changed

+7
-9
lines changed

3 files changed

+7
-9
lines changed

Src/Fido2.AspNet/DistributedCacheMetadataService.cs

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,13 +44,11 @@ public virtual MetadataBLOBPayloadEntry GetEntry(Guid aaguid)
4444
if (!IsInitialized())
4545
throw new InvalidOperationException("MetadataService must be initialized");
4646

47-
if (_entries.ContainsKey(aaguid))
47+
if (_entries.TryGetValue(aaguid, out MetadataBLOBPayloadEntry entry))
4848
{
49-
var entry = _entries[aaguid];
50-
51-
if (_metadataStatements.ContainsKey(aaguid))
49+
if (_metadataStatements.TryGetValue(aaguid, out var statement))
5250
{
53-
entry.MetadataStatement = _metadataStatements[aaguid];
51+
entry.MetadataStatement = statement;
5452
}
5553

5654
return entry;

Src/Fido2/AttestationFormat/Tpm.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -479,9 +479,9 @@ public static (ushort size, byte[] name) NameFromTPM2BName(ReadOnlySpan<byte> ab
479479
if (Enum.IsDefined(typeof(TpmAlg), size))
480480
{
481481
var tpmalg = (TpmAlg)size;
482-
if (tpmAlgToDigestSizeMap.ContainsKey(tpmalg))
482+
if (tpmAlgToDigestSizeMap.TryGetValue(tpmalg, out ushort tplAlgDigestSize))
483483
{
484-
name = AuthDataHelper.GetSizedByteArray(ab, ref offset, tpmAlgToDigestSizeMap[tpmalg]);
484+
name = AuthDataHelper.GetSizedByteArray(ab, ref offset, tplAlgDigestSize);
485485
}
486486
else
487487
{

Src/Fido2/SimpleMetadataService.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,9 @@ public bool ConformanceTesting()
3232

3333
if (_entries.TryGetValue(aaguid, out MetadataBLOBPayloadEntry? entry))
3434
{
35-
if (_metadataStatements.ContainsKey(aaguid))
35+
if (_metadataStatements.TryGetValue(aaguid, out var metadataStatement))
3636
{
37-
entry.MetadataStatement = _metadataStatements[aaguid];
37+
entry.MetadataStatement = metadataStatement;
3838
}
3939

4040
return entry;

0 commit comments

Comments
 (0)