]> jfr.im git - irc/quakenet/snircd-patchqueue.git/blobdiff - welcome.patch
welcome: do all logging stuff in welcome_log
[irc/quakenet/snircd-patchqueue.git] / welcome.patch
index caed87308e236fc653f33d2df3b4f670a6bd5149..83fa1b6c2b674e709e5c3a6b107781b41bae535a 100644 (file)
@@ -77,7 +77,7 @@ ircd/ircd_lexer.l
 ircd/ircd_parser.y
 add PRIV_LOCAL_WELCOME PRIV_WELCOME
 
-diff -r 3a7d0a771452 include/client.h
+diff -r 4676d2565f9b include/client.h
 --- a/include/client.h
 +++ b/include/client.h
 @@ -142,6 +142,8 @@
@@ -89,7 +89,7 @@ diff -r 3a7d0a771452 include/client.h
      PRIV_LAST_PRIV /**< number of privileges */
    };
  
-diff -r 3a7d0a771452 include/handlers.h
+diff -r 4676d2565f9b include/handlers.h
 --- a/include/handlers.h
 +++ b/include/handlers.h
 @@ -138,6 +138,7 @@
@@ -116,7 +116,7 @@ diff -r 3a7d0a771452 include/handlers.h
  extern int ms_whois(struct Client*, struct Client*, int, char*[]);
  extern int ms_xquery(struct Client*, struct Client*, int, char*[]);
  extern int ms_xreply(struct Client*, struct Client*, int, char*[]);
-diff -r 3a7d0a771452 include/ircd_features.h
+diff -r 4676d2565f9b include/ircd_features.h
 --- a/include/ircd_features.h
 +++ b/include/ircd_features.h
 @@ -101,6 +101,7 @@
@@ -135,7 +135,7 @@ diff -r 3a7d0a771452 include/ircd_features.h
    FEAT_HIS_STATS_w,
    FEAT_HIS_STATS_x,
    FEAT_HIS_STATS_y,
-diff -r 3a7d0a771452 include/msg.h
+diff -r 4676d2565f9b include/msg.h
 --- a/include/msg.h
 +++ b/include/msg.h
 @@ -196,6 +196,10 @@
@@ -149,7 +149,7 @@ diff -r 3a7d0a771452 include/msg.h
  #define MSG_WALLCHOPS           "WALLCHOPS"     /* WC */
  #define TOK_WALLCHOPS           "WC"
  #define CMD_WALLCHOPS         MSG_WALLCHOPS, TOK_WALLCHOPS
-diff -r 3a7d0a771452 include/numeric.h
+diff -r 4676d2565f9b include/numeric.h
 --- a/include/numeric.h
 +++ b/include/numeric.h
 @@ -116,6 +116,7 @@
@@ -169,10 +169,10 @@ diff -r 3a7d0a771452 include/numeric.h
  #define ERR_SILELISTFULL     511        /* Undernet extension */
  /*      ERR_NOTIFYFULL       512           aircd */
  /*    ERR_TOOMANYWATCH     512           Numeric List: Dalnet */
-diff -r 3a7d0a771452 include/welcome.h
+diff -r 4676d2565f9b include/welcome.h
 --- /dev/null
 +++ b/include/welcome.h
-@@ -0,0 +1,60 @@
+@@ -0,0 +1,63 @@
 +#ifndef INCLUDED_welcome_h
 +#define INCLUDED_welcome_h
 +/*
@@ -208,6 +208,8 @@ diff -r 3a7d0a771452 include/welcome.h
 +
 +/* Maximum number of welcome entries (per type; X global, X local) */
 +#define WELCOME_MAX_ENTRIES  10
++/* Maximum length of a welcome message */
++#define WELCOMELEN TOPICLEN
 +/* Maximum timestamp drift in seconds allowed ahead of our idea of nettime
 + * before we throw a warning to ops
 + */
