ws2_32: ioctlsocket should try to check if argp is valid
Jeff Zaroyko
jeffzaroyko at gmail.com
Tue Sep 9 12:21:10 CDT 2008
On Wed, Sep 10, 2008 at 2:57 AM, Juan Lang <juan.lang at gmail.com> wrote:
> Hi Jeff, I have a stylistic complaint about your tests in this patch:
>
> + if(sock == INVALID_SOCKET)
> + {
> + ok(0, "Creating the socket failed: %d, skipping test\n",
> WSAGetLastError());
> + return;
> + }
> If you intend to skip a test, shouldn't you use skip instead?
> + ret = ioctlsocket(sock, cmds[i], (u_long *)1);
> + if(ret != SOCKET_ERROR)
> + ok(0, "ioctlsocket succeeded unexpectedly\n");
> + else
> + {
> I'm not a fan of the ok(0) approach here. Why not use ok(ret ==
> SOCKET_ERROR) instead?
> --Juan
>
Good point, I'll use skip instead and for your second concern, I guess
I can use the return value of ok(ret == SOCKET_ERROR,.. in an if
expression.
I'll resend.
Thanks
-Jeff
More information about the wine-devel
mailing list