Commit Diff

Commit:
81cc5f82b50de80bff9d2d1c37fc726baea5fac5
From:
Alexander Barton <alex@barton.de>
Date:
Message:
Channel lists: Fix duplicate check and error messages - Check correct list for duplicates when adding items. - Don't generate any messages when adding duplicates or removing non-existing items (this is how ircd-seven and ircu behave). - Code cleanup: Add_Ban_Invite(), Del_Ban_Invite().
Actions:
Patch | Tree
commit - 78a3b4c7d64e87845d6babef9a4d1619f9691aba
commit + 81cc5f82b50de80bff9d2d1c37fc726baea5fac5
blob - 82e6a5aef0fb405d23b4bdc61f14174ffc39e9f3
blob + 0340d3f8cdfdecefa882bbe4d09373af7ee3e34b
--- src/ngircd/irc-mode.c
+++ src/ngircd/irc-mode.c
@@ -831,60 +831,80 @@ IRC_AWAY( CLIENT *Client, REQUEST *Req )
 } /* IRC_AWAY */
 
 
+/**
+ * Add entries to channel ban and invite lists.
+ *
+ * @param what Can be 'I' for invite or 'b' for ban list.
+ * @param Prefix The originator of the command.
+ * @param Client The sender of the command.
+ * @param Channel The channel of which the list should be modified.
+ * @param Pattern The pattern to add to the list.
+ * @return CONNECTED or DISCONNECTED.
+ */
 static bool
-Add_Ban_Invite(int what, CLIENT *Prefix, CLIENT *Client, CHANNEL *Channel, const char *Pattern)
+Add_Ban_Invite(int what, CLIENT *Prefix, CLIENT *Client, CHANNEL *Channel,
+	       const char *Pattern)
 {
 	const char *mask;
-	bool already;
-	bool ret;
 
-	assert( Client != NULL );
-	assert( Channel != NULL );
-	assert( Pattern != NULL );
+	assert(Client != NULL);
+	assert(Channel != NULL);
+	assert(Pattern != NULL);
 	assert(what == 'I' || what == 'b');
 
 	mask = Lists_MakeMask(Pattern);
 
-	already = Lists_CheckDupeMask(Channel_GetListInvites(Channel), mask);
-	if (!already) {
-		if (what == 'I')
-			ret = Channel_AddInvite(Channel, mask, false);
-		else
-			ret = Channel_AddBan(Channel, mask);
-		if (!ret)
+	if (what == 'I') {
+		if (Lists_CheckDupeMask(Channel_GetListInvites(Channel), mask))
 			return CONNECTED;
-	}
-	if (already && (Client_Type(Prefix) == CLIENT_SERVER))
-		return CONNECTED;
-
-	if (what == 'I')
+		if (!Channel_AddInvite(Channel, mask, false))
+			return CONNECTED;
 		return Send_ListChange("+I", Prefix, Client, Channel, mask);
-	return Send_ListChange("+b", Prefix, Client, Channel, mask);
+	} else {
+		if (Lists_CheckDupeMask(Channel_GetListBans(Channel), mask))
+			return CONNECTED;
+		if (!Channel_AddBan(Channel, mask))
+			return CONNECTED;
+		return Send_ListChange("+b", Prefix, Client, Channel, mask);
+	}
 }
 
 
+/**
+ * Delete entries from channel ban and invite lists.
+ *
+ * @param what Can be 'I' for invite or 'b' for ban list.
+ * @param Prefix The originator of the command.
+ * @param Client The sender of the command.
+ * @param Channel The channel of which the list should be modified.
+ * @param Pattern The pattern to add to the list.
+ * @return CONNECTED or DISCONNECTED.
+ */
 static bool
-Del_Ban_Invite(int what, CLIENT *Prefix, CLIENT *Client, CHANNEL *Channel, const char *Pattern)
+Del_Ban_Invite(int what, CLIENT *Prefix, CLIENT *Client, CHANNEL *Channel,
+	       const char *Pattern)
 {
 	const char *mask;
-	struct list_head *list;
 
-	assert( Client != NULL );
-	assert( Channel != NULL );
-	assert( Pattern != NULL );
+	assert(Client != NULL);
+	assert(Channel != NULL);
+	assert(Pattern != NULL);
 	assert(what == 'I' || what == 'b');
 
-	mask = Lists_MakeMask( Pattern );
+	mask = Lists_MakeMask(Pattern);
 
-	if (what == 'I')
-		list = Channel_GetListInvites(Channel);
-	else
-		list = Channel_GetListBans(Channel);
-
-	Lists_Del(list, mask);
-	if (what == 'I')
+	if (what == 'I') {
+		if (!Lists_CheckDupeMask(Channel_GetListInvites(Channel), mask))
+			return CONNECTED;
+		Lists_Del(Channel_GetListInvites(Channel), mask);
 		return Send_ListChange( "-I", Prefix, Client, Channel, mask );
-	return Send_ListChange( "-b", Prefix, Client, Channel, mask );
+	} else {
+		if (!Lists_CheckDupeMask(Channel_GetListBans(Channel), mask))
+			return CONNECTED;
+		Lists_Del(Channel_GetListBans(Channel), mask);
+		return Send_ListChange( "-b", Prefix, Client, Channel, mask );
+	}
+
 }