Alexandre Julliard : user32: Only call the driver DestroyCursorIcon entry point for valid handles.

[Alexandre Julliard]

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

Author: Alexandre Julliard <julliard(a)winehq.org>
Date:   Mon Apr 26 14:44:00 2010 -0500

user32: Only call the driver DestroyCursorIcon entry point for valid handles.

---

 dlls/user32/cursoricon.c |    4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/dlls/user32/cursoricon.c b/dlls/user32/cursoricon.c
index 0d74276..27d3040 100644
--- a/dlls/user32/cursoricon.c
+++ b/dlls/user32/cursoricon.c
@@ -172,6 +172,7 @@ static BOOL free_icon_handle( HICON handle )
         ULONG_PTR param = obj->param;
         HeapFree( GetProcessHeap(), 0, obj );
         if (wow_handlers.free_icon_param && param) wow_handlers.free_icon_param( param );
+        USER_Driver->pDestroyCursorIcon( handle );
         return TRUE;
     }
     return FALSE;
@@ -1561,10 +1562,7 @@ BOOL WINAPI DestroyIcon( HICON hIcon )
     TRACE_(icon)("%p\n", hIcon );
 
     if (CURSORICON_DelSharedIcon( hIcon ) == -1)
-    {
-        USER_Driver->pDestroyCursorIcon( hIcon );
         free_icon_handle( hIcon );
-    }
     return TRUE;
 }