[PATCH] ws2_32/tests: Fix memcmp size in test_TransmitFile().
Bruno Jesus
00cpxxx at gmail.com
Sun Jan 22 11:34:54 CST 2017
On Fri, Jan 20, 2017 at 5:54 PM, Zebediah Figura <z.figura12 at gmail.com> wrote:
> Fixes https://bugs.winehq.org/show_bug.cgi?id=42221
>
> Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
> ---
> dlls/ws2_32/tests/sock.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/dlls/ws2_32/tests/sock.c b/dlls/ws2_32/tests/sock.c
> index 5856718..3c3e443 100644
> --- a/dlls/ws2_32/tests/sock.c
> +++ b/dlls/ws2_32/tests/sock.c
> @@ -8304,9 +8304,9 @@ static void test_TransmitFile(void)
> iret = recv(dest, buf, sizeof(buf), 0);
> ok(iret == sizeof(header_msg)+sizeof(footer_msg)+2,
> "Returned an unexpected buffer from TransmitFile: %d\n", iret );
> - ok(memcmp(&buf[0], &header_msg[0], sizeof(header_msg)+1) == 0,
> + ok(memcmp(&buf[0], &header_msg[0], sizeof(header_msg)) == 0,
> "TransmitFile header buffer did not match!\n");
> - ok(memcmp(&buf[sizeof(header_msg)+1], &footer_msg[0], sizeof(footer_msg)+1) == 0,
> + ok(memcmp(&buf[sizeof(header_msg)+1], &footer_msg[0], sizeof(footer_msg)) == 0,
> "TransmitFile footer buffer did not match!\n");
Hi, you also have to change the sending lengths because it is also
using +1. I believe this was a strlen/sizeof misfortune, right?
Best wishes,
Bruno
More information about the wine-devel
mailing list