Scripts: Add Script to Analyze Crypto Usage#241
Draft
Flickdm wants to merge 2 commits into
Draft
Conversation
Apply all 6 conceded fixes from code review battle: - Fixed INF path concatenation with separator and normalization - Added MAX_LIBRARY_ENTRY_LINES guard to prevent unbounded loop - Switched provider detection to path component matching - Reworded provider exclusion comment for accuracy - Added full_path field to JSON for NULL library disambiguation - Fixed library entry concatenation with proper separators All changes validated and tested for syntax correctness.
Add --validate-families mode to report crypto family prefixes discovered from headers but not explicitly listed in CRYPTO_FAMILIES. Includes: - discover_package_roots_from_workspace() for header-based discovery - find_auto_discovered_families() to extract families from function names - output_family_validation() to format the report Make --report optional to support validation mode without a build report. Signed-off-by: Doug Flick <dougflick@microsoft.com>
mu-automation
Bot
added
language:python
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Analyze crypto function usage for an EDK2 platform from a BUILD_REPORT.TXT.
Parses the build report to find modules that consume BaseCryptLib/TlsLib,
then scans their source files to identify which specific crypto functions
are called. Produces a per-phase, per-module usage report.
For details on how to complete these options and their meaning refer to CONTRIBUTING.md.
How This Was Tested
Physical Platforms and virtual platforms
Integration Instructions
N/A