Fix buffer overrun in FTWDxe driver

[sureshkumarpMSFT]

Description

There is buffer overrun in FtwGetLastWriteHeader function when the buffer examined is malformed. In this case the code is trying to access beyond FtwWorkSpaceSize in any of the code prior to (Offset >= FtwWorkSpaceSize) check

Changed the code to anticipate this early and terminate the loop earlier when offset + EFI_FAULT_TOLERANT_WRITE_HEADER is greater than FtwWorkSpaceSize

• [x ] Impacts functionality?

  • Functionality - Does the change ultimately impact how firmware functions?
  • Terminates the loops without doing a buffer overrun

• [ x] Impacts security?

  • Security - Does the change have a direct security impact on an application, flow, or firmware?
  • Terminates the loops without doing a buffer overrun

• [ ] Breaking change?

  • Breaking change - Will anyone consuming this change experience a break in build or boot behavior?
  • Examples: Add a new library class, move a module to a different repo, call a function in a new library class in a pre-existing module, ...

• [ ] Includes tests?

  • Tests - Does the change include any explicit test code?
  • Examples: Unit tests, integration tests, robot tests, ...

• [ ] Includes documentation?

  • Documentation - Does the change contain explicit documentation additions outside direct code modifications (and comments)?
  • Examples: Update readme file, add feature readme file, link to documentation on an a separate Web page, ...

How This Was Tested

Tested this in platform which has a SPI chip with improper firmware blocks. Without this fix boot crashed and with this change system continued to boot.

Integration Instructions

N/A

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 1.35%. Comparing base (11d1a1c) to head (f9d9cd3). Report is 4 commits behind head on release/202311.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## release/202311 #1045 +/- ## ================================================== - Coverage 1.35% 1.35% -0.01% ================================================== Files 1304 1304 Lines 333854 333857 +3 Branches 5103 5103 ================================================== Hits 4512 4512 - Misses 329259 329262 +3 Partials 83 83 ``` | [Flag](https://app.codecov.io/gh/microsoft/mu_basecore/pull/1045/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | Coverage Δ | | |---|---|---| | [MdeModulePkg](https://app.codecov.io/gh/microsoft/mu_basecore/pull/1045/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | `0.68% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[os-d]

Closing in favor of: https://github.com/microsoft/mu_basecore/pull/1131.