msi: The input buffer length is the number of characters excluding the NULL terminator, so fix an off by one error [RESEND]

[Alexandre Julliard]

"James Hawkins" <truiken at gmail.com> writes:

> Is there anything wrong with this patch?

Yes, the length does include the terminating null. That's what the
rest of the function expects, and AFAICT it's what the callers expect
too (and it's the standard behavior of API functions that take a
buffer length).

-- 
Alexandre Julliard
julliard at winehq.org