Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add NULL Check to allow IpmiSendCommandInternal flow through avoinding assert as routine does not consider NULL para… #243

Merged
merged 3 commits into from
Jul 26, 2024
Merged

Add NULL Check to allow IpmiSendCommandInternal flow through avoinding assert as routine does not consider NULL para… #243

merged 3 commits into from
Jul 26, 2024

Commits on Jun 28, 2024

  1. Add NULL Check as IpmiSendCommandInternal does not consider NULL para… 

    …meter for ResponseData and ResponseDataSize Parameter

Current Behavior:

IpmiSendCommandInternal does not consider NULL parameter for ResponseData and ResponseDataSize Parameter

Expected Behavior:

NULL can be passed as parameter for

IN OUT UINT8 *ResponseData,
IN OUT UINT8 *ResponseDataSize

in IpmiSendCommandInternal Routine


FIX APPLIED:

Add NULL Check as IpmiSendCommandInternal does not consider NULL parameter for ResponseData and ResponseDataSize Parameter
    @anandakrishnanl
    anandakrishnanl authored Jun 28, 2024
    Configuration menu
    Copy the full SHA
    9daadaf View commit details
    Browse the repository at this point in the history

Commits on Jul 26, 2024

  1. Merge branch 'main' into Issue#-242-IpmiSendInternalNullParameterCheck

    @apop5
    apop5 authored Jul 26, 2024
    Configuration menu
    Copy the full SHA
    63581b3 View commit details
    Browse the repository at this point in the history

  2. Fixed Uncrustify. Added requrested checks. Updated unit tests to test… 

    …s null/nonnull interface
    @apop5
    apop5 committed Jul 26, 2024
    Configuration menu
    Copy the full SHA
    80a5452 View commit details
    Browse the repository at this point in the history