405534 - AvmDebugMsg eats last output character

Status: VERIFIED FIXED

(Tamarin Graveyard :: Virtual Machine, defect)

Description

[Mark Hammond [:markh] [:mhammond]]

Attachment: include \0 in count of bytes passed to StringCbVPrintf()

The vararg version of AvmDebugMsg prints one character less than was specified - which is likely to be the trailing \n.

The problem is that StringCbVPrintf() is passed the result of _vscprintf() - but according to the docs, _vscprintf() returns the count excluding the trailing \0, while StringCbVPrintf() expects space for the \0 to be included.

Attaching a patch that fixes the problem.

Comment 1

[Mark Hammond [:markh] [:mhammond]]

thanks ed - checked in 237:0161c573f53f