[ros-diffs] [jimtabor] 44989: [User32] - Patch by James Hawkins : Don't try to free a handle with a value of 1, which is the dde handle value for asynchronous operations. - Will sync to wine after a review of new WOW implementation.

jimtabor at svn.reactos.org jimtabor at svn.reactos.org
Thu Jan 7 20:05:57 CET 2010


Author: jimtabor
Date: Thu Jan  7 20:05:57 2010
New Revision: 44989

URL: http://svn.reactos.org/svn/reactos?rev=44989&view=rev
Log:
[User32]
- Patch by James Hawkins : Don't try to free a handle with a value of 1, which is the dde handle value for asynchronous operations.
- Will sync to wine after a review of new WOW implementation.

Modified:
    trunk/reactos/dll/win32/user32/misc/dde.c

Modified: trunk/reactos/dll/win32/user32/misc/dde.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/user32/misc/dde.c?rev=44989&r1=44988&r2=44989&view=diff
==============================================================================
--- trunk/reactos/dll/win32/user32/misc/dde.c [iso-8859-1] (original)
+++ trunk/reactos/dll/win32/user32/misc/dde.c [iso-8859-1] Thu Jan  7 20:05:57 2010
@@ -1456,6 +1456,11 @@
 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;
 }
 




More information about the Ros-diffs mailing list