aboutsummaryrefslogtreecommitdiffstats
path: root/addressbook/backend/pas
diff options
context:
space:
mode:
authorChris Toshok <toshok@ximian.com>2001-07-18 10:11:23 +0800
committerChris Toshok <toshok@src.gnome.org>2001-07-18 10:11:23 +0800
commita72b3b2245ff0aaf6bd35cd229357e5aec4378ad (patch)
treedde55b7ff00bac4ed73353e31694e19a40dd0fae /addressbook/backend/pas
parent0e03733290dff59622b64fc332d75f9c217ec2b8 (diff)
downloadgsoc2013-evolution-a72b3b2245ff0aaf6bd35cd229357e5aec4378ad.tar
gsoc2013-evolution-a72b3b2245ff0aaf6bd35cd229357e5aec4378ad.tar.gz
gsoc2013-evolution-a72b3b2245ff0aaf6bd35cd229357e5aec4378ad.tar.bz2
gsoc2013-evolution-a72b3b2245ff0aaf6bd35cd229357e5aec4378ad.tar.lz
gsoc2013-evolution-a72b3b2245ff0aaf6bd35cd229357e5aec4378ad.tar.xz
gsoc2013-evolution-a72b3b2245ff0aaf6bd35cd229357e5aec4378ad.tar.zst
gsoc2013-evolution-a72b3b2245ff0aaf6bd35cd229357e5aec4378ad.zip
[ Fixes bugs #4611 - crash searching in the name field at Bigfoot for "\"
2001-07-17 Chris Toshok <toshok@ximian.com> [ Fixes bugs #4611 - crash searching in the name field at Bigfoot for "\" and #4554 - general ldap search crash ] * backend/pas/pas-backend-ldap.c (func_contains): the length of the big query string needs to take into account the length of the footer as well as the header - fix random memory corruption here. * backend/pas/pas-backend-ldap.c (pas_backend_ldap_build_query): list can be NULL, specifically if there's a parsing error in the sexp, so deal with it. svn path=/trunk/; revision=11192
Diffstat (limited to 'addressbook/backend/pas')
-rw-r--r--addressbook/backend/pas/pas-backend-ldap.c18
1 files changed, 12 insertions, 6 deletions
diff --git a/addressbook/backend/pas/pas-backend-ldap.c b/addressbook/backend/pas/pas-backend-ldap.c
index e1312ee5b9..056e56f28f 100644
--- a/addressbook/backend/pas/pas-backend-ldap.c
+++ b/addressbook/backend/pas/pas-backend-ldap.c
@@ -1552,7 +1552,7 @@ func_contains(struct _ESExp *f, int argc, struct _ESExpResult **argv, void *data
match_str = g_strdup_printf("=*%s%s)",
str, one_star ? "" : "*");
- query_length = strlen (header);
+ query_length = strlen (header) + strlen (footer);
for (i = 0; i < num_prop_infos; i ++) {
query_length += 1 + strlen(prop_info[i].ldap_attr) + strlen (match_str);
@@ -1727,13 +1727,19 @@ pas_backend_ldap_build_query (gchar *query)
e_sexp_result_free(sexp, r);
e_sexp_unref (sexp);
- if (list->next) {
- g_warning ("conversion to ldap query string failed");
- retval = NULL;
- g_list_foreach (list, (GFunc)g_free, NULL);
+ if (list) {
+ if (list->next) {
+ g_warning ("conversion to ldap query string failed");
+ retval = NULL;
+ g_list_foreach (list, (GFunc)g_free, NULL);
+ }
+ else {
+ retval = list->data;
+ }
}
else {
- retval = list->data;
+ g_warning ("conversion to ldap query string failed");
+ retval = NULL;
}
g_list_free (list);