Jacek Caban : mshtml: Event handler may be already free in release_event_target.

Alexandre Julliard julliard at winehq.org
Tue Mar 29 11:43:06 CDT 2011


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

Author: Jacek Caban <jacek at codeweavers.com>
Date:   Tue Mar 29 13:36:46 2011 +0200

mshtml: Event handler may be already free in release_event_target.

---

 dlls/mshtml/htmlevent.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/dlls/mshtml/htmlevent.c b/dlls/mshtml/htmlevent.c
index d1430cd..8dbf554 100644
--- a/dlls/mshtml/htmlevent.c
+++ b/dlls/mshtml/htmlevent.c
@@ -1343,7 +1343,8 @@ void release_event_target(event_target_t *event_target)
             if(event_target->event_table[i]->handler_prop)
                 IDispatch_Release(event_target->event_table[i]->handler_prop);
             for(j=0; j < event_target->event_table[i]->handler_cnt; j++)
-                IDispatch_Release(event_target->event_table[i]->handlers[j]);
+                if(event_target->event_table[i]->handlers[j])
+                    IDispatch_Release(event_target->event_table[i]->handlers[j]);
         }
     }
 




More information about the wine-cvs mailing list