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.

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

Correct syslog message in UT_BSP_Unlock #1693

Closed
jphickey opened this issue Jul 20, 2021 · 0 comments · Fixed by #1694 or #1665
Closed

Correct syslog message in UT_BSP_Unlock #1693

jphickey opened this issue Jul 20, 2021 · 0 comments · Fixed by #1694 or #1665
Assignees
Labels
Milestone

Comments

@jphickey
Copy link
Contributor

Describe the bug
The log message in UT_BSP_Unlock has the wrong function name (OS_MutSemTake). Appears to be a copy/paste error from UT_BSP_Lock above.

To Reproduce
N/A (does not fail in normal operation)

Expected behavior
Log message should have OS_MutSemGive, not OS_MutSemTake

Code snips

CFE_ES_WriteToSysLog("%s(): Error from OS_MutSemTake(): %d\n", __func__, (int)rc);

System observed on:
Ubuntu

Additional context
Mismatch noted as part of review for #1599

Reporter Info
Joseph Hickey, Vantage Systems, Inc.

@jphickey jphickey added the bug label Jul 20, 2021
@jphickey jphickey self-assigned this Jul 20, 2021
jphickey added a commit to jphickey/cFE that referenced this issue Jul 20, 2021
Corrects the log message in UT_BSP_Unlock (in cfe_assert module)
to match the OSAL function called (OS_MutSemGive).
astrogeco added a commit that referenced this issue Jul 21, 2021
Fix #1693, correct function name in UT_BSP_Unlock
paulober pushed a commit to paulober/cFE that referenced this issue Aug 1, 2021
Corrects the log message in UT_BSP_Unlock (in cfe_assert module)
to match the OSAL function called (OS_MutSemGive).
@skliper skliper added this to the 7.0.0 milestone Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants