Fix #91, Redo SymbolNames malloc to remove out-of-bounds write

[thnkslprpt]

Checklist

• 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

• Fixes Out-of-bounds Write for VerboseStr #91
  • Removes while loop and uses strlen for malloc rather than i variable.

Testing performed
Just GitHub CI Actions so far.

Expected behavior changes
Logic stays the same, but the possibility of an out-of-bounds write is removed.

Contributor Info
Avi Weiss @thnkslprpt

[jphickey]

This doesn't seem right ... where did the call to fgetc() go?

I also don't see a buffer overrun in the original. It may write to the last char, but that's OK, as it ensures its got a null term afterward. Doesn't seem like the original code was broken.