Skip to content

Commit

Permalink
Fix #104, Convert int32 return codes and variables to CFE_Status_t
Browse files Browse the repository at this point in the history
  • Loading branch information
@thnkslprpt
thnkslprpt committed Apr 22, 2023
1 parent 6418164 commit 1c93153
Show file tree
Hide file tree
Showing 9 changed files with 44 additions and 45 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/codeql-build.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,14 @@ name: CodeQl Analysis
on:
push:
pull_request:


jobs:
codeql:
name: Codeql Analysis
uses: nasa/cFS/.github/workflows/codeql-reusable.yml@main
with:
component-path: apps/ds
component-path: apps/ds
prep: 'make prep; make -C build/tools/elf2cfetbl'
make: 'make -C build/native/default_cpu1/apps/ds'
setup: |
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/format-check.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,3 @@ jobs:
format-check:
name: Run format check
uses: nasa/cFS/.github/workflows/format-check.yml@main

2 changes: 1 addition & 1 deletion .github/workflows/static-analysis.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,4 @@ jobs:
name: Run cppcheck
uses: nasa/cFS/.github/workflows/static-analysis.yml@main
with:
strict-dir-list: './fsw'
strict-dir-list: './fsw'
2 changes: 1 addition & 1 deletion docs/dox_src/cfs_ds.dox
View file
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@

This document provides a complete specification for the commands and telemetry associated
with the CFS Data Storage (DS) application software. The document is intended primarily
for users of the software (operations personal, test engineers, and maintenance personnel).
for users of the software (operations personnel, test engineers, and maintenance personnel).
The last section of the document, the deployment guide section, is intended for mission
developers when deploying and configuring the application software for a mission
flight software build environment.
Expand Down
8 changes: 4 additions & 4 deletions fsw/src/ds_app.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -173,10 +173,10 @@ void DS_AppMain(void)
/* */
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */

