[ros-diffs] 01/01: [ADVAPI32] Ignore the return value of the HandlerFunctionEx as well

[Eric Kohl]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=aba6ce5b84e34ce02b14b…

commit aba6ce5b84e34ce02b14b689c66be3cc7830e9eb
Author:     Eric Kohl <eric.kohl(a)reactos.org>
AuthorDate: Thu Apr 19 06:09:57 2018 +0200
Commit:     Eric Kohl <eric.kohl(a)reactos.org>
CommitDate: Thu Apr 19 06:09:57 2018 +0200

    [ADVAPI32] Ignore the return value of the HandlerFunctionEx as well
    
    Ignore the return value of the HandlerFunctionEx because we already ignore the return
value of the HandlerFunction.
    This fixes a winetest failure.
---
 dll/win32/advapi32/service/sctrl.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/dll/win32/advapi32/service/sctrl.c b/dll/win32/advapi32/service/sctrl.c
index 40970f0811..4b53a74c0c 100644
--- a/dll/win32/advapi32/service/sctrl.c
+++ b/dll/win32/advapi32/service/sctrl.c
@@ -536,9 +536,10 @@ ScControlService(PACTIVE_SERVICE lpService,
     else if (lpService->HandlerFunctionEx)
     {
         /* FIXME: Send correct 2nd and 3rd parameters */
-        dwError = (lpService->HandlerFunctionEx)(ControlPacket->dwControl,
-                                                 0, NULL,
-                                                 lpService->HandlerContext);
+        (lpService->HandlerFunctionEx)(ControlPacket->dwControl,
+                                       0, NULL,
+                                       lpService->HandlerContext);
+        dwError = ERROR_SUCCESS;
     }
 
     TRACE("ScControlService() done (Error %lu)\n", dwError);