Buffer overflow possibility on command processing from strcpy use and in DS_FileCreateDest w/ bad size to strncpy

[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
Pathname from cmd used in strcpy, could overflow table Pathname if not terminated:

DS/fsw/src/ds_cmds.c

Line 766 in 9d0651c

strcpy(pDest->Pathname, DS_DestPathCmd->Pathname);

If DestFile->Pathname isn't null terminated garbage could be added to the Workname which gets copied back to the file name:

DS/fsw/src/ds_file.c

Line 588 in 9d0651c

strncpy(Workname, DestFile->Pathname, WorknameLen - 1);

To Reproduce
This was actually happening in the test, although the test overflowed the buffer to get this condition:

DS/unit-test/ds_file_tests.c

Lines 1038 to 1042 in 9d0651c

	/* Set to fail the condition "if (TotalLength != (WorknameLen - 1))" */
	for (i = 0; i < WorknameLen - 1; i++)
	{
	DS_AppData.DestFileTblPtr->File[FileIndex].Pathname[i] = 'a';
	}

Expected behavior
Only copy up to the size of the Pathname.

Code snips
See above

System observed on:
CI

Additional context
None

Reporter Info
Jacob Hageman - NASA/GSFC