CCB: 2023.04.13

Items for Discussion

@chillfig

MM PR #69 Fix #65, Adds EventString null termination

@skliper

CF PR #379 Fix #377, Remove error event on nominal semaphore timeout

@jphickey

HS PR #90 Fix #85, move system monitoring to PSP

HS PR #91 Fix #89, add EDS file for HS

LC PR #88 Fix #82, use separate dispatcher for messages

cFE PR #2288 Fix #2287, remove return value doxygen markup

sch_lab PR #139 Fix #138, adjust table based on apps present

to_lab PR #153 Fix #152, adjust subscription table based on apps present

@thnkslprpt

CF PR #337 Fix #336, Apply consistent Event ID names to common events

CF PR #339 Fix #338, Move 'EID' to the end of Event ID names

CF PR #376 Fix #57, Remove local error return codes in CF_ValidateConfigTable()

CS PR #53 Fix #52, Apply consistent Event ID names to common events

CS PR #68 Fix #67, Combine consecutive, mutually-exclusive status checks

CS PR #70 Fix #69, Simplify CS_AttemptTableReshare

CS PR #77 Fix #76, Move function prototypes out of cs_app.c

CS PR #79 Fix #78, Convert int32 return codes and variables to CFE_Status_t

DS PR #61 Fix #60, Apply consistent Event ID names to common events

FM PR #60 Fix #59, Apply consistent Event ID names to common events

FM PR #96 Fix #95, Update command handler function message pointers and return

HK PR #27 Fix #26, Apply consistent Event ID names to common events

HS PR #51 Fix #50, Apply consistent Event ID names to common events

LC PR #46 Fix #45, Apply consistent Event ID names to common events

LC PR #47 Fix #14, Refactor LC_SampleAPs to remove extraneous if statement

LC PR #48 Fix #16, Convert LC state macros to enums

LC PR #50 Fix #49, Move variables declared mid-function to the top

LC PR #54 Fix #53, Remove stray refs to old event type constants

LC PR #68 Fix #36, Refactor LC_TableInit to remove multiple returns

LC PR #78 Fix #77, Add break to default case of switch in LC_VerifyMsgLength

LC PR #80 Fix #79, Combine consecutive, mutually-exclusive status checks

MD PR #37 Fix #36, Apply consistent Event ID names to common events

MD PR #49 Fix #48, Remove unintentionally repeated identical assert

MM PR #51 Fix #50, Apply consistent Event ID names to common events

MM PR #52 Fix #8, Remove unnecessary characters from memory dump event

MM PR #53 Fix #21, Add MM_SegmentBreak() to MM_FillMem()

MM PR #68 Fix #67, Use size_t for 'size' variables

SC PR #61 Fix #60, Apply consistent Event ID names to common events

SC PR #77 Fix #76, Squash static analysis (CodeQL) uninitialized variable warning

SC PR #79 Fix #78, Split up multiple-variable declaration statements

SC PR #81 Fix #5, Use OS_stat instead of OS_OpenCreate to verify file existence

cFE PR #2201 Fix #1458, Factor out construction of FullName to reduce code duplication

cFE PR #2211 Fix #2210, Move variables declared mid-function to the top

cFE PR #2212 Fix #1428, Refactor error handling for EVS_GetApplicationInfo()

cFE PR #2221 Fix #2003, Update event type constants in Dev Guide

cFE PR #2235 Fix #1985, Check return value of CFE_ES_PutPoolBuf

cFE PR #2241 Fix #1117, Rename unclear FileWrite EIDs

cFE PR #2246 Fix #1497, Use CFE_TIME_SysTime_t instead of indiv. sec/subsec members

cFE PR #2247 Fix #1332, Resolve compiler warnings re. signedness comparisons

cFE PR #2248 Fix #1449, Factor out common code for bitmask check

cFE PR #2250 Fix #2249, Align mismatched function prototype/implem. parameter name

cFE PR #2261 Fix #2260, Convert CFE_TBL_INFO_TABLE_LOCKED into a negative error code

cFE PR #2264 Fix #921, Update remaining cFE source/tests to use CFE_Status_t

cFE PR #2265 Fix #1093, Align Pipe ID/Name reporting in SB event strings

cFE PR #2266 Fix #1303, Add struct for Version Info in ES HK Tlm Packet