int32 DS_AppInitialize(void)
CFE_Status_t DS_AppInitialize(void)
{
int32 Result;
int32 i = 0;
CFE_Status_t Result;
int32 i = 0;

/*
** Initialize global data structure...
Expand Down Expand Up @@ -279,7 +279,7 @@ void DS_AppSendHkCmd(void)
{
DS_HkPacket_t HkPacket;
int32 i = 0;
int32 Status = 0;
CFE_Status_t Status = 0;
char FilterTblName[CFE_MISSION_TBL_MAX_NAME_LENGTH] = {0};
CFE_TBL_Info_t FilterTblInfo;

Expand Down
2 changes: 1 addition & 1 deletion fsw/src/ds_app.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ void DS_AppMain(void);
* \return Execution status, see \ref CFEReturnCodes
* \retval #CFE_SUCCESS \copybrief CFE_SUCCESS
*/
int32 DS_AppInitialize(void);
CFE_Status_t DS_AppInitialize(void);

/**
* \brief Application housekeeping request command handler
Expand Down
40 changes: 20 additions & 20 deletions fsw/src/ds_table.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -44,13 +44,13 @@
/* */
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */

int32 DS_TableInit(void)
CFE_Status_t DS_TableInit(void)
{
int32 Result1;
int32 Result2;
bool NeedToLoadDestTable = false;
bool NeedToLoadFilterTable = false;
uint16 TableRegisterFlags = CFE_TBL_OPT_SNGL_BUFFER | CFE_TBL_OPT_LOAD_DUMP;
CFE_Status_t Result1;
CFE_Status_t Result2;
bool NeedToLoadDestTable = false;
bool NeedToLoadFilterTable = false;
uint16 TableRegisterFlags = CFE_TBL_OPT_SNGL_BUFFER | CFE_TBL_OPT_LOAD_DUMP;

#if (DS_MAKE_TABLES_CRITICAL == 1)
TableRegisterFlags |= CFE_TBL_OPT_CRITICAL;
Expand Down Expand Up @@ -184,8 +184,8 @@ int32 DS_TableInit(void)

void DS_TableManageDestFile(void)
{
int32 i = 0;
int32 Result;
int32 i = 0;
CFE_Status_t Result;

/*
** Pointer will be NULL until first successful table load...
Expand Down Expand Up @@ -292,7 +292,7 @@ void DS_TableManageDestFile(void)

void DS_TableManageFilter(void)
{
int32 Result;
CFE_Status_t Result;

/*
** Pointer will be NULL until first successful filter table load...
Expand Down Expand Up @@ -394,10 +394,10 @@ void DS_TableManageFilter(void)
/* */
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */

int32 DS_TableVerifyDestFile(const void *TableData)
CFE_Status_t DS_TableVerifyDestFile(const void *TableData)
{
DS_DestFileTable_t *DestFileTable = (DS_DestFileTable_t *)TableData;
int32 Result = CFE_SUCCESS;
CFE_Status_t Result = CFE_SUCCESS;
int32 i = 0;

int32 CountGood = 0;
Expand Down Expand Up @@ -510,10 +510,10 @@ bool DS_TableVerifyDestFileEntry(DS_DestFileEntry_t *DestFileEntry, uint8 TableI
/* */
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */

int32 DS_TableVerifyFilter(const void *TableData)
CFE_Status_t DS_TableVerifyFilter(const void *TableData)
{
DS_FilterTable_t *FilterTable = (DS_FilterTable_t *)TableData;
int32 Result = CFE_SUCCESS;
CFE_Status_t Result = CFE_SUCCESS;
int32 i = 0;

int32 CountGood = 0;
Expand Down Expand Up @@ -869,12 +869,12 @@ void DS_TableUnsubscribe(void)
/* */
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */

int32 DS_TableCreateCDS(void)
CFE_Status_t DS_TableCreateCDS(void)
{
/* Store file sequence counts and task ena/dis state in CDS */
uint32 DataStoreBuffer[DS_DEST_FILE_CNT + 1] = {0};
int32 Result;
int32 i = 0;
uint32 DataStoreBuffer[DS_DEST_FILE_CNT + 1] = {0};
CFE_Status_t Result;
int32 i = 0;

/*
** Request for CDS area from cFE Executive Services...
Expand Down Expand Up @@ -943,9 +943,9 @@ int32 DS_TableCreateCDS(void)
void DS_TableUpdateCDS(void)
{
/* Store file sequence counts and task ena/dis state in CDS */
uint32 DataStoreBuffer[DS_DEST_FILE_CNT + 1] = {0};
int32 Result;
int32 i = 0;
uint32 DataStoreBuffer[DS_DEST_FILE_CNT + 1] = {0};
CFE_Status_t Result;
int32 i = 0;

/*
** Handle is non-zero when CDS is active...
Expand Down
6 changes: 3 additions & 3 deletions unit-test/stubs/ds_app_stubs.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -31,13 +31,13 @@
* Generated stub function for DS_AppInitialize()
* ----------------------------------------------------
*/
int32 DS_AppInitialize(void)
CFE_Status_t DS_AppInitialize(void)
{
UT_GenStub_SetupReturnBuffer(DS_AppInitialize, int32);
UT_GenStub_SetupReturnBuffer(DS_AppInitialize, CFE_Status_t);

UT_GenStub_Execute(DS_AppInitialize, Basic, NULL);

return UT_GenStub_GetReturnValue(DS_AppInitialize, int32);
return UT_GenStub_GetReturnValue(DS_AppInitialize, CFE_Status_t);
}

/*
Expand Down
24 changes: 12 additions & 12 deletions unit-test/stubs/ds_table_stubs.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -48,13 +48,13 @@ int32 DS_TableAddMsgID(CFE_SB_MsgId_t MessageID, int32 FilterIndex)
* Generated stub function for DS_TableCreateCDS()
* ----------------------------------------------------
*/
int32 DS_TableCreateCDS(void)
CFE_Status_t DS_TableCreateCDS(void)
{
UT_GenStub_SetupReturnBuffer(DS_TableCreateCDS, int32);
UT_GenStub_SetupReturnBuffer(DS_TableCreateCDS, CFE_Status_t);

UT_GenStub_Execute(DS_TableCreateCDS, Basic, NULL);

return UT_GenStub_GetReturnValue(DS_TableCreateCDS, int32);
return UT_GenStub_GetReturnValue(DS_TableCreateCDS, CFE_Status_t);
}

/*
Expand Down Expand Up @@ -122,13 +122,13 @@ uint32 DS_TableHashFunction(CFE_SB_MsgId_t MessageID)
* Generated stub function for DS_TableInit()
* ----------------------------------------------------
*/
int32 DS_TableInit(void)
CFE_Status_t DS_TableInit(void)
{
UT_GenStub_SetupReturnBuffer(DS_TableInit, int32);
UT_GenStub_SetupReturnBuffer(DS_TableInit, CFE_Status_t);

UT_GenStub_Execute(DS_TableInit, Basic, NULL);

return UT_GenStub_GetReturnValue(DS_TableInit, int32);
return UT_GenStub_GetReturnValue(DS_TableInit, CFE_Status_t);
}

/*
Expand Down Expand Up @@ -223,15 +223,15 @@ bool DS_TableVerifyCount(uint32 SequenceCount)
* Generated stub function for DS_TableVerifyDestFile()
* ----------------------------------------------------
*/
int32 DS_TableVerifyDestFile(const void *TableData)
CFE_Status_t DS_TableVerifyDestFile(const void *TableData)
{
UT_GenStub_SetupReturnBuffer(DS_TableVerifyDestFile, int32);
UT_GenStub_SetupReturnBuffer(DS_TableVerifyDestFile, CFE_Status_t);

UT_GenStub_AddParam(DS_TableVerifyDestFile, const void *, TableData);

UT_GenStub_Execute(DS_TableVerifyDestFile, Basic, NULL);

return UT_GenStub_GetReturnValue(DS_TableVerifyDestFile, int32);
return UT_GenStub_GetReturnValue(DS_TableVerifyDestFile, CFE_Status_t);
}

/*
Expand Down Expand Up @@ -273,15 +273,15 @@ bool DS_TableVerifyFileIndex(uint16 FileTableIndex)
* Generated stub function for DS_TableVerifyFilter()
* ----------------------------------------------------
*/
int32 DS_TableVerifyFilter(const void *TableData)
CFE_Status_t DS_TableVerifyFilter(const void *TableData)
{
UT_GenStub_SetupReturnBuffer(DS_TableVerifyFilter, int32);
UT_GenStub_SetupReturnBuffer(DS_TableVerifyFilter, CFE_Status_t);

UT_GenStub_AddParam(DS_TableVerifyFilter, const void *, TableData);

UT_GenStub_Execute(DS_TableVerifyFilter, Basic, NULL);

return UT_GenStub_GetReturnValue(DS_TableVerifyFilter, int32);
return UT_GenStub_GetReturnValue(DS_TableVerifyFilter, CFE_Status_t);
}

/*
Expand Down

0 comments on commit 1c93153

Please sign in to comment.