Signed-off-by: Owen Rudge orudge@codeweavers.com --- dlls/wsdapi/discovery.c | 20 +++++++++++++++++--- dlls/wsdapi/soap.c | 13 +++++++++++++ dlls/wsdapi/wsdapi_internal.h | 2 ++ 3 files changed, 32 insertions(+), 3 deletions(-)
On Mon, Apr 09, 2018 at 10:43:48PM +0100, Owen Rudge wrote:
Signed-off-by: Owen Rudge orudge@codeweavers.com
dlls/wsdapi/discovery.c | 20 +++++++++++++++++--- dlls/wsdapi/soap.c | 13 +++++++++++++ dlls/wsdapi/wsdapi_internal.h | 2 ++ 3 files changed, 32 insertions(+), 3 deletions(-)
This is consistently giving me test failures. They are fixed after applying [4/5], so I suspect it's a patch ordering issue.
../../../tools/runtest -q -P wine -T ../../.. -M wsdapi.dll -p wsdapi_test.exe.so discovery && touch discovery.ok discovery.c:599: Test failed: Publish failed: 80070057 discovery.c:612: Test failed: No messages received discovery.c:645: Test failed: Hello message not received discovery.c:647: Test failed: AppSequence not received discovery.c:650: Test failed: Custom header not received discovery.c:651: Test failed: Wine namespace not received
This is consistently giving me test failures. They are fixed after applying [4/5], so I suspect it's a patch ordering issue.
That's odd, I'm not able to reproduce that at all. There's nothing in the third patch that should cause any difference to the output, and the changes are quite straightforward. It looks like the Publish method is returning E_INVALIDARG in your case. Could you perhaps re-run the discovery tests with WINEDEBUG=+wsdapi set and send me the log?
Thanks,
Owen