Skip to content

Pipename refactor removed it from the send pipe info file dump #995

New issue
New issue
Closed
#1102
Closed
Pipename refactor removed it from the send pipe info file dump#995
#1102
Labels
backportbugcFE-SBwontfix
Milestone
Bootes
@skliper

Description

@skliper
skliper
opened on Nov 3, 2020

Describe the bug
#404 removed pipename from the structure that is written when the send pipe info command is sent. Need to add this information back in. Also related to #982. Likely need to reconsider what is output (memory addresses likely not helpful).

To Reproduce
Observe file output from a CFE_SB_SEND_PIPE_INFO_CC command before and after #404 merge.

Expected behavior
#404 shouldn't have changed output file.

Code snips

cFE/fsw/cfe-core/src/sb/cfe_sb_task.c

Lines 1015 to 1029 in e0d1ce8

for(i=0;i<CFE_PLATFORM_SB_MAX_PIPES;i++){
if(CFE_SB.PipeTbl[i].InUse==CFE_SB_IN_USE){
Status = OS_write (fd, &(CFE_SB.PipeTbl[i]), sizeof(CFE_SB_PipeD_t));
if(Status != sizeof(CFE_SB_PipeD_t)){
CFE_SB_FileWriteByteCntErr(Filename,sizeof(CFE_SB_PipeD_t),Status);
OS_close(fd);
return CFE_SB_FILE_IO_ERR;
}/* end if */
FileSize += Status;
EntryCount ++;
}/* end if */

System observed on:
All.

Additional context
Observed in build verification test of 6.8

Reporter Info
Jacob Hageman - NASA/GSFC

Ping @wmoleski

Metadata

Metadata

Assignees

No one assigned

    Labels

    backportbugcFE-SBwontfix

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions