Hi,
While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?
Full results can be found at
https://testbot.winehq.org/JobDetails.pl?Key=24000
Your paranoid android.
=== build (build) ===
Make failed
Hi,
While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?
Full results can be found at
https://testbot.winehq.org/JobDetails.pl?Key=24002
Your paranoid android.
=== build (build) ===
Make failed
Hi,
While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?
Full results can be found at
https://testbot.winehq.org/JobDetails.pl?Key=24001
Your paranoid android.
=== build (build) ===
Make failed
On Tue, Jul 5, 2016 at 10:33 AM, Matthieu Nottale
<matthieu.nottale(a)infinit.io> wrote:
> IP dual stack (v4+v6) should be disabled by default, but previous code
> was setting IPV6_V6ONLY in bind() which prevented user to override it.
> This patch moves setting IPV6_V6ONLY to socket creation time.
It still looks like there is a formatting problem with the patch,
please attach it in a new attempt or use git send-email, also see the
comments below.
> Signed-off-by: Matthieu Nottale <matthieu.nottale(a)infinit.io>
> ---
> dlls/ws2_32/socket.c | 22 ++++++++-------------
> dlls/ws2_32/tests/sock.c | 51
> ++++++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 59 insertions(+), 14 deletions(-)
>
> + enabled = 2;
> + len = sizeof(enabled);
> + ret = getsockopt(v6, IPPROTO_IPV6, IPV6_V6ONLY, &enabled, &len);
> + if (ret) {
> + ok(!ret, "Failed to check IPV6_V6ONLY ((LastError: %d).\n",
> WSAGetLastError());
> + goto end;
> + }
If the ok() is supposed to always fail use ok(0, ...). But according
to msdn this option is not present in 2003/XP, so you would end up
with an error. Better to use win_skip then to avoid this problem.
The testbot will test against many different Windows versions so we
need to ensure it works everywhere, before resending it would be
useful to do a manual run at https://testbot.winehq.org/ to check the
patch.
> + ok(enabled == 1, "IPV6_V6ONLY is not enabled by default.\n");
> + ret = setsockopt(v6, IPPROTO_IPV6, IPV6_V6ONLY, &enabled, len);
> + if (ret) {
> + ok(!ret, "Failed to disable IPV6_V6ONLY (LastError: %d).\n",
> WSAGetLastError());
> + goto end;
> + }
This should never fail as the previous test already took care of
checking for IPV6_V6ONLY support, so drop the if and goto.
> + enabled = 2;
> + len = sizeof(enabled);
> + ret = getsockopt(v6, IPPROTO_IPV6, IPV6_V6ONLY, &enabled, &len);
> + if (ret) {
> + ok(!ret, "Failed to check IPV6_V6ONLY (LastError: %d).\n",
> WSAGetLastError());
> + goto end;
> + }
Again I think getsockopt must always work at this point.
> + ok(!enabled, "IPV6_V6ONLY is enabled after bind.\n");
> + v4 = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
> + if (v4 == INVALID_SOCKET) {
> + skip("Could not create IPv4 socket (LastError: %d).\n",
> + WSAGetLastError());
> + goto end;
> + }
> + WSASetLastError(0xdeadbeef);
> + ret = bind(v4, (struct sockaddr*)&sin4, sizeof(sin4));
Please also test ok(ret,...);
Best wishes,
Bruno
On 05.07.2016 8:47, Alex Henrie wrote:
> + if (0) /* crashes on all versions of Windows */
> + {
> + r = FolderItems_get_Count(items, NULL);
> +todo_wine
> + ok(r == E_INVALIDARG, "expected E_INVALIDARG, got %08x\n", r);
> + }
Why do you want to handle this case if it crashes consistently on
Windows? And todo_wine is not going to do anything in commented block
obviously.
On 27 June 2016 at 23:54, Aaryaman Vasishta <jem456.vasishta(a)gmail.com> wrote:
> static HRESULT WINAPI d3drm_viewport1_Init(IDirect3DRMViewport *iface, IDirect3DRMDevice *device,
> IDirect3DRMFrame *camera, DWORD x, DWORD y, DWORD width, DWORD height)
> {
> - FIXME("iface %p, device %p, camera %p, x %u, y %u, width %u, height %u stub!\n",
> - iface, device, camera, x, y, width, height);
> + struct d3drm_viewport *viewport = impl_from_IDirect3DRMViewport(iface);
> + struct d3drm_frame *frame = impl_from_IDirect3DRMFrame(camera);
Using impl_from_IDirect3DRMFrame() here is unusual.
> + if (!device || !camera)
> + return D3DRMERR_BADOBJECT;
And note that this only works because of the d3drm_frame structure
layout. You'll want to use unsafe_impl_from_IDirect3DRMFrame() if
"camera" can be NULL.
On 04.07.2016 10:55, Piotr Caban wrote:
> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com>
> ---
> dlls/kernel32/kernel32.spec | 2 +-
> dlls/ntdll/ntdll.spec | 1 +
> dlls/ntdll/signal_x86_64.c | 73 +++++++++++++++++++++++++--------------------
> 3 files changed, 42 insertions(+), 34 deletions(-)
>
Your patch also changes unwinding behavior in case of MSVCRT_JUMP_BUFFER.
Previously the instruction pointer stored in the jump buffer was ignored,
now it will overwrite the value passed as target_ip. Is that intentional?
Hi,
While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?
Full results can be found at
https://testbot.winehq.org/JobDetails.pl?Key=23992
Your paranoid android.
=== build (build) ===
Patch failed to apply
On Tue, Jul 05, 2016 at 03:36:30PM +0300, Andrey Gusev wrote:
> diff --git a/dlls/krnl386.exe16/file.c b/dlls/krnl386.exe16/file.c
> index 4e7f7c4..44b1e75 100644
> --- a/dlls/krnl386.exe16/file.c
> +++ b/dlls/krnl386.exe16/file.c
> @@ -549,7 +549,7 @@ INT16 WINAPI GetPrivateProfileString16( LPCSTR section, LPCSTR entry,
> if (!entry)
> {
> /* We have to return the list of keys in the section but without the values
> - * so we need to massage the results of GetPrivateProfileSectionA.
> + * so we need to message the results of GetPrivateProfileSectionA.
> */
> UINT ret, oldlen = len, size = min( len, 1024 );
> LPSTR data, src;
No, again the original is correct.
Huw.