ogt_vox: fix buffer overrun when nTRN node names are longer than 64 bytes

[jpaver]

• when nTRN node names were longer than 64 bytes, on non-MSC_VER platforms, _vox_strcpy_static would overwrite the bytes after the name storage. Yikes!
• A direct fix is to implement _vox_strcpy_static so that it's the same between MSC_VER platforms and non-MSC_VER platforms, namely:
  • assert if size would exceed storage
  • never exceed the size of the storage we're writing to (in case asserts are disabled)
• we also bump up nTRN node name storage from 65 bytes to 256 bytes which does kick the can down the road but should practically prevent users from hitting this assert.
• also added CHUNK_HEADER_LEN, NAME_MAX_LEN constants and fixed a few comments.

Fixes issue https://github.com/jpaver/opengametools/issues/45

[jpaver]

Hey @mgerhardy thanks for filing that issue. I did manage to get a repro that hits the same assert you mentioned in the issue, and this commit fixes it for me. Could you verify that you're seeing no issues after this commit too? Thanks!

[mgerhardy]

Works for me, too - thanks for the fast fix