CS icon indicating copy to clipboard operation
CS copied to clipboard

Requirement verification failure

Open dmknutsen opened this issue 3 years ago • 2 comments

Describe the bug The following requirements fail verification testing because for each - an event message is sent, but Checksumming is not disabled:

9007.2 (User Memory) CS shall send an event message and disable Checksumming of the User-Defined Memory, if the state is not one of the following: a) enabled b) disabled c) empty

9007.1 (User Memory) If the address range for any of the User-Defined Memory is Invalid, CS shall send an event message and disable User-Defined Memory Checksumming

9006.1 (Tables) CS shall send an event message and disable Table Checksumming, if the state is not one of the following: a) enabled b) disabled c) empty

9005.1 (Apps) CS shall send an event message and disable Application code segment Checksumming, if the state is not one of the following: a) enabled b) disabled c) empty

9003.2 (Non-Volatile) CS shall send an event message and disable Non-volatile Checksumming, if the state is not one of the following: a) enabled b) disabled c) empty

9003.1 (Non-Volatile) If the address range for any of the Non-volatile segments is Invalid, CS shall send an event message and disable Non-volatile Checksumming

Expected behavior Checksumming will need to be disabled in each case or the requirements will need to be updated

Code snips Example from cs_table_processing.c: if (Result != CS_TABLE_ERROR) { CFE_EVS_SendEvent (CS_VAL_EEPROM_STATE_ERR_EID, CFE_EVS_EventType_ERROR, "Eeprom Table Validate: Illegal State Field (0x%04X) found in Entry ID %d", (unsigned short)StateField, (int)OuterLoop); Result = CS_TABLE_ERROR;
}

Reporter Info Dan Knutsen NASA Goddard

dmknutsen avatar Oct 28 '22 15:10 dmknutsen

I believe checksumming is being disabled in CS_InitAllTables()

Example: https://github.com/nasa/CS/blob/d9c10344ac279819e8841d063fea80381eb3af75/fsw/src/cs_init.c#L111-L116

havencarlson avatar Sep 13 '23 17:09 havencarlson