Fix format specifiers and arguments in com_dotnet

[cmb69]

This is mostly about minor glitches (signedness or length confusion), but also fixes two occasions where zend_string still have be regarded as char *.

---

clang (16.0.0) was very helpful with this.

[Girgias]

Surprised this didn't cause issues before.

[cmb69]

Surprised this didn't cause issues before.

Well, I assume that is because

• com_dotnet is rarely used
• most of the issues are minor (signedness, length)
• a couple of the mismatches are used in trace() which will only output something, when attached to a debugger

I've now added a test for the only issue I consider a problem, but this also only happens if you try to access a property of a COM object, which is unknown; you fix that code, and go on with life.

[cmb69]

Surprised this didn't cause issues before.

Well, I assume that is because

• com_dotnet is rarely used
• most of the issues are minor (signedness, length)
• a couple of the mismatches are used in trace() which will only output something, when attached to a debugger

I've now added a test for the only issue I consider a problem, but this also only happens if you try to access a property of a COM object, which is unknown; you fix that code, and go on with life.

PS: forgot to mention that there couldn't be any real OOB reads, due to the layout of zend_string and the fact that its val is always zero terminated.

[Girgias]

Thanks for the added test!