Hello Nikolay,
Am 24.03.2015 um 16:22 schrieb Nikolay Sivov:
Yes, this looks much better. Now to make it shorter please combine this tests with a fix into a single patch - this way you won't need exception handling noise.
This current testbot run does at least succeed on (my local) wine and windows for both issues, the null description and the invalid handle.
But I somehow I doubt that wine wants to handle all access violations as invalid handle? (At least OpenService crashes also on an invalid handle, so probably other functions will do too.)
Also, as you said, validation could just be done on server side, I assume that would require far more work?
So should that invalid handle part wait until a real application relies on this?
https://testbot.winehq.org/JobDetails.pl?Key=12409
Kind regards, Bernhard