mshtml: Event handler may be already free in release_event_target.

a96606e6 · Jacek Caban · Alexandre Julliard · 3e422b9e · a96606e6
Commit a96606e6 authored Mar 29, 2011 by Jacek Caban Committed by Alexandre Julliard Mar 29, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

htmlevent.c dlls/mshtml/htmlevent.c +2 -1

No files found.
--- 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]);
        }
    }