[3/8] crypt32: Remove unused variable and avoid a memory leak on error path.

Thomas Faber thomas.faber at reactos.org
Mon Oct 28 12:55:04 CDT 2013


Hi,

On 2013-10-28 09:28, Dmitry Timoshkov wrote:
> -            ret = ReadFile(file, blob->pbData, blob->cbData, &read, NULL);
> +            ret = ReadFile(file, blob->pbData, blob->cbData, NULL, NULL);

the lpNumberOfBytesRead parameter is not optional on Windows for
synchronous I/O, so could you keep it please?
I've had to add it back in elsewhere to prevent Wine functions from
crashing on Win.
Thanks!

Best,
Thomas



More information about the wine-devel mailing list