[ Fix bug #4705 - LDAP storage gets saved with corrupted binddn]

2001-07-16  Chris Toshok  <toshok@ximian.com>

	[ Fix bug #4705 - LDAP storage gets saved with corrupted binddn]

	* gui/component/addressbook-config.c (addressbook_dialog_apply):
	call addressbook_storage_write_sources here after we're done
	rebuilding them.

	* gui/component/addressbook-storage.c
	(addressbook_storage_clear_sources): don't write the source file
	here.
	(addressbook_storage_add_source): same.
	(addressbook_storage_remove_source): same.
	(addressbook_storage_write_sources): new function, write the
	source file out.
	(addressbook_source_copy): g_strdup the binddn so we don't end up
	free'ing it multiple times thanks to copies freeing theirs.

	* gui/component/addressbook-storage.h: add prototype for
	address_storage_write_sources.

svn path=/trunk/; revision=11146

3 files changed, 8 insertions, 5 deletions

diff --git a/addressbook/gui/component/addressbook-config.c b/addressbook/gui/component/addressbook-config.c
index d23fb1ced2..77614a89f5 100644
--- a/addressbook/gui/component/addressbook-config.c
+++ b/addressbook/gui/component/addressbook-config.c
@@ -898,6 +898,8 @@ addressbook_dialog_apply (GnomePropertyBox *property_box, gint page_num, Address
 		AddressbookSource *source = (AddressbookSource*)gtk_clist_get_row_data (GTK_CLIST (dialog->clistSources), i);
 		addressbook_storage_add_source (addressbook_source_copy (source));
 	}
+
+	addressbook_storage_write_sources();
 }
 
 static void
diff --git a/addressbook/gui/component/addressbook-storage.c b/addressbook/gui/component/addressbook-storage.c
index 1f8e01f464..531051defd 100644
--- a/addressbook/gui/component/addressbook-storage.c
+++ b/addressbook/gui/component/addressbook-storage.c
@@ -413,8 +413,6 @@ addressbook_storage_add_source (AddressbookSource *source)
 				      source->uri, source->description, FALSE);
 
 	g_free (path);
-
-	save_source_data (storage_path);
 }
 
 void
@@ -446,8 +444,6 @@ addressbook_storage_remove_source (const char *name)
 	evolution_storage_removed_folder (storage, path);
 
 	g_free (path);
-
-	save_source_data (storage_path);
 }
 
 GList *
@@ -506,7 +502,11 @@ addressbook_storage_clear_sources ()
 	g_list_foreach (sources, (GFunc)addressbook_source_foreach, NULL);
 	g_list_free (sources);
 	sources = NULL;
+}
 
+void
+addressbook_storage_write_sources ()
+{
 	save_source_data (storage_path);
 }
 
@@ -527,7 +527,7 @@ addressbook_source_copy (const AddressbookSource *source)
 		copy->ldap.rootdn = g_strdup (source->ldap.rootdn);
 		copy->ldap.scope = source->ldap.scope;
 		copy->ldap.auth = source->ldap.auth;
-		copy->ldap.binddn = source->ldap.binddn;
+		copy->ldap.binddn = g_strdup (source->ldap.binddn);
 		copy->ldap.remember_passwd = source->ldap.remember_passwd;
 	}
 	else {
diff --git a/addressbook/gui/component/addressbook-storage.h b/addressbook/gui/component/addressbook-storage.h
index fa88405e38..f2a1910e13 100644
--- a/addressbook/gui/component/addressbook-storage.h
+++ b/addressbook/gui/component/addressbook-storage.h
@@ -70,6 +70,7 @@ void addressbook_storage_setup (EvolutionShellComponent *shell_component,
 GList             *addressbook_storage_get_sources (void);
 AddressbookSource *addressbook_storage_get_source_by_uri (const char *uri);
 void               addressbook_storage_clear_sources (void);
+void               addressbook_storage_write_sources (void);
 AddressbookSource *addressbook_source_copy (const AddressbookSource *source);
 void               addressbook_source_free (AddressbookSource *source);
 void               addressbook_storage_init_source_uri (AddressbookSource *source);