SecurityPkg: Optimization by moving PeiServicesLocatePpi outside loop

[AshrafAliS]

Description

This update refactors the code by moving the LocatePpi function call outside of the for loop where it was previously called repeatedly. By relocating the LocatePpi invocation outside of the loop, we can eliminate locating the same PPI again and again.

• Breaking change?
  • Breaking change - Does this PR cause a break in build or boot behavior?
  • Examples: Does it add a new library class or move a module to a different repo.
• Impacts security?
  • Security - Does this PR have a direct security impact?
  • Examples: Crypto algorithm change or buffer overflow fix.
• Includes tests?
  • Tests - Does this PR include any explicit test code?
  • Examples: Unit tests or integration tests.

How This Was Tested

• Code review testing.
• Added qquivalent logic to EmulatorPkg and confirmed It doesn't cause issues.

Integration Instructions

N/A

[jyao1]

It seems good to me.

Would you please clarify that how you validate this patch ?

[AshrafAliS]

It seems good to me.

Would you please clarify that how you validate this patch ?

i did the 2 level of validation from my side.

1. replicating the same logic in the EmulatorPkg to see if there is any change in logic/flow. i verified that it it's not breaking anything.
2. after applying the change, tested in the intel platform as well to make sure FV hashes are verified correctly.