Fix #71, table processing fixes

[jphickey]

Checklist (Please check before submitting)

• I reviewed the Contributing Guide.
• I signed and emailed the appropriate Contributor License Agreement to GSFC-SoftwareRelease@mail.nasa.gov and copied cfs-program@lists.nasa.gov.

Describe the contribution
First check that table name is null-terminated before comparing any strings, then the normal strcmp() can be safely used. This reverses the direction of the inner check loop, so it is reading entries that have been already validated otherwise, rather than reading entries that have not yet been checked at all.

Fixes #61
Fixes #71

Testing performed
Build and run CS and all tests

Expected behavior changes
Un-terminated/Long table names will be detected as part of validation. Importantly, the code no longer invokes strlen() on a string that has not been checked for null termination.

System(s) tested on
Debian

Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

[github-advanced-security]

You have successfully added a new CppCheck configuration cppcheck. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab.

[jphickey]

Also Fixes #61, duplicate issue