Re: Patch for some leaks in nautilus



On Tue, 2005-07-05 at 09:30 +0200, Kjartan Maraas wrote:
> Hi.
> 
> I've been running with the below patch for some time now with no ill
> effects. Please review.

@@ -1347,6 +1347,7 @@ emblem_dialog_clicked (GtkWidget *dialog
                                                       stripped_keyword,
                                                       stripped_keyword,
                                                       GTK_WINDOW (property_browser));
+               g_object_unref (pixbuf);
 
                nautilus_emblem_refresh_list ();

need to check if pixbuf != NULL before unrefing.

Otherwise this looks good.

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Alexander Larsson                                            Red Hat, Inc 
                   alexl redhat com    alla lysator liu se 
He's a bookish skateboarding romance novelist whom everyone believes is mad. 
She's a violent foul-mouthed journalist with a song in her heart and a spring 
in her step. They fight crime! 




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]