@@ -216,7 +218,7 @@ diff -r 3a7d0a771452 include/welcome.h
 +/* Describes a Welcome message entry. */
 +struct Welcome {
 +  time_t             timestamp;            /**< Timestamp of the welcome */
-+  char               text[TOPICLEN + 1];   /**< Message */
++  char               text[WELCOMELEN + 1]; /**< Message */
 +  char               who[ACCOUNTLEN + 1];  /**< Who set it */
 +};
 +
@@ -224,16 +226,17 @@ diff -r 3a7d0a771452 include/welcome.h
 +#define WELCOME_LOCAL    0x01 /**< welcome is local */
 +/** Welcome action flags */
 +#define WELCOME_ANNOUNCE 0x02 /**< announce change to users */
++#define WELCOME_INSERT   0x04 /**< insert welcome message, move down all others one place */
 +
-+extern int welcome_do(struct Client *cptr, struct Client *sptr, char *name, char *text,
-+  char *who, time_t timestamp, unsigned int flags);
++extern int welcome_do(struct Client *cptr, struct Client *sptr, char *name,
++  time_t timestamp, char *who, char *text, unsigned int flags);
 +extern void welcome_announce(int name);
 +extern void welcome_burst(struct Client *cptr);
 +extern int welcome_list(struct Client *sptr, int connect);
 +extern void welcome_stats(struct Client *sptr, const struct StatDesc *sd, char *param);
 +
 +#endif /* INCLUDED_welcome_h */
-diff -r 3a7d0a771452 ircd/Makefile.in
+diff -r 4676d2565f9b ircd/Makefile.in
 --- a/ircd/Makefile.in
 +++ b/ircd/Makefile.in
 @@ -186,6 +186,7 @@
@@ -278,7 +281,7 @@ diff -r 3a7d0a771452 ircd/Makefile.in
  whocmds.o: whocmds.c ../config.h ../include/whocmds.h \
    ../include/channel.h ../include/ircd_defs.h ../include/res.h \
    ../config.h ../include/client.h ../include/dbuf.h ../include/msgq.h \
-diff -r 3a7d0a771452 ircd/client.c
+diff -r 4676d2565f9b ircd/client.c
 --- a/ircd/client.c
 +++ b/ircd/client.c
 @@ -177,6 +177,7 @@
@@ -306,7 +309,7 @@ diff -r 3a7d0a771452 ircd/client.c
  #undef P
    { 0, 0 }
  };
-diff -r 3a7d0a771452 ircd/ircd_features.c
+diff -r 4676d2565f9b ircd/ircd_features.c
 --- a/ircd/ircd_features.c
 +++ b/ircd/ircd_features.c
 @@ -366,6 +366,7 @@
@@ -325,7 +328,7 @@ diff -r 3a7d0a771452 ircd/ircd_features.c
    F_B(HIS_STATS_w, 0, 1, 0),
    F_B(HIS_STATS_x, 0, 1, 0),
    F_B(HIS_STATS_y, 0, 1, 0),
-diff -r 3a7d0a771452 ircd/ircd_lexer.l
+diff -r 4676d2565f9b ircd/ircd_lexer.l
 --- a/ircd/ircd_lexer.l
 +++ b/ircd/ircd_lexer.l
 @@ -166,6 +166,8 @@
@@ -337,7 +340,7 @@ diff -r 3a7d0a771452 ircd/ircd_lexer.l
    { NULL, 0 }
  };
  static int ntokens;
-diff -r 3a7d0a771452 ircd/ircd_parser.y
+diff -r 4676d2565f9b ircd/ircd_parser.y
 --- a/ircd/ircd_parser.y
 +++ b/ircd/ircd_parser.y
 @@ -189,6 +189,7 @@
@@ -357,10 +360,10 @@ diff -r 3a7d0a771452 ircd/ircd_parser.y
            TPRIV_PARANOID { $$ = PRIV_PARANOID; } ;
  yesorno: YES { $$ = 1; } | NO { $$ = 0; };
  
