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

remove "SenderReporting" from SB private "global" #933

Closed
CDKnightNASA opened this issue Oct 1, 2020 · 1 comment · Fixed by #1099 or #1109
Closed

remove "SenderReporting" from SB private "global" #933

CDKnightNASA opened this issue Oct 1, 2020 · 1 comment · Fixed by #1099 or #1109
Milestone

Comments

@CDKnightNASA
Copy link
Contributor

The SenderReporting field exists in the SB global struct but is only set once and never used.

uint32 SenderReporting;

@CDKnightNASA CDKnightNASA added the good first issue Good for newcomers label Oct 1, 2020
@jwilmot
Copy link

jwilmot commented Oct 1, 2020

FYI, this was part of SB and SBN at least up to 2013. It was intended for last sender, local and over SBN. It was added DCR469:1 Added GetLastSender code ** Revision 1.68 2007/04/24 11:07:18EDT
Not sure who took it out or why. Too late for this now.
From cfe_sb_api.c
/* store the sender information /
if(CFE_SB.SenderReporting != 0)
{
if (SenderPtr == NULL) { / This is a normal SB send call /
BufDscPtr->Sender.ProcessorId = CFE_PSP_GetProcessorId();
strncpy(&BufDscPtr->Sender.AppName[0],CFE_SB_GetAppName(TskId,FullName),OS_MAX_API_NAME);
}
else { / Caller wants to set the Sender metadata (Likely an SBN call) */
BufDscPtr->Sender.ProcessorId = SenderPtr->ProcessorId;
strncpy(&BufDscPtr->Sender.AppName[0], &SenderPtr->AppName[0], OS_MAX_API_NAME);
}

}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment