user32: Don't try to free a handle with a value of 1, which is the dde handle…

user32: Don't try to free a handle with a value of 1, which is the dde handle value for asynchronous operations.

user32: Don't try to free a handle with a value of 1, which is the dde handle…
289047c7 · James Hawkins · Alexandre Julliard · f10d5608 · 289047c7
Commit 289047c7 authored Jan 04, 2010 by James Hawkins Committed by Alexandre Julliard Jan 05, 2010
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 0 deletions

dde_misc.c dlls/user32/dde_misc.c +5 -0

No files found.
--- a/dlls/user32/dde_misc.c
+++ b/dlls/user32/dde_misc.c
@@ -1468,6 +1468,11 @@ BOOL WINAPI DdeUnaccessData(HDDEDATA hData)
 BOOL WINAPI DdeFreeDataHandle(HDDEDATA hData)
 {
    TRACE("(%p)\n", hData);
+
+    /* 1 is the handle value returned by an asynchronous operation. */
+    if (hData == (HDDEDATA)1)
+        return TRUE;
+
    return GlobalFree(hData) == 0;
 }