Skip to content

v3.0.5

Compare
Choose a tag to compare
@github-actions github-actions released this 26 Jul 22:22
· 10 commits to refs/heads/main since this release
253124f

What's Changed

  • Add NULL Check to allow IpmiSendCommandInternal flow through avoinding assert as routine does not consider NULL para… @anandakrishnanl (#243)
    Change Details
      ## Description

    IpmiSendCommandInternal did not allow NULL parameter for ResponseData and ResponseDataSize Parameters.

    Expected Behavior:
    NULL can be passed as parameter for some specific IPMI commands.

    FIX APPLIED:
    Allow IpmiSendCommandInternal to allow a NULL parameter as long as ResponseDataSize is zero.

    • 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

    Tested with Oem Specific Ipmi Command. Prior to change, cpu exception occurred. After fix, system was able to continue.

    Integration Instructions

    N/A

      </blockquote>
      <hr>
    </details>
    

Full Changelog: v3.0.4...v3.0.5