April 2014 - Wine-Devel - list.winehq.org

Re: [2/2] oleaut32: Fix loading of an empty picture from a non-statable stream.
by Marvin April 10, 2014

April 10, 2014

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=6260 Your paranoid android. === w2000pro (32 bit olepicture) === olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:202: Test failed: OLP (NULL,..) does not return 0, but 0x8000ffff olepicture.c:203: Test failed: OLP (NULL,..) returns NULL, instead of !NULL olepicture.c:435: Test failed: OleLoadPicture failed with error 0x8000ffff olepicture: unhandled exception c0000005 at 0040A7BD

1 0

Re: [1/2] oleaut32/tests: Add a test for loading an empty picture from a non-statable stream.
by Marvin April 10, 2014

April 10, 2014

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=6259 Your paranoid android. === w2000pro (32 bit olepicture) === Unable to set the VM system time: the "nc -q0 '10.42.42.139' '4242'" command returned 1

1 0

Re: dpnet: Improve Error checking in Get/Set SP (try 4)
by Stefan Dösinger April 10, 2014

April 10, 2014

> + hr = IDirectPlay8Address_SetSP(localaddr, &GUID_NULL); > + ok(hr == S_OK, "got 0x%08x\n", hr); > + > + hr = IDirectPlay8Address_SetSP(localaddr, &IID_Random); > + ok(hr == S_OK, "got 0x%08x\n", hr); That’s a step forward, but I also meant that you should call GetSP after setting GUID_NULL manually. > +DEFINE_GUID(GUID_NULL,0,0,0,0,0,0,0,0,0,0,0); Why do you have to define this? (I really don’t know; It may be correct, or maybe you just need #define INITGUID, or something else.) > static HRESULT WINAPI IDirectPlay8AddressImpl_SetUserData(IDirectPlay8Address *iface, > @@ -313,6 +323,8 @@ HRESULT DPNET_CreateDirectPlay8Address(LPCLASSFACTORY iface, LPUNKNOWN punkOuter > return E_OUTOFMEMORY; > } > client->IDirectPlay8Address_iface.lpVtbl = &DirectPlay8Address_Vtbl; > + client->SP_guid = GUID_NULL; > + client->init = FALSE; > client->ref = 0; /* will be inited with QueryInterface */ > return IDirectPlay8AddressImpl_QueryInterface (&client->IDirectPlay8Address_iface, riid, ppobj); > } You shouldn’t have to set client->init = FALSE because you allocate client with HEAP_ZERO_MEMORY. If you prefer to set it for clarity that’s ok with me. The same applies to SP_guid, for two reasons: First, it’s already initialized to zeroes, and you’re not reading it unless client->init is set to TRUE, which only happens after a successful SetSP call. IMHO you can also correct the indentation in this function, but I’ll leave the decision up to you. Cheers, Stefan

2 2

Re: [PATCH] qedit/tests: Fix a file handle leak on error path (Coverity)
by Marvin April 10, 2014

April 10, 2014

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=6258 Your paranoid android. === w2000pro (32 bit mediadet) === Unable to set the VM system time: the "nc -q0 '10.42.42.139' '4242'" command returned 1

1 0

Re: dpnet: Improve Error checking in Get/Set SP (try 4)
by Marvin April 10, 2014

April 10, 2014

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=6257 Your paranoid android. === w2000pro (32 bit address) === Unable to set the VM system time: the "nc -q0 '10.42.42.139' '4242'" command returned 1

1 0

Re: [5/5] taskschd: Forward IRegisteredTask::get_Enabled to SchRpcGetTaskInfo.
by Marvin April 9, 2014

April 9, 2014

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=6250 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [4/5] taskschd: Forward IRegisteredTask::get_State to SchRpcGetTaskInfo.
by Marvin April 9, 2014

April 9, 2014

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=6249 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [3/5] schedsvc: Add SchRpcGetTaskInfo stub implementation.
by Marvin April 9, 2014

April 9, 2014

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=6248 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [2/5] schedsvc: Add some tests for SchRpcGetTaskInfo.
by Marvin April 9, 2014

April 9, 2014

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=6247 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [1/5] schedsvc: Return correct error code when a being loaded xml doesn't exist.
by Marvin April 9, 2014

April 9, 2014

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=6246 Your paranoid android. === build (build) === Patch failed to apply

1 0