When this function was called multiple times (e.g. multiple messages in a pkt), it would keep appending to the current (static) string, eventually overflowing the buffer and corrupting the stack. Although reported in pktdump, it could have also happened with fmsgdump if passed multiple stored message (.msg) files.
Fixes #295 reported by Nelgin
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
From Rob Swindell@VERT to Git commit to main/sbbs/master on Sun Mar 20 21:16:19 2022