Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
FdSizeReport: Handle empty FV sections in Build Report (#664)
## Description Updates the FdSizeReportGenerator to handle the scenario where the build report contains an empty FD region. This scenario was not previously seen but has been verified and has been noted to happen when a nested FV does not contain any modules. In this scenario, the "Size: " line is not present and there is no subsection table where the included files would normally be present. Example of empty FD region: ``` >----------------------------------------------------------------------------------------------------------------------< Nested FV Type: FV Base Address: 0x0 <----------------------------------------------------------------------------------------------------------------------> ``` - [ ] Impacts functionality? - **Functionality** - Does the change ultimately impact how firmware functions? - Examples: Add a new library, publish a new PPI, update an algorithm, ... - [ ] Impacts security? - **Security** - Does the change have a direct security impact on an application, flow, or firmware? - Examples: Crypto algorithm change, buffer overflow fix, parameter validation improvement, ... - [ ] 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 Reproduced error on partner platform and verified changes resolved the issue. ## Integration Instructions N/A
- Loading branch information