[evolution-patches] A NULL printf patch was committed



Harish,
We found a NULL printf problem again. As we agreed before, I had committed the attached patch into both CVS HEAD and gnome-2-12 branch.
   Thanks!
      Harry
Index: ChangeLog
===================================================================
RCS file: /cvs/gnome/evolution/plugins/copy-tool/ChangeLog,v
retrieving revision 1.8
diff -u -r1.8 ChangeLog
--- ChangeLog	12 May 2005 04:27:37 -0000	1.8
+++ ChangeLog	29 Nov 2005 11:11:34 -0000
@@ -1,3 +1,10 @@
+2005-11-29  Simon Zheng  <Simon Zheng Sun Com>
+
+	Fix for 322733.
+
+	* copy-tool.c: (ct_selection_get): Move printf() statement after 
+	NULL checking. 
+
 2005-05-11  Not Zed  <NotZed Ximian com>
 
 	* Makefile.am: added built_sources/cleanfiles.
Index: copy-tool.c
===================================================================
RCS file: /cvs/gnome/evolution/plugins/copy-tool/copy-tool.c,v
retrieving revision 1.1
diff -u -r1.1 copy-tool.c
--- copy-tool.c	21 Oct 2004 08:39:32 -0000	1.1
+++ copy-tool.c	29 Nov 2005 11:11:34 -0000
@@ -44,10 +44,10 @@
 static void
 ct_selection_get(GtkWidget *widget, GtkSelectionData *data, guint info, guint time_stamp, void *dummy)
 {
-	printf("get selection, address is '%s'\n", address_uri);
-
 	if (address_uri == NULL)
 		return;
+
+	printf("get selection, address is '%s'\n", address_uri);
 
 	if (strncmp (address_uri, "mailto:";, 7) == 0) {
 		CamelInternetAddress *cia = camel_internet_address_new();


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