-diff -r 3a7d0a771452 ircd/m_welcome.c
+diff -r 4676d2565f9b ircd/m_welcome.c
 --- /dev/null
 +++ b/ircd/m_welcome.c
-@@ -0,0 +1,284 @@
+@@ -0,0 +1,296 @@
 +/*
 + * IRC - Internet Relay Chat, ircd/m_welcome.c
 + * Copyright (C) 1990 Jarkko Oikarinen and
@@ -575,9 +578,15 @@ diff -r 3a7d0a771452 ircd/m_welcome.c
 +     name++;
 +     flags |= WELCOME_ANNOUNCE;
 +  }
-+  
++
++  /* check for insert prefix */
++  if (*name == '+') {
++     name++;
++     flags |= WELCOME_INSERT;
++  }
++
 +  /* and do it */
-+  return welcome_do(cptr, sptr, name, text, who, timestamp, flags);
++  return welcome_do(cptr, sptr, name, timestamp, who, text, flags);
 +}
 +
 +
@@ -642,10 +651,16 @@ diff -r 3a7d0a771452 ircd/m_welcome.c
 +     flags |= WELCOME_ANNOUNCE;
 +  }
 +
++  /* check for insert prefix */
++  if (*name == '+') {
++     name++;
++     flags |= WELCOME_INSERT;
++  }
++
 +  /* and do it */
-+  return welcome_do(cptr, sptr, name, text, who, timestamp, flags);
++  return welcome_do(cptr, sptr, name, timestamp, who, text, flags);
 +}
-diff -r 3a7d0a771452 ircd/parse.c
+diff -r 4676d2565f9b ircd/parse.c
 --- a/ircd/parse.c
 +++ b/ircd/parse.c
 @@ -661,6 +661,15 @@
@@ -664,7 +679,7 @@ diff -r 3a7d0a771452 ircd/parse.c
  
    /* This command is an alias for QUIT during the unregistered part of
     * of the server.  This is because someone jumping via a broken web
-diff -r 3a7d0a771452 ircd/s_err.c
+diff -r 4676d2565f9b ircd/s_err.c
 --- a/ircd/s_err.c
 +++ b/ircd/s_err.c
 @@ -486,7 +486,7 @@
@@ -685,7 +700,7 @@ diff -r 3a7d0a771452 ircd/s_err.c
  /* 510 */
    { 0 },
  /* 511 */
-diff -r 3a7d0a771452 ircd/s_serv.c
+diff -r 4676d2565f9b ircd/s_serv.c
 --- a/ircd/s_serv.c
 +++ b/ircd/s_serv.c
 @@ -57,6 +57,7 @@
@@ -704,7 +719,7 @@ diff -r 3a7d0a771452 ircd/s_serv.c
  
    /*
     * Pass on my client information to the new server
-diff -r 3a7d0a771452 ircd/s_stats.c
+diff -r 4676d2565f9b ircd/s_stats.c
 --- a/ircd/s_stats.c
 +++ b/ircd/s_stats.c
 @@ -54,6 +54,7 @@
@@ -729,7 +744,7 @@ diff -r 3a7d0a771452 ircd/s_stats.c
    { 'x', "memusage", STAT_FLAG_OPERFEAT, FEAT_HIS_STATS_x,
      stats_meminfo, 0,
      "List usage information." },
-diff -r 3a7d0a771452 ircd/s_user.c
+diff -r 4676d2565f9b ircd/s_user.c
 --- a/ircd/s_user.c
 +++ b/ircd/s_user.c
 @@ -63,6 +63,7 @@
@@ -750,10 +765,10 @@ diff -r 3a7d0a771452 ircd/s_user.c
    }
    else {
      struct Client *acptr = user->server;
-diff -r 3a7d0a771452 ircd/welcome.c
+diff -r 4676d2565f9b ircd/welcome.c
 --- /dev/null
 +++ b/ircd/welcome.c
-@@ -0,0 +1,371 @@
+@@ -0,0 +1,570 @@
 +/*
 + * IRC - Internet Relay Chat, ircd/welcome.c
 + * Copyright (C) 1990 Jarkko Oikarinen and
@@ -821,7 +836,7 @@ diff -r 3a7d0a771452 ircd/welcome.c
 +  assert(name >= 0 && name <= 2 * WELCOME_MAX_ENTRIES - 1);
 +
 +  /* store it */
