[evolution-data-server] Bug #590575 - Do not crash while searching unmatched categories.



commit f5ec8e25c4784bfc4c1bad8c82eacd08d11130ac
Author: Chenthill Palanisamy <pchenthill novell com>
Date:   Thu Aug 20 21:20:52 2009 +0530

    Bug #590575 - Do not crash while searching unmatched categories.

 .../backends/groupwise/e-book-backend-groupwise.c  |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
---
diff --git a/addressbook/backends/groupwise/e-book-backend-groupwise.c b/addressbook/backends/groupwise/e-book-backend-groupwise.c
index f9dbc5f..4790740 100644
--- a/addressbook/backends/groupwise/e-book-backend-groupwise.c
+++ b/addressbook/backends/groupwise/e-book-backend-groupwise.c
@@ -1780,7 +1780,7 @@ func_exists(struct _ESExp *f, gint argc, struct _ESExpResult **argv, gpointer da
 	if (argc == 1
 	    && argv[0]->type == ESEXP_RES_STRING) {
 		gchar *propname = argv[0]->value.string;
-		gchar *str = argv[1]->value.string;
+		gchar *str = "";
 		const gchar *gw_field_name;
 
 		gw_field_name = NULL;
@@ -1791,6 +1791,7 @@ func_exists(struct _ESExp *f, gint argc, struct _ESExpResult **argv, gpointer da
 		else if (g_str_equal (propname, "file_as") || g_str_equal (propname, "nickname"))
 			 gw_field_name = "name";
 
+		/* FIXME the whole function looks useless. Why does one need str argument to the filter here ?*/
 		if (gw_field_name) {
 
 			if (g_str_equal (gw_field_name, "fullName")) {



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