cFE PR #2267 Fix #1443, Replace for loop copy with single memcpy

cFE PR #2268 Fix #1424, Add event for NumEventFilters over max and remove FilterLimit variable

cFE PR #2269 Fix #1422, Replace UT_DisplayPkt() with UtPrintx()

cFE PR #2270 Fix #1409, Add additional context to log file write error event

cFE PR #2271 Fix #1408, Use LogFullFlag instead of checking LogCount

cFE PR #2272 Fix #1407, Add error report on EVS failure during log header write

cFE PR #2273 Fix #1466, Add status return to CFE_ES_WaitForStartupSync()

cFE PR #2274 Fix #1528, Move NO_SUCH_TABLE_ERR_EID into FindTableInRegistry and make optional

cFE PR #2276 Fix #1515, Remove unnecessary memset from CFE_TBL_LoadFromFile

cFE PR #2278 Fix #1708, Align padding explicitly in cfe_tbl_msg.h

cFE PR #2279 Fix #1723, Add NULL check for AppFileName parameter in CFE_ES_ReloadApp

cFE PR #2281 Fix #2280, Move calls to CFE_ES_GetAppName() higher to reduce code duplication

cFE PR #2283 Fix #2282, Move calls to CFE_SB_GetPipeName() up higher to reduce duplication

cFE PR #2285 Fix #1473, Add helper for CFE_SB_Write...InfoCmd() with common logic

cFE PR #2286 Fix #1467, Add syslog write for CFE_ES_WaitForSystemState() timeout

cFS-GroundSystem PR #233 Fix #218, Update LogOverflowCounter Description

cFS-GroundSystem PR #235 Fix #234, Update incorrect variable descriptions in tlm.txt

cFS-GroundSystem PR #236 Fix #220, Add missing description fields to cfe-es-hk-tlm.txt

ci_lab PR #130 Fix #129, Apply consistent Event ID names to common events

ci_lab PR #131 Fix #52, Add version information to NOOP event

ci_lab PR #132 Fix #96, Check returns from CFE calls during init

elf2cfetbl PR #122 Fix #91, Redo SymbolNames malloc to remove out-of-bounds write

elf2cfetbl PR #123 Fix #85, Add checks for all return values from fseek()

elf2cfetbl PR #124 Fix #86, Check return from chmod

elf2cfetbl PR #125 Fix #87, Check return value of stat()

elf2cfetbl PR #127 Fix #126, Move variables declared mid-function to the top

elf2cfetbl PR #134 Fix #133, Update references to CFE_FS_Header_t time members

osal PR #1331 Fix #1330, Move variables declared mid-function to the top

osal PR #1354 Fix #1353, Align mismatched function prototype/implem. parameter names

osal PR #1357 Fix #1356, Remove stray remaining 'goto' in OSAL test code

osal PR #1364 Fix #1109, Remove unimplemented test stub prototype

osal PR #1368 Fix #1107, Remove unreachable branch (superfluous if condition)

osal PR #1377 Fix #958, Convert some UT_GetStubCount to UtAssert_STUB_COUNT

osal PR #1378 Fix #1103, Implement OS_ERR_INVALID_PRIORITY return code check for RTEMS

osal PR #1379 Fix #1193, Change format of UtAssert_StringBufCompare

psp PR #380 Fix #379, Align mismatched function prototype/implem. parameter names

psp PR #381 Fix #274, Cleanup and add doxygen for cfe_psp.h

psp PR #383 Fix #382, Replace exit(1) and exit(-1) with EXIT_FAILURE macro

psp PR #388 Fix #387, Replace hand-written function names with func

sample_app PR #189 Fix #188, Apply consistent Event ID names to common events

sample_app PR #195 Fix #194, Remove component-specific cFE header #includes

sample_app PR #198 Fix #197, Refactor SAMPLE_APP_Init/Process to remove multiple returns

sample_app PR #200 Fix #199, Add test for missing branch in SAMPLE_APP_Process()

sample_app PR #201 Fix #141, Zero out global data structure during initialization

sch_lab PR #134 Fix #133, Remove component-specific cFE header #includes

to_lab PR #134 Fix #133, Apply consistent Event ID names to common events

to_lab PR #146 Fix #145, Refactor mutually exclusive logic in if, else if block

to_lab PR #148 Fix #147, Add check for failure of CFE_EVS_Register() during initialization