From d70245c8870426382959d9d23199a34e72fc7aa5 Mon Sep 17 00:00:00 2001
From: James Booth <boothj5@gmail.com>
Date: Sun, 22 Sep 2013 01:26:40 +0100
Subject: Fixe rejoining room after nickname conflict

fixes #241
---
 src/muc.c           |  4 +++-
 src/profanity.c     |  5 +++++
 src/xmpp/presence.c | 10 ++++++----
 3 files changed, 14 insertions(+), 5 deletions(-)

(limited to 'src')

diff --git a/src/muc.c b/src/muc.c
index 93202a54..1dcc457f 100644
--- a/src/muc.c
+++ b/src/muc.c
@@ -145,7 +145,9 @@ muc_join_room(const char * const room, const char * const nick)
 void
 muc_leave_room(const char * const room)
 {
-    g_hash_table_remove(rooms, room);
+    if (rooms != NULL) {
+        g_hash_table_remove(rooms, room);
+    }
 }
 
 /*
diff --git a/src/profanity.c b/src/profanity.c
index b18cbe91..bcb59395 100644
--- a/src/profanity.c
+++ b/src/profanity.c
@@ -196,6 +196,11 @@ prof_handle_error_message(const char *from, const char *err_msg)
         } else {
             cons_show_error("Error received from server: %s", err_msg);
         }
+        // remove the room from muc
+        Jid *room_jid = jid_create(from);
+        muc_leave_room(room_jid->barejid);
+        jid_destroy(room_jid);
+
     } else {
         cons_show_error("Error received from server: %s", err_msg);
     }
diff --git a/src/xmpp/presence.c b/src/xmpp/presence.c
index 809ddf6f..ba9c6c02 100644
--- a/src/xmpp/presence.c
+++ b/src/xmpp/presence.c
@@ -320,10 +320,12 @@ presence_leave_chat_room(const char * const room_jid)
     xmpp_conn_t *conn = connection_get_conn();
     char *nick = muc_get_room_nick(room_jid);
 
-    xmpp_stanza_t *presence = stanza_create_room_leave_presence(ctx, room_jid,
-        nick);
-    xmpp_send(conn, presence);
-    xmpp_stanza_release(presence);
+    if (nick != NULL) {
+        xmpp_stanza_t *presence = stanza_create_room_leave_presence(ctx, room_jid,
+            nick);
+        xmpp_send(conn, presence);
+        xmpp_stanza_release(presence);
+    }
 }
 
 static int
-- 
cgit 1.4.1-2-gfad0