[PATCH] advapi32: succeed for null pointer in ChangeServiceConfig2

[Bernhard Übelacker]

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