Skip to content

Commit

Permalink
Fix #1929, Remove redundant status check in CFE_ES_RegisterCDSEx()
Browse files Browse the repository at this point in the history
  • Loading branch information
thnkslprpt committed May 6, 2023
1 parent e3b6767 commit 5236e56
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions modules/es/fsw/src/cfe_es_cds.c
Original file line number Diff line number Diff line change
Expand Up @@ -427,14 +427,7 @@ int32 CFE_ES_RegisterCDSEx(CFE_ES_CDSHandle_t *HandlePtr, size_t UserBlockSize,
CFE_ES_WriteToSysLog("%s: Failed to update CDS Registry (Stat=0x%08X)\n", __func__,
(unsigned int)RegUpdateStatus);

/*
* Return failure only if this was the primary error,
* do not overwrite a preexisting error.
*/
if (Status == CFE_SUCCESS)
{
Status = RegUpdateStatus;
}
Status = RegUpdateStatus;
}

if (Status == CFE_SUCCESS && !IsNewOffset)
Expand Down

0 comments on commit 5236e56

Please sign in to comment.