Properly report channel bans in .bans

Found by: @wilkowy
Patch by: Geo
Fixes: #1297

Prior to this patch, .bans was inconsistent. With a console of '*', Eggdrop only reported channels bans of the first channel in the channel record, and .bans #channel didn't report the bans on the requested channel.

src/mod/channels.mod/userchan.c

@@ -752,7 +752,7 @@ static void display_invite(int idx, int number, maskrec *invite,
 static void tell_bans(int idx, int show_inact, char *match)
 {
   int k = 1;
-  char *chname;
+  char *chname = NULL;
   struct chanset_t *chan = NULL;
   maskrec *u;
 
@@ -788,7 +788,16 @@ static void tell_bans(int idx, int show_inact, char *match)
     } else
       display_ban(idx, k++, u, chan, show_inact);
   }
-  if (chan) {
+  for (chan = chanset; chan; chan = chan->next) {
+    /* Show a channel's bans if:
+     * - the console is set to *
+     * - the console is set to this channel
+     * - we're matching a mask
+     * - this channel was specifically requested in .bans
+     */
+    if ( (!strcmp(dcc[idx].u.chat->con_chan, "*")) ||
+         (!strcmp(dcc[idx].u.chat->con_chan, chan->dname) && (!chname || match[0])) ||
+         (chname && !strcasecmp(chname, chan->dname)) ) {
     if (show_inact)
       dprintf(idx, "%s %s:   (! = %s, * = %s)\n",
               BANS_BYCHANNEL, chan->dname, MODES_NOTACTIVE2, MODES_NOTBYBOT);
@@ -801,6 +810,9 @@ static void tell_bans(int idx, int show_inact, char *match)
             (wild_match(match, u->desc)) || (wild_match(match, u->user)))
           display_ban(idx, k, u, chan, 1);
         k++;
+      } else if (chname && !strcasecmp(chname, chan->dname)) {
+        display_ban(idx, k, u, chan, 0);
+        k++;
       } else
         display_ban(idx, k++, u, chan, show_inact);
     }
@@ -831,6 +843,7 @@ static void tell_bans(int idx, int show_inact, char *match)
         }
       }
     }
+    }
   }
   if (k == 1)
     dprintf(idx, "(There are no bans, permanent or otherwise.)\n");