Sebastian Lackner : server: Wake up APC if we fail to allocate a handle in target process.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 41f7f19cb578c60d65dd8321d1de63aa9c09e271
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=41f7f19cb578c60d65dd8321d1de63aa9c09e271

Author: Sebastian Lackner <sebastian at fds-team.de>
Date:   Thu Nov  5 05:41:58 2015 +0100

server: Wake up APC if we fail to allocate a handle in target process.

Signed-off-by: Sebastian Lackner <sebastian at fds-team.de>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 server/thread.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/server/thread.c b/server/thread.c
index d65ea89..bad2231 100644
--- a/server/thread.c
+++ b/server/thread.c
@@ -1500,10 +1500,10 @@ DECL_HANDLER(select)
             /* Optimization: ignore APC_NONE calls, they are only used to
              * wake up a thread, but since we got here the thread woke up already.
              */
-            if (apc->call.type != APC_NONE)
+            if (apc->call.type != APC_NONE &&
+                (reply->apc_handle = alloc_handle( current->process, apc, SYNCHRONIZE, 0 )))
             {
-                if ((reply->apc_handle = alloc_handle( current->process, apc, SYNCHRONIZE, 0 )))
-                    reply->call = apc->call;
+                reply->call = apc->call;
                 release_object( apc );
                 break;
             }