Non-distinct identifiers static analysis warning

[skliper]

Checklist (Please check before submitting)

• I reviewed the Contributing Guide.
• I performed a cursory search to see if the bug report is relevant, not redundant, nor in conflict with other tickets.

Describe the bug
DS_CloseFileCmd conflict here:

DS/fsw/src/ds_cmds.c

Lines 963 to 965 in 2a6c6a2

	void DS_CloseFileCmd(const CFE_SB_Buffer_t *BufPtr)
	{
	const DS_CloseFile_Payload_t *DS_CloseFileCmd;

To Reproduce
Static analysis

Expected behavior
Unique identifiers. TBH calling this variable a Cmd is misleading, it's a pointer to the payload. Suggest changing to PayloadPtr for all of these command processing routines (which would resolve this issue as well).

System observed on:
CodeSonar static analysis

Additional context
None

Reporter Info
Jacob Hageman - NASA/GSFC

[thnkslprpt]

2 more instances in the same file that differ only in case (capitalization):

DS/fsw/src/ds_cmds.c

Lines 1116 to 1118 in 2a6c6a2

	void DS_AddMIDCmd(const CFE_SB_Buffer_t *BufPtr)
	{
	const DS_AddRemoveMid_Payload_t *DS_AddMidCmd;

DS/fsw/src/ds_cmds.c

Lines 1214 to 1216 in 2a6c6a2

	void DS_RemoveMIDCmd(const CFE_SB_Buffer_t *BufPtr)
	{
	const DS_AddRemoveMid_Payload_t *DS_RemoveMidCmd;

PayloadPtr, CommandPtr or CmdPtr seem like the clearest options.

PayloadPtr is currently only used for the HK payload in other apps, whereas CmdPtr seems to be preferred for the other command payload types.