Some style fixes to the SASL patch.
authorMikael Hallendal <micke@imendio.com>
Fri, 23 Feb 2007 23:32:42 +0100
changeset 220 f6b9482ae89e
parent 219 6daf884bc7bd
child 221 e924c37e384b
Some style fixes to the SASL patch.
loudmouth/lm-connection.c
--- a/loudmouth/lm-connection.c	Fri Feb 23 16:58:30 2007 +0100
+++ b/loudmouth/lm-connection.c	Fri Feb 23 23:32:42 2007 +0100
@@ -669,11 +669,14 @@
 									 "id"));;
 
 	xmpp_version = lm_message_node_get_attribute (m->node, "version");
-	if (xmpp_version && !strcmp(xmpp_version, "1.0")) {
-		lm_verbose ("XMPP1.0 stream received: %s\n", connection->stream_id);
+	if (xmpp_version && strcmp (xmpp_version, "1.0") == 0) {
+		lm_verbose ("XMPP 1.0 stream received: %s\n",
+			    connection->stream_id);
+
 		connection->use_xmpp = TRUE;
 	} else {
-		lm_verbose ("Old Jabber stream received: %s\n", connection->stream_id);
+		lm_verbose ("Old Jabber stream received: %s\n", 
+			    connection->stream_id);
 	}
 	
 	connection->state = LM_CONNECTION_STATE_OPEN;
@@ -867,21 +870,25 @@
 			    LmMessage *message,
 			    gpointer user_data)
 {
-	LmMessageNode *bind_node;
-	LmMessage *bind_msg;
-	const gchar *ns;
-	int result;
+	LmMessageNode    *bind_node;
+	LmMessageHandler *new_handler;
+	LmMessage        *bind_msg;
+	const gchar      *ns;
+	int               result;
 
 	bind_node = lm_message_node_find_child (message->node, "bind");
 	if (bind_node) {
 		ns = lm_message_node_get_attribute (bind_node, "xmlns");
-		if (!ns || strcmp (ns, XMPP_NS_BIND))
+		if (!ns || strcmp (ns, XMPP_NS_BIND) != 0) {
 			return LM_HANDLER_RESULT_ALLOW_MORE_HANDLERS;
+		}
 
 		bind_msg = lm_message_new_with_sub_type (NULL,
-			LM_MESSAGE_TYPE_IQ, LM_MESSAGE_SUB_TYPE_SET);
+							 LM_MESSAGE_TYPE_IQ, 
+							 LM_MESSAGE_SUB_TYPE_SET);
 
-		bind_node = lm_message_node_add_child (bind_msg->node, "bind", NULL);
+		bind_node = lm_message_node_add_child (bind_msg->node, 
+						       "bind", NULL);
 		lm_message_node_set_attributes (bind_node,
 						"xmlns", XMPP_NS_BIND,
 						NULL);
@@ -889,11 +896,11 @@
 		lm_message_node_add_child (bind_node, "resource",
 					   connection->resource);
 
-		handler = lm_message_handler_new (connection_bind_reply,
-						  NULL, NULL);
+		new_handler = lm_message_handler_new (connection_bind_reply,
+						      NULL, NULL);
 		result = lm_connection_send_with_reply (connection, bind_msg, 
-							handler, NULL);
-		lm_message_handler_unref (handler);
+							new_handler, NULL);
+		lm_message_handler_unref (new_handler);
 		lm_message_unref (bind_msg);
 
 		if (result < 0) {
@@ -901,6 +908,7 @@
 			_lm_connection_do_close (connection);
 		}
 	}
+
 	return LM_HANDLER_RESULT_ALLOW_MORE_HANDLERS;
 }