# HG changeset patch
# User Richard Laager <rlaager@wiktel.com>
# Date 1145509004 0
# Node ID 915f26ce1d079825ca5cf9437a492de4e552d945
# Parent  ee026e289bc40da16f0d349ce34ca85790858f44
[gaim-migrate @ 16068]
Suppress CID 108.

Event assign_zero: Variable "(irc)->roomlist" assigned value 0.
278  			irc->roomlist = NULL;

Event var_deref_model: Variable "(irc)->roomlist" tracked as NULL was passed to a function that dereferences it.
288  			gaim_roomlist_room_add_field(irc->roomlist, room, args[1]);

nosnilmot noted:
'name can't be both "323" and "322" at the same time, so this will never happen'

In the 323 case, this saves us a strcmp() that we know is useless, and adding the return is consistent with the 321 block above.

committer: Tailor Script <tailor@pidgin.im>

diff -r ee026e289bc4 -r 915f26ce1d07 src/protocols/irc/msgs.c
--- a/src/protocols/irc/msgs.c	Thu Apr 20 04:49:12 2006 +0000
+++ b/src/protocols/irc/msgs.c	Thu Apr 20 04:56:44 2006 +0000
@@ -276,6 +276,7 @@
 		gaim_roomlist_set_in_progress(irc->roomlist, FALSE);
 		gaim_roomlist_unref(irc->roomlist);
 		irc->roomlist = NULL;
+		return;
 	}
 
 	if (!strcmp(name, "322")) {