changeset 30605:40a8aae4b276

jabber: Document the other cyrus hack (ignoring EXTERNAL)
author Paul Aurich <paul@darkrain42.org>
date Sat, 12 Jun 2010 23:01:44 +0000
parents e9d540664ea3
children c3b77c0fa79c
files libpurple/protocols/jabber/auth_cyrus.c
diffstat 1 files changed, 6 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/libpurple/protocols/jabber/auth_cyrus.c	Sat Jun 12 20:18:31 2010 +0000
+++ b/libpurple/protocols/jabber/auth_cyrus.c	Sat Jun 12 23:01:44 2010 +0000
@@ -408,6 +408,12 @@
 	{
 		char *mech_name = xmlnode_get_data(mechnode);
 
+		/* Ignore blank mechanisms and EXTERNAL.  External isn't
+		 * supported, and Cyrus SASL's mechanism returns
+		 * SASL_NOMECH when the caller (us) doesn't configure it.
+		 * Except SASL_NOMECH is supposed to mean "no concordant
+		 * mechanisms"...  Easiest just to blacklist it (for now).
+		 */
 		if (!mech_name || !*mech_name ||
 				g_str_equal(mech_name, "EXTERNAL")) {
 			g_free(mech_name);