-+  ircd_strncpy(WelcomeArray[name].text, text, TOPICLEN);
++  ircd_strncpy(WelcomeArray[name].text, text, WELCOMELEN);
 +  ircd_strncpy(WelcomeArray[name].who, who, ACCOUNTLEN);
 +  WelcomeArray[name].timestamp = timestamp;
 +
@@ -829,23 +844,254 @@ diff -r 3a7d0a771452 ircd/welcome.c
 +}
 +
 +
++/** Propagate a welcome message.
++ * @param[in] cptr Local client that sent us the welcome.
++ * @param[in] sptr Originator of the welcome.
++ * @param[in] nameint Name of the message.
++ * @param[in] timestamp Timestamp of when the message was set.
++ * @param[in] who Who set this message.
++ * @param[in] text The welcome message.
++ * @param[in] flags Flags to set on welcome.
++ * @return Zero
++ */
++int
++welcome_propagate(struct Client *cptr, struct Client *sptr, int nameint,
++  time_t timestamp, char *who, char *text, unsigned int flags)
++{
++  assert(!(flags & WELCOME_LOCAL));
++
++  sendcmdto_serv_butone(sptr, CMD_WELCOME, cptr, "* %s%s%d %Tu %s :%s",
++    (flags & WELCOME_ANNOUNCE) ? "!" : "", (flags & WELCOME_INSERT) ? "+" : "",
++    nameint, timestamp, who, text);
++
++  return 0;
++}
++
++
++/** Log a welcome message.
++ * @param[in] sptr Originator of the welcome.
++ * @param[in] msg The message to show.
++ * @param[in] who Who set this message.
++ * @param[in] flags Flags to set on welcome.
++ * @return Zero
++ */
++int
++welcome_log(struct Client *sptr, char *msg, char *who, unsigned int flags)
++{
++
++  /* inform ops */
++  sendto_opmask_butone(0, SNO_OLDSNO, "%s %s",
++    (feature_bool(FEAT_HIS_SNOTICES) || IsServer(sptr)) ?
++    get_client_name_and_opername(sptr) : cli_name((cli_user(sptr))->server), msg);
++
++  /* log it */
++  log_write(LS_NETWORK, L_INFO, LOG_NOSNOTICE, "%#C (%s) %s", sptr, who, msg);
++
++  /* welcome by remote user, inform oper of success */
++  if ((flags & WELCOME_LOCAL) && IsUser(sptr) && !MyUser(sptr)) {
++    sendcmdto_one(&me, CMD_NOTICE, sptr, "%C :%s %s",
++      sptr, get_client_name_and_opername(sptr), msg);
++
++    /* TODO: wallops all local changes, by both local and remote opers? */
++    /* tell all opers about the local message being set remotely */
++    sendwallto_group_butone(&me, WALL_WALLOPS, 0, "%s %s", get_client_name_and_opername(sptr), msg);
++  }
++
++  return 0;
++}
++
++
++/** Set a welcome message.
++ * @param[in] cptr Local client that sent us the welcome.
++ * @param[in] sptr Originator of the welcome.
++ * @param[in] nameint Name of the message.
++ * @param[in] namearray Array entry.
++ * @param[in] timestamp Timestamp of when the message was set.
++ * @param[in] who Who set this message.
++ * @param[in] text The message.
++ * @param[in] flags Flags to set on welcome.
++ * @return Zero
++ */
++int
++welcome_set(struct Client *cptr, struct Client *sptr, int nameint,
++  int namearray, time_t timestamp, char *who, char *text, unsigned int flags)
++{
++  char msg[BUFSIZE];                    /* msg for snomask, logging, etc. */
++  int new = 0;
++
++  /* debug */
++  Debug((DEBUG_DEBUG, "welcome_set(\"%s\", \"%s\", %d, %d, %Tu, \"%s\", \"%s\", 0x%04x)",
++   cli_name(cptr), cli_name(sptr), nameint, namearray, timestamp, who, text, flags));
++
++  /* not set */
++  if (*WelcomeArray[namearray].text == 0)
++    new = 1;
++
++  /* update */
++  welcome_make(namearray, text, who, timestamp);
++
++  /* create msg for snomask, logging, etc. */
++  ircd_snprintf(0, msg, 0, "%s%s%s WELCOME %d \"%s\" [%Tu]",
++    new ? "setting" : "changing",
++    (flags & WELCOME_ANNOUNCE) ? " and announcing " : " ",
++    (flags & WELCOME_LOCAL) ? "local" : "global",
++    nameint, text, timestamp);
++
++  /* log it */
++  welcome_log(sptr, msg, who, flags);
++
++  /* propagate it */
++  if (!(flags & WELCOME_LOCAL))
++    welcome_propagate(cptr, sptr, nameint, timestamp, who, text, flags);
++
++  /* announce it */
++  if (flags & WELCOME_ANNOUNCE)
++    welcome_announce(namearray);
++
++  return 0;
++}
++
++
++/** Unset a welcome message.
++ * @param[in] cptr Local client that sent us the welcome.
++ * @param[in] sptr Originator of the welcome.
++ * @param[in] nameint Name of the message.
++ * @param[in] namearray Array entry.
++ * @param[in] timestamp Timestamp of when the message was set.
++ * @param[in] who Who set this message.
++ * @param[in] flags Flags to set on welcome.
++ * @return Zero
++ */
++int
++welcome_unset(struct Client *cptr, struct Client *sptr, int nameint,
++  int namearray, time_t timestamp, char *who, unsigned int flags)
++{
++  char msg[BUFSIZE];                    /* msg for snomask, logging, etc. */
++  char text[WELCOMELEN + 1];         /* save old text */ 
++
++  /* debug */
++  Debug((DEBUG_DEBUG, "welcome_unset(\"%s\", \"%s\", %d, %d, %Tu, \"%s\", 0x%04x)",
++   cli_name(cptr), cli_name(sptr), nameint, namearray, timestamp, who, flags));
++
++  /* save text */
++  ircd_strncpy(text, WelcomeArray[namearray].text, WELCOMELEN); 
++
++  /* update */
++  welcome_make(namearray, "", who, timestamp);
++
++  /* create msg for snomask, logging, etc. */
++  ircd_snprintf(0, msg, 0, "unsetting %s WELCOME %d \"%s\" [%Tu]",
++    (flags & WELCOME_LOCAL) ? "local" : "global", nameint, text, timestamp);
++
++  /* log it */
++  welcome_log(sptr, msg, who, flags);
++
++  /* propagate it, but not when inserting */
++  if (!(flags & (WELCOME_LOCAL|WELCOME_INSERT)))
++    welcome_propagate(cptr, sptr, nameint, timestamp, who, "", flags);
++
++  return 0;
++}
++
++
++/** Insert a welcome message.
++ * @param[in] cptr Local client that sent us the welcome.
++ * @param[in] sptr Originator of the welcome.
++ * @param[in] nameint Name of the message.
++ * @param[in] namearray Array entry.
++ * @param[in] timestamp Timestamp of when the message was set.
++ * @param[in] who Who set this message.
++ * @param[in] text The welcome message.
++ * @param[in] flags Flags to set on welcome.
++ * @return Zero
++ */
++int
++welcome_insert(struct Client *cptr, struct Client *sptr, int nameint,
++  int namearray, time_t timestamp, char *who, char *text, unsigned int flags)
++{
++  char msg[BUFSIZE];                    /* msg for snomask, logging, etc. */
++  int i;                                /* loop variable */
++  int empty = -1;                       /* first empty spot in array after arrayname */
++  int end = WELCOME_MAX_ENTRIES -1;     /* last element to check in array */
++  int last = end;                       /* last welcome message to feed to welcome_unset */
++
++  /* debug */
++  Debug((DEBUG_DEBUG, "welcome_insert(\"%s\", \"%s\", %d, %d, %Tu, \"%s\", \"%s\", 0x%04x)",
++   cli_name(cptr), cli_name(sptr), nameint, namearray, timestamp, who, text, flags));
++
++  /* not set yet, do not insert */
++  if (WelcomeArray[namearray].timestamp == 0)
++    return 0;
++
++  /* last global entry */
++  if (!(flags & WELCOME_LOCAL) && (nameint == WELCOME_MAX_ENTRIES))
++    return 0;
++
++  /* last local entry */
++  if ((flags & WELCOME_LOCAL) && (nameint == 2 * WELCOME_MAX_ENTRIES)) 
++    return 0;
++
++  /* correct end for local offset */
++  if (flags & WELCOME_LOCAL)
++    end += WELCOME_MAX_ENTRIES;
++
++  /* found first empty spot */
++  for (i = namearray; i <= end; i++) {
++    if (*WelcomeArray[i].text == 0) {
++      empty = i;
++      break;
++    }
++  }
++
++  /* no empty spot, need to unset last */
++  if (empty == -1) {
++    welcome_unset(cptr, sptr, end, namearray, timestamp, who, flags);
++    empty = end;
++  }
++
++  /* move entries down, update timestamp */
++  for (i = empty; i > namearray; i--)
++    welcome_make(i, WelcomeArray[i-1].text, WelcomeArray[i-1].who, timestamp);
++
++  /* correct empty for local offset */
++  if (flags & WELCOME_LOCAL)
++    empty -= WELCOME_MAX_ENTRIES;
++
++  /* create msg for snomask, logging, etc. */
++  if (nameint == empty)
++    ircd_snprintf(0, msg, 0, "moving %s WELCOME message %d one place down",
++      (flags & WELCOME_LOCAL) ? "local" : "global", nameint);  
++  else 
++    ircd_snprintf(0, msg, 0, "moving %s WELCOME message %d %s %d one place down",
++      (flags & WELCOME_LOCAL) ? "local" : "global", nameint, (empty - nameint > 1) ? "to" : "and" , empty);
++
++  /* log it */
++  welcome_log(sptr, msg, who, flags);
++
++  /* set it */
++  welcome_set(cptr, sptr, nameint, namearray, timestamp, who, text, flags);
++
++  return 0;
++}
++
++
 +/** Change a welcome message.
 + * @param[in] cptr Local client that sent us the welcome.
 + * @param[in] sptr Originator of the welcome.
 + * @param[in] name Name of the message.
-+ * @param[in] text The welcome message.
 + * @param[in] timestamp Timestamp of when the message was set.
++ * @param[in] who Who set this message.
++ * @param[in] text The welcome message.
 + * @param[in] flags Flags to set on welcome.
 + * @return Zero
 + */
 +int
-+welcome_do(struct Client *cptr, struct Client *sptr, char *name, char *text,
-+  char *who, time_t timestamp, unsigned int flags)
++welcome_do(struct Client *cptr, struct Client *sptr, char *name,
++  time_t timestamp, char *who, char *text, unsigned int flags)
 +{
 +  int nameint = atoi(name);                /* transform to int */
 +  int namearray = nameint - 1;             /* used to test the array element */
-+  char oldtext[TOPICLEN + 1];              /* save old text when unsetting */
-+  static time_t rate;
++  static time_t rate;                      /* rate limit snomask message */
 +
 +  assert(NULL != cptr);
 +  assert(NULL != sptr);
@@ -854,8 +1100,17 @@ diff -r 3a7d0a771452 ircd/welcome.c
 +  assert(NULL != who);
 +
 +  /* debug */
-+  Debug((DEBUG_DEBUG, "welcome_do(\"%s\", \"%s\", \"%s\", \"%s\" \"%s\", %Tu, 0x%04x)",
-+   cli_name(cptr), cli_name(sptr), name, text, who, timestamp, flags));
++  Debug((DEBUG_DEBUG, "welcome_do(\"%s\", \"%s\", \"%s\", %Tu, \"%s\", \"%s\", 0x%04x)",
++   cli_name(cptr), cli_name(sptr), name, timestamp, who, text, flags));
++
++  /* name empty after taking off the prefixes? */
++  if (EmptyString(name)) {
++    if (IsUser(sptr))
++      sendcmdto_one(&me, CMD_NOTICE, sptr, "%C :WELCOME: No message number given", sptr);
++    else 
++      protocol_violation(cptr, "WELCOME: No message number given by %C", sptr);
++    return 0;
++  }
 +
 +  /* check name */
 +  if (nameint < 1 || nameint > WELCOME_MAX_ENTRIES) {
@@ -863,8 +1118,16 @@ diff -r 3a7d0a771452 ircd/welcome.c
 +      sendcmdto_one(&me, CMD_NOTICE, sptr,
 +        "%C :WELCOME: Invalid message number %s - should between 1 and %d",
 +        sptr, name, WELCOME_MAX_ENTRIES);
-+    else 
-+      protocol_violation(cptr, "WELCOME: Invalid message number %s from %C", name, sptr);
++    else {
++      protocol_violation(cptr, "WELCOME: Invalid message number %s from %C - should be between 1 and %d",
++        name, sptr, WELCOME_MAX_ENTRIES);
++      /* nameint greater than MAX, perhaps we are upgrading, but used extra slots too soon?
++       *   propagate it manually
++       * TODO: cant do announce here?
++       */
++      if (nameint > WELCOME_MAX_ENTRIES && !(flags & WELCOME_LOCAL))
++        welcome_propagate(cptr, sptr, nameint, timestamp, who, text, flags);
++    }
 +    return 0;
 +  }
 +
@@ -872,6 +1135,9 @@ diff -r 3a7d0a771452 ircd/welcome.c
 +  if (flags & WELCOME_LOCAL)
 +    namearray += WELCOME_MAX_ENTRIES;
 +
++  /* must be true by now */
++  assert(namearray >= 0 && namearray <= 2 * WELCOME_MAX_ENTRIES - 1);
++
 +  /* cannot unset welcome that is not set */
 +  if (WelcomeArray[namearray].timestamp == 0 && EmptyString(text)) {
 +
@@ -891,7 +1157,7 @@ diff -r 3a7d0a771452 ircd/welcome.c
 +  if (WelcomeArray[namearray].timestamp != 0) {
 +
 +    /* global */
-+    if (namearray < WELCOME_MAX_ENTRIES) {
++    if (!(flags & WELCOME_LOCAL)) {
 +
 +      /* netburst and we got the same or a newer one
 +       *
@@ -931,70 +1197,18 @@ diff -r 3a7d0a771452 ircd/welcome.c
 +        sptr, cli_user(sptr)->server, timestamp - TStime()); 
 +  }
 +
-+  /* unsetting - do not announce, save text */
++  /* unset */
 +  if (EmptyString(text)) {
-+    flags &= ~WELCOME_ANNOUNCE;
-+    ircd_strncpy(oldtext, WelcomeArray[namearray].text, TOPICLEN);
-+  }
-+
-+  /* update */
-+  welcome_make(namearray, text, who, timestamp);
-+
-+  /* inform ops */
-+  sendto_opmask_butone(0, SNO_OLDSNO, "%s %s%s%s WELCOME %d \"%s\" [%Tu]",
-+    (feature_bool(FEAT_HIS_SNOTICES) || IsServer(sptr)) ?
-+    get_client_name_and_opername(sptr) : cli_name((cli_user(sptr))->server),
-+    EmptyString(text) ? "unsetting" : "changing",
-+    (flags & WELCOME_ANNOUNCE) ? " and announcing " : " ",
-+    (flags & WELCOME_LOCAL) ? "local" : "global",
-+    nameint,
-+    EmptyString(text) ? oldtext : WelcomeArray[namearray].text,
-+    WelcomeArray[namearray].timestamp);
-+
-+  /* log it */
-+  log_write(LS_NETWORK, L_INFO, LOG_NOSNOTICE, "%#C (%s) %s%s%s WELCOME %d \"%s\" [%Tu]",
-+    sptr, WelcomeArray[namearray].who,
-+    EmptyString(text) ? "unsetting" : "changing",
-+    (flags & WELCOME_ANNOUNCE) ? " and announcing " : " ",
-+    (flags & WELCOME_LOCAL) ? "local" : "global",
-+    nameint,
-+    EmptyString(text) ? oldtext : WelcomeArray[namearray].text,
-+    WelcomeArray[namearray].timestamp);
-+
-+  /* welcome set by remote user, inform oper of success */
-+  if ((flags & WELCOME_LOCAL) && IsUser(sptr) && !MyUser(sptr)) {
-+    sendcmdto_one(&me, CMD_NOTICE, sptr, "%C :%s %s%s local WELCOME %d \"%s\" [%Tu]",
-+      sptr, get_client_name_and_opername(sptr),
-+      EmptyString(text) ? "unsetting" : "changing",
-+      (flags & WELCOME_ANNOUNCE) ? " and announcing" : "",
-+      nameint,
-+      EmptyString(text) ? oldtext : WelcomeArray[namearray].text,
-+      WelcomeArray[namearray].timestamp);
-+
-+    /* TODO: wallops all local changes, by both local and remote opers? */
-+    /* tell all opers about the local message being set remotely */
-+    sendwallto_group_butone(&me, WALL_WALLOPS, 0,
-+      "%s %s%s local WELCOME %d \"%s\" [%Tu]",
-+      get_client_name_and_opername(sptr),
-+      EmptyString(text) ? "unsetting" : "changing",
-+      (flags & WELCOME_ANNOUNCE) ? " and announcing" : "",
-+      nameint,
-+      EmptyString(text) ? oldtext : WelcomeArray[namearray].text,
-+      WelcomeArray[namearray].timestamp);
++    flags &= ~WELCOME_INSERT;
++    return welcome_unset(cptr, sptr, nameint, namearray, timestamp, who, flags);
 +  }
 +
-+  /* propagate it */
-+  if (!(flags & WELCOME_LOCAL))
-+    sendcmdto_serv_butone(sptr, CMD_WELCOME, cptr, "* %s%d %Tu %s :%s",
-+      (flags & WELCOME_ANNOUNCE) ? "!" : "", nameint,
-+      WelcomeArray[namearray].timestamp, WelcomeArray[namearray].who,
-+      WelcomeArray[namearray].text);
-+
-+  /* announce it */
-+  if (flags & WELCOME_ANNOUNCE)
-+    welcome_announce(namearray);
++  /* insert */
++  if (flags & WELCOME_INSERT)
++    return welcome_insert(cptr, sptr, nameint, namearray, timestamp, who, text, flags);
 +
-+  return 0;
++  /* new or change */
++  return welcome_set(cptr, sptr, nameint, namearray, timestamp, who, text, flags);
 +}
 +
 +