Closed Bug 3565 Opened 26 years ago Closed 26 years ago

MLK: GTK event handler memory leak

Categories

(Core :: DOM: UI Events & Focus Handling, defect, P3)

Sun
Solaris
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: bruce, Assigned: pavlov)

Details

MLK: 1312 bytes leaked in 82 blocks * This memory was allocated from: malloc [rtlib.o] __bUiLtIn_nEw [libgcc.a] __builtin_new [rtlib.o] InitDrawEvent(_GdkRectangle*,void*,nsPaintEvent&,unsigned int) [nsGtkEventHandler.cpp:231] handle_draw_event(_GtkWidget*,_GdkRectangle*,void*) [nsGtkEventHandler.cpp:424] gtk_marshal_NONE__POINTER [gtkmarshal.c:216] gtk_handlers_run [gtksignal.c:1909] gtk_signal_real_emit [gtksignal.c:1469] gtk_signal_emit [gtksignal.c:552] gtk_widget_draw [gtkwidget.c:2310] gtk_layout_draw [gtklayout.c:636] gtk_marshal_NONE__POINTER [gtkmarshal.c:216] gtk_signal_real_emit [gtksignal.c:1432] gtk_signal_emit [gtksignal.c:552] gtk_widget_draw [gtkwidget.c:2310] gtk_layout_draw [gtklayout.c:636] gtk_marshal_NONE__POINTER [gtkmarshal.c:216] gtk_signal_real_emit [gtksignal.c:1432] gtk_signal_emit [gtksignal.c:552] gtk_widget_draw [gtkwidget.c:2310] gtk_layout_draw [gtklayout.c:636] gtk_marshal_NONE__POINTER [gtkmarshal.c:216] gtk_signal_real_emit [gtksignal.c:1432] gtk_signal_emit [gtksignal.c:552] gtk_widget_draw [gtkwidget.c:2310] gtk_layout_draw [gtklayout.c:636] gtk_marshal_NONE__POINTER [gtkmarshal.c:216] gtk_signal_real_emit [gtksignal.c:1432] gtk_signal_emit [gtksignal.c:552] gtk_widget_draw [gtkwidget.c:2310] * Block of 16 bytes (82 times); last block at 0x763c18
GTK 1.2.0 .. Solaris 2.6, gcc 2.7.2.3, purify.
it looks to me like we'll need to create an Uninit procedure for each of the Init procedures in the event handling code and then go through and properly uninit the events. fun fun fun.
QA Contact: 4015 → 3847
QA contact re-assigned according to the product areas we're currently working on.
Assignee: ramiro → pavlov
ramiro, i'll fix this one.
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
patch commited. marking fix.
Status: RESOLVED → VERIFIED
Looks good.
sorry for bugzilla spam. trying to get these off my radar
Status: RESOLVED → VERIFIED
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.