]> jfr.im git - irc/quakenet/snircd-patchqueue.git/blobdiff - welcome.patch
showumodehtoclients: show usermode +h to clients
[irc/quakenet/snircd-patchqueue.git] / welcome.patch
index 54420fe8eea9a52d7c690c673c57133c08bb7e80..9509de8f633933678b02357650cbc1ea0fb3ec71 100644 (file)
@@ -77,22 +77,22 @@ ircd/ircd_lexer.l
 ircd/ircd_parser.y
 add PRIV_LOCAL_WELCOME PRIV_WELCOME
 
-diff -r 0e1d8ae7ca15 include/client.h
---- a/include/client.h Sun Feb 15 16:04:36 2009 +0100
-+++ b/include/client.h Sun Feb 15 16:05:56 2009 +0100
-@@ -149,6 +149,8 @@
-     PRIV_GLINE_LOOKUP,   /* staff can use /GLINE to lookup a gline */
-     PRIV_HIDE_CHANS,     /* oper can set usermode +n */
-     PRIV_CHECK_CHANNEL,  /* staff can /CHECK #channel */
+diff -r a9b437e961ec include/client.h
+--- a/include/client.h
++++ b/include/client.h
+@@ -142,6 +142,8 @@
+     PRIV_USER_PRIVACY,  /* oper can bypass user privacy +x etc gives i.e. see real ip's */
+     PRIV_CHANNEL_PRIVACY, /* oper can bypass channel privacy i.e. can see modes on channels they are not on and channel keys */ 
+     PRIV_SERVERINFO,     /* oper can use /get, /stats, /hash, retrieve remote information */
 +    PRIV_WELCOME,        /* oper can WELCOME */
 +    PRIV_LOCAL_WELCOME,  /* oper can local WELCOME */
      PRIV_LAST_PRIV /**< number of privileges */
    };
  
-diff -r 0e1d8ae7ca15 include/handlers.h
---- a/include/handlers.h       Sun Feb 15 16:04:36 2009 +0100
-+++ b/include/handlers.h       Sun Feb 15 16:05:56 2009 +0100
-@@ -154,6 +154,7 @@
+diff -r a9b437e961ec include/handlers.h
+--- a/include/handlers.h
++++ b/include/handlers.h
+@@ -138,6 +138,7 @@
  extern int m_version(struct Client*, struct Client*, int, char*[]);
  extern int m_wallchops(struct Client*, struct Client*, int, char*[]);
  extern int m_wallvoices(struct Client*, struct Client*, int, char*[]);
@@ -100,33 +100,25 @@ diff -r 0e1d8ae7ca15 include/handlers.h
  extern int m_who(struct Client*, struct Client*, int, char*[]);
  extern int m_whois(struct Client*, struct Client*, int, char*[]);
  extern int m_whowas(struct Client*, struct Client*, int, char*[]);
-@@ -189,6 +190,7 @@
+@@ -172,6 +173,7 @@
  extern int mo_version(struct Client*, struct Client*, int, char*[]);
  extern int mo_wallops(struct Client*, struct Client*, int, char*[]);
  extern int mo_wallusers(struct Client*, struct Client*, int, char*[]);
 +extern int mo_welcome(struct Client*, struct Client*, int, char*[]);
+ extern int mo_xquery(struct Client*, struct Client*, int, char*[]);
  extern int mr_error(struct Client*, struct Client*, int, char*[]);
  extern int mr_error(struct Client*, struct Client*, int, char*[]);
- extern int mr_pong(struct Client*, struct Client*, int, char*[]);
-@@ -247,6 +249,7 @@
+@@ -230,6 +232,7 @@
  extern int ms_wallops(struct Client*, struct Client*, int, char*[]);
  extern int ms_wallusers(struct Client*, struct Client*, int, char*[]);
  extern int ms_wallvoices(struct Client*, struct Client*, int, char*[]);
 +extern int ms_welcome(struct Client*, struct Client*, int, char*[]);
  extern int ms_whois(struct Client*, struct Client*, int, char*[]);
- extern int mh_nohelp(struct Client*, struct Client*, int, char*[]);
-@@ -313,6 +316,7 @@
- extern int mh_wallops(struct Client*, struct Client*, int, char*[]);
- extern int mh_wallusers(struct Client*, struct Client*, int, char*[]);
- extern int mh_wallvoices(struct Client*, struct Client*, int, char*[]);
-+extern int mh_welcome(struct Client*, struct Client*, int, char*[]);
- extern int mh_who(struct Client*, struct Client*, int, char*[]);
- extern int mh_whois(struct Client*, struct Client*, int, char*[]);
- extern int mh_whowas(struct Client*, struct Client*, int, char*[]);
-diff -r 0e1d8ae7ca15 include/ircd_features.h
---- a/include/ircd_features.h  Sun Feb 15 16:04:36 2009 +0100
-+++ b/include/ircd_features.h  Sun Feb 15 16:05:56 2009 +0100
+ extern int ms_xquery(struct Client*, struct Client*, int, char*[]);
+ extern int ms_xreply(struct Client*, struct Client*, int, char*[]);
+diff -r a9b437e961ec include/ircd_features.h
+--- a/include/ircd_features.h
++++ b/include/ircd_features.h
 @@ -101,6 +101,7 @@
    FEAT_IRCD_RES_TIMEOUT,
    FEAT_AUTH_TIMEOUT,
@@ -135,7 +127,7 @@ diff -r 0e1d8ae7ca15 include/ircd_features.h
  
    /* features that affect all operators */
    FEAT_EXTENDED_CHECKCMD,
-@@ -143,6 +144,7 @@
+@@ -142,6 +143,7 @@
    FEAT_HIS_STATS_u,
    FEAT_HIS_STATS_U,
    FEAT_HIS_STATS_v,
@@ -143,10 +135,10 @@ diff -r 0e1d8ae7ca15 include/ircd_features.h
    FEAT_HIS_STATS_w,
    FEAT_HIS_STATS_x,
    FEAT_HIS_STATS_y,
-diff -r 0e1d8ae7ca15 include/msg.h
---- a/include/msg.h    Sun Feb 15 16:04:36 2009 +0100
-+++ b/include/msg.h    Sun Feb 15 16:05:56 2009 +0100
-@@ -200,6 +200,10 @@
+diff -r a9b437e961ec include/msg.h
+--- a/include/msg.h
++++ b/include/msg.h
+@@ -196,6 +196,10 @@
  #define TOK_NOTICE              "O"
  #define CMD_NOTICE            MSG_NOTICE, TOK_NOTICE
  
@@ -157,18 +149,18 @@ diff -r 0e1d8ae7ca15 include/msg.h
  #define MSG_WALLCHOPS           "WALLCHOPS"     /* WC */
  #define TOK_WALLCHOPS           "WC"
  #define CMD_WALLCHOPS         MSG_WALLCHOPS, TOK_WALLCHOPS
-diff -r 0e1d8ae7ca15 include/numeric.h
---- a/include/numeric.h        Sun Feb 15 16:04:36 2009 +0100
-+++ b/include/numeric.h        Sun Feb 15 16:05:56 2009 +0100
+diff -r a9b437e961ec include/numeric.h
+--- a/include/numeric.h
++++ b/include/numeric.h
 @@ -116,6 +116,7 @@
        RPL_STATSGLINE       227           Dalnet 
        RPL_STATSVLINE       227           unreal */
  #define RPL_STATSALINE       226        /* Hybrid, Undernet */
 +#define RPL_STATSWELCOME     227        /* QuakeNet extension */
  #define RPL_STATSQLINE       228        /* Undernet extension */
- #define RPL_STATSHEADER      230        /* QuakeNet extension */
  
-@@ -445,6 +446,8 @@
+ /*      RPL_SERVICEINFO      231      unused */
+@@ -440,6 +441,8 @@
  /*      ERR_GHOSTEDCLIENT    503           efnet */
  /*    ERR_VWORLDWARN       503           austnet */
  
@@ -177,10 +169,10 @@ diff -r 0e1d8ae7ca15 include/numeric.h
  #define ERR_SILELISTFULL     511        /* Undernet extension */
  /*      ERR_NOTIFYFULL       512           aircd */
  /*    ERR_TOOMANYWATCH     512           Numeric List: Dalnet */
-diff -r 0e1d8ae7ca15 include/welcome.h
---- /dev/null  Thu Jan 01 00:00:00 1970 +0000
-+++ b/include/welcome.h        Sun Feb 15 16:05:56 2009 +0100
-@@ -0,0 +1,60 @@
+diff -r a9b437e961ec include/welcome.h
+--- /dev/null
++++ b/include/welcome.h
+@@ -0,0 +1,75 @@
 +#ifndef INCLUDED_welcome_h
 +#define INCLUDED_welcome_h
 +/*
@@ -216,15 +208,29 @@ diff -r 0e1d8ae7ca15 include/welcome.h
 +
 +/* Maximum number of welcome entries (per type; X global, X local) */
 +#define WELCOME_MAX_ENTRIES  10
-+/* Maximum timestamp drift in seconds allowed ahead of our idea of nettime
-+ * before we throw a warning to ops
-+ */
-+#define WELCOME_MAX_DRIFT   600
++/* Maximum length of a welcome message */
++#define WELCOMELEN TOPICLEN
++
++
++/* Test if a welcome entry is in a valid range */
++#define WelcomeIsValid(x)     ((x) >= 0 && (x) <= 2 * WELCOME_MAX_ENTRIES - 1)
++/* Test if a welcome entry is set */
++#define WelcomeIsSet(x)       (WelcomeArray[(x)].timestamp > 0)
++/* Test if a welcome entry is empty */
++#define WelcomeIsEmpty(x)     (*WelcomeArray[(x)].text == 0)
++
++/* Get welcome timestamp */
++#define WelcomeTS(x)          (WelcomeArray[(x)].timestamp)
++/* Get welcome text */
++#define WelcomeText(x)        (WelcomeArray[(x)].text)
++/* Get welcome who info */
++#define WelcomeWho(x)         (WelcomeArray[(x)].who)
++
 +
 +/* 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 */
 +};
 +
@@ -232,19 +238,20 @@ diff -r 0e1d8ae7ca15 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 0e1d8ae7ca15 ircd/Makefile.in
---- a/ircd/Makefile.in Sun Feb 15 16:04:36 2009 +0100
-+++ b/ircd/Makefile.in Sun Feb 15 16:05:56 2009 +0100
-@@ -188,6 +188,7 @@
+diff -r a9b437e961ec ircd/Makefile.in
+--- a/ircd/Makefile.in
++++ b/ircd/Makefile.in
+@@ -186,6 +186,7 @@
        m_wallops.c \
        m_wallusers.c \
        m_wallvoices.c \
@@ -260,7 +267,7 @@ diff -r 0e1d8ae7ca15 ircd/Makefile.in
        whocmds.c \
        whowas.c \
        y.tab.c
-@@ -1180,6 +1182,11 @@
+@@ -1161,6 +1163,11 @@
    ../include/ircd_reply.h ../include/ircd_string.h \
    ../include/ircd_chattr.h ../include/msg.h ../include/numeric.h \
    ../include/numnicks.h ../include/s_user.h ../include/send.h
@@ -272,7 +279,7 @@ diff -r 0e1d8ae7ca15 ircd/Makefile.in
  m_who.o: m_who.c ../config.h ../include/channel.h ../include/ircd_defs.h \
    ../include/res.h ../config.h ../include/client.h ../include/dbuf.h \
    ../include/msgq.h ../include/ircd_events.h ../include/ircd_handler.h \
-@@ -1441,6 +1448,13 @@
+@@ -1422,6 +1429,13 @@
    ../include/numnicks.h ../include/querycmds.h ../include/ircd_features.h \
    ../include/s_misc.h ../include/s_stats.h ../include/send.h \
    ../include/struct.h ../include/sys.h
@@ -286,36 +293,37 @@ diff -r 0e1d8ae7ca15 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 0e1d8ae7ca15 ircd/client.c
---- a/ircd/client.c    Sun Feb 15 16:04:36 2009 +0100
-+++ b/ircd/client.c    Sun Feb 15 16:05:56 2009 +0100
-@@ -184,6 +184,7 @@
+diff -r a9b437e961ec ircd/client.c
+--- a/ircd/client.c
++++ b/ircd/client.c
+@@ -177,6 +177,7 @@
      FlagSet(&privs_local, PRIV_WHOX);
      FlagSet(&privs_local, PRIV_DISPLAY);
      FlagSet(&privs_local, PRIV_FORCE_LOCAL_OPMODE);
 +    FlagSet(&privs_local, PRIV_LOCAL_WELCOME);
-     FlagClr(&privs_local, PRIV_STAFF);
-     FlagClr(&privs_local, PRIV_CLAIM_NICK);
-     FlagClr(&privs_local, PRIV_GLINE_LOOKUP);
-@@ -247,6 +248,7 @@
+     privs_defaults_set = 1;
+   }
+@@ -223,6 +224,7 @@
+     ClrPriv(client, PRIV_JUPE);
      ClrPriv(client, PRIV_OPMODE);
-     ClrPriv(client, PRIV_OPKICK);
      ClrPriv(client, PRIV_BADCHAN);
 +    ClrPriv(client, PRIV_WELCOME);
    }
-   /* TODO: better way than this? */
-   /* do not let staff have privs they should not have */
-@@ -283,6 +285,7 @@
-   P(USER_PRIVACY),
-   P(STAFF),          P(CLAIM_NICK),     P(GLINE_LOOKUP),  P(HIDE_CHANS),
-   P(CHECK_CHANNEL),
-+  P(WELCOME),        P(LOCAL_WELCOME),
+ }
+@@ -244,7 +246,7 @@
+   P(CHANSERV),       P(XTRA_OPER),      P(NOIDLE),        P(FREEFORM),
+   P(PARANOID),       P(CHECK),          P(WALL),          P(CLOSE),
+   P(ROUTE),          P(ROUTEINFO),      P(SERVERINFO),    P(CHANNEL_PRIVACY),
+-  P(USER_PRIVACY),
++  P(USER_PRIVACY),   P(WELCOME),        P(LOCAL_WELCOME),
  #undef P
    { 0, 0 }
  };
-diff -r 0e1d8ae7ca15 ircd/ircd_features.c
---- a/ircd/ircd_features.c     Sun Feb 15 16:04:36 2009 +0100
-+++ b/ircd/ircd_features.c     Sun Feb 15 16:05:56 2009 +0100
+diff -r a9b437e961ec ircd/ircd_features.c
+--- a/ircd/ircd_features.c
++++ b/ircd/ircd_features.c
 @@ -366,6 +366,7 @@
    F_I(IRCD_RES_TIMEOUT, 0, 4, 0),
    F_I(AUTH_TIMEOUT, 0, 9, 0),
@@ -324,7 +332,7 @@ diff -r 0e1d8ae7ca15 ircd/ircd_features.c
  
    /* features that affect all operators */
    F_B(EXTENDED_CHECKCMD, 0, 0, 0),
-@@ -408,6 +409,7 @@
+@@ -407,6 +408,7 @@
    F_B(HIS_STATS_u, 0, 1, 0),
    F_B(HIS_STATS_U, 0, 1, 0),
    F_B(HIS_STATS_v, 0, 1, 0),
@@ -332,42 +340,42 @@ diff -r 0e1d8ae7ca15 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 0e1d8ae7ca15 ircd/ircd_lexer.l
---- a/ircd/ircd_lexer.l        Sun Feb 15 16:04:36 2009 +0100
-+++ b/ircd/ircd_lexer.l        Sun Feb 15 16:05:56 2009 +0100
-@@ -173,6 +173,8 @@
-   { "gline_lookup", TPRIV_GLINE_LOOKUP },
-   { "hide_chans", TPRIV_HIDE_CHANS },
-   { "check_channel", TPRIV_CHECK_CHANNEL },
+diff -r a9b437e961ec ircd/ircd_lexer.l
+--- a/ircd/ircd_lexer.l
++++ b/ircd/ircd_lexer.l
+@@ -166,6 +166,8 @@
+   { "serverinfo", TPRIV_SERVERINFO },
+   { "user_privacy", TPRIV_USER_PRIVACY },
+   { "channel_privacy", TPRIV_CHANNEL_PRIVACY },
 +  { "local_welcome", TPRIV_LOCAL_WELCOME },
 +  { "welcome", TPRIV_WELCOME },
    { NULL, 0 }
  };
  static int ntokens;
-diff -r 0e1d8ae7ca15 ircd/ircd_parser.y
---- a/ircd/ircd_parser.y       Sun Feb 15 16:04:36 2009 +0100
-+++ b/ircd/ircd_parser.y       Sun Feb 15 16:05:56 2009 +0100
-@@ -192,6 +192,7 @@
- %token TPRIV_CHANNEL_PRIVACY TPRIV_USER_PRIVACY TPRIV_LIST_CHAN
- %token TPRIV_STAFF TPRIV_CLAIM_NICK TPRIV_GLINE_LOOKUP TPRIV_HIDE_CHANS
- %token TPRIV_CHECK_CHANNEL
+diff -r a9b437e961ec ircd/ircd_parser.y
+--- a/ircd/ircd_parser.y
++++ b/ircd/ircd_parser.y
+@@ -189,6 +189,7 @@
+ %token TPRIV_CHANSERV TPRIV_XTRA_OPER TPRIV_NOIDLE TPRIV_FREEFORM TPRIV_PARANOID
+ %token TPRIV_CHECK TPRIV_WALL TPRIV_CLOSE TPRIV_ROUTE TPRIV_ROUTEINFO TPRIV_SERVERINFO
+ %token TPRIV_CHANNEL_PRIVACY TPRIV_USER_PRIVACY TPRIV_LIST_CHAN 
 +%token TPRIV_LOCAL_WELCOME TPRIV_WELCOME
  /* and some types... */
  %type <num> sizespec
  %type <num> timespec timefactor factoredtimes factoredtime
-@@ -715,6 +716,8 @@
-           TPRIV_GLINE_LOOKUP { $$ = PRIV_GLINE_LOOKUP; } |
-           TPRIV_CHECK_CHANNEL { $$ = PRIV_CHECK_CHANNEL; } |
-           TPRIV_HIDE_CHANS { $$ = PRIV_HIDE_CHANS; } |
+@@ -703,6 +704,8 @@
+           TPRIV_SERVERINFO { $$ = PRIV_SERVERINFO ; } |
+           TPRIV_CHANNEL_PRIVACY { $$ = PRIV_CHANNEL_PRIVACY ; } |
+           TPRIV_USER_PRIVACY { $$ = PRIV_USER_PRIVACY ; } |
 +          TPRIV_LOCAL_WELCOME { $$ = PRIV_LOCAL_WELCOME; } |
 +          TPRIV_WELCOME { $$ = PRIV_WELCOME; } |
            TPRIV_PARANOID { $$ = PRIV_PARANOID; } ;
  yesorno: YES { $$ = 1; } | NO { $$ = 0; };
  
-diff -r 0e1d8ae7ca15 ircd/m_welcome.c
---- /dev/null  Thu Jan 01 00:00:00 1970 +0000
-+++ b/ircd/m_welcome.c Sun Feb 15 16:05:56 2009 +0100
-@@ -0,0 +1,308 @@
+diff -r a9b437e961ec ircd/m_welcome.c
+--- /dev/null
++++ b/ircd/m_welcome.c
+@@ -0,0 +1,296 @@
 +/*
 + * IRC - Internet Relay Chat, ircd/m_welcome.c
 + * Copyright (C) 1990 Jarkko Oikarinen and
@@ -487,14 +495,11 @@ diff -r 0e1d8ae7ca15 ircd/m_welcome.c
 +  /* remote listing request, see if it is for me or a remote server
 +   *  check FEAT_HIS_REMOTE to decide if an ordinary user can do this
 +   */
-+  if (parc == 2) {
-+    if (hunt_server_cmd(sptr, CMD_WELCOME, cptr, feature_int(FEAT_HIS_REMOTE),
-+      "%C", 1, parc, parv) != HUNTED_ISME)
-+      return 0;
-+    return welcome_list(sptr, 0);
-+  }
++  if ((parc > 1) && (hunt_server_cmd(sptr, CMD_WELCOME, cptr, feature_int(FEAT_HIS_REMOTE),
++        "%C", 1, parc, parv) != HUNTED_ISME))
++    return 0;
 +
-+  /* just local listing */
++  /* local listing */
 +  return welcome_list(sptr, 0);
 +}
 +
@@ -526,17 +531,14 @@ diff -r 0e1d8ae7ca15 ircd/m_welcome.c
 +  if (!feature_bool(FEAT_WELCOME))
 +    return send_reply(sptr, ERR_DISABLED, "WELCOME");
 +
-+  /* listing */
-+  if (parc < 2)
-+    return welcome_list(sptr, 0);
-+
 +  /* TODO: move feature check here? */
 +  /* remote listing request, see if it is for me or a remote server */
-+  if (parc == 2) {
-+    if (hunt_server_cmd(sptr, CMD_WELCOME, cptr, 0, "%C", 1, parc, parv) != HUNTED_ISME)
-+      return 0;
++  if ((parc == 2) && (hunt_server_cmd(sptr, CMD_WELCOME, cptr, 0, "%C", 1, parc, parv) != HUNTED_ISME))
++    return 0;
++
++  /* local listing */
++  if (parc <= 2)
 +    return welcome_list(sptr, 0);
-+  }
 +
 +  /* check PRIVS */
 +  /* local - need PRIV LOCAL_WELCOME or WELCOME */
@@ -588,9 +590,15 @@ diff -r 0e1d8ae7ca15 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);
 +}
 +
 +
@@ -618,6 +626,8 @@ diff -r 0e1d8ae7ca15 ircd/m_welcome.c
 +
 +  /* remote listing request, see if it is for me or a remote server */
 +  if (parc == 2) {
++    if (IsServer(sptr))
++      return protocol_violation(cptr, "WELCOME listing request from server %C", sptr);
 +    if (hunt_server_cmd(sptr, CMD_WELCOME, cptr, 0, "%C", 1, parc, parv) != HUNTED_ISME)
 +      return 0;
 +    return welcome_list(sptr, 0);
@@ -653,35 +663,21 @@ diff -r 0e1d8ae7ca15 ircd/m_welcome.c
 +     flags |= WELCOME_ANNOUNCE;
 +  }
 +
-+  /* and do it */
-+  return welcome_do(cptr, sptr, name, text, who, timestamp, flags);
-+}
-+
-+
-+/*
-+ * mh_welcome - help message handler
-+ */
-+int mh_welcome(struct Client* cptr, struct Client* sptr, int parc, char* parv[])
-+{
-+  if (!IsAnOper(sptr)) {
-+    send_reply(sptr, SND_EXPLICIT | RPL_HELP,
-+      "WELCOME :WELCOME");
-+    send_reply(sptr, SND_EXPLICIT | RPL_HELP,
-+      "WELCOME :Shows welcome messages set on the server.");  
-+  } else {
-+    send_reply(sptr, SND_EXPLICIT | RPL_HELP,
-+      "WELCOME :WELCOME [<target>] [[!]<name> :<message>]");
-+    send_reply(sptr, SND_EXPLICIT | RPL_HELP,
-+      "WELCOME :Shows or sets welcome messages on a server.");
++  /* check for insert prefix */
++  if (*name == '+') {
++     name++;
++     flags |= WELCOME_INSERT;
 +  }
-+  return 0;
++
++  /* and do it */
++  return welcome_do(cptr, sptr, name, timestamp, who, text, flags);
 +}
-diff -r 0e1d8ae7ca15 ircd/parse.c
---- a/ircd/parse.c     Sun Feb 15 16:04:36 2009 +0100
-+++ b/ircd/parse.c     Sun Feb 15 16:05:56 2009 +0100
-@@ -674,6 +674,15 @@
-     /* UNREG, CLIENT, SERVER, OPER, SERVICE, HELP */
-     { m_unregistered, m_not_oper, ms_opkick, mo_opkick, m_ignore, mh_nohelp }
+diff -r a9b437e961ec ircd/parse.c
+--- a/ircd/parse.c
++++ b/ircd/parse.c
+@@ -661,6 +661,15 @@
+     /* UNREG, CLIENT, SERVER, OPER, SERVICE */
+     { m_unregistered, m_not_oper, ms_check, mo_check, m_ignore }
    },
 +  
 +  /* add command for WELCOME */
@@ -690,14 +686,14 @@ diff -r 0e1d8ae7ca15 ircd/parse.c
 +    TOK_WELCOME,
 +    0, MAXPARA, MFLG_SLOW, 0, NULL,
 +    /* UNREG, CLIENT, SERVER, OPER, SERVICE, HELP */
-+    { m_unregistered, m_welcome, ms_welcome, mo_welcome, m_ignore, mh_welcome }
++    { m_unregistered, m_welcome, ms_welcome, mo_welcome, m_ignore }
 +  },
  
    /* 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 0e1d8ae7ca15 ircd/s_err.c
---- a/ircd/s_err.c     Sun Feb 15 16:04:36 2009 +0100
-+++ b/ircd/s_err.c     Sun Feb 15 16:05:56 2009 +0100
+diff -r a9b437e961ec ircd/s_err.c
+--- a/ircd/s_err.c
++++ b/ircd/s_err.c
 @@ -486,7 +486,7 @@
  /* 226 */
    { RPL_STATSALINE, "%s", "226" },
@@ -716,9 +712,9 @@ diff -r 0e1d8ae7ca15 ircd/s_err.c
  /* 510 */
    { 0 },
  /* 511 */
-diff -r 0e1d8ae7ca15 ircd/s_serv.c
---- a/ircd/s_serv.c    Sun Feb 15 16:04:36 2009 +0100
-+++ b/ircd/s_serv.c    Sun Feb 15 16:05:56 2009 +0100
+diff -r a9b437e961ec ircd/s_serv.c
+--- a/ircd/s_serv.c
++++ b/ircd/s_serv.c
 @@ -57,6 +57,7 @@
  #include "struct.h"
  #include "sys.h"
@@ -735,9 +731,9 @@ diff -r 0e1d8ae7ca15 ircd/s_serv.c
  
    /*
     * Pass on my client information to the new server
-diff -r 0e1d8ae7ca15 ircd/s_stats.c
---- a/ircd/s_stats.c   Sun Feb 15 16:04:36 2009 +0100
-+++ b/ircd/s_stats.c   Sun Feb 15 16:05:56 2009 +0100
+diff -r a9b437e961ec ircd/s_stats.c
+--- a/ircd/s_stats.c
++++ b/ircd/s_stats.c
 @@ -54,6 +54,7 @@
  #include "send.h"
  #include "struct.h"
@@ -746,7 +742,7 @@ diff -r 0e1d8ae7ca15 ircd/s_stats.c
  
  #include <stdio.h>
  #include <stdlib.h>
-@@ -691,9 +692,12 @@
+@@ -654,9 +655,12 @@
    { 'V', "vserversmach", (STAT_FLAG_OPERFEAT | STAT_FLAG_VARPARAM | STAT_FLAG_CASESENS), FEAT_HIS_STATS_v,
      stats_servers_verbose, 0,
      "Verbose server information." },
@@ -760,9 +756,9 @@ diff -r 0e1d8ae7ca15 ircd/s_stats.c
    { 'x', "memusage", STAT_FLAG_OPERFEAT, FEAT_HIS_STATS_x,
      stats_meminfo, 0,
      "List usage information." },
-diff -r 0e1d8ae7ca15 ircd/s_user.c
---- a/ircd/s_user.c    Sun Feb 15 16:04:36 2009 +0100
-+++ b/ircd/s_user.c    Sun Feb 15 16:05:56 2009 +0100
+diff -r a9b437e961ec ircd/s_user.c
+--- a/ircd/s_user.c
++++ b/ircd/s_user.c
 @@ -63,6 +63,7 @@
  #include "userload.h"
  #include "version.h"
@@ -771,7 +767,7 @@ diff -r 0e1d8ae7ca15 ircd/s_user.c
  
  #include "handlers.h" /* m_motd and m_lusers */
  
-@@ -413,6 +414,9 @@
+@@ -410,6 +411,9 @@
                             cli_info(sptr), NumNick(cptr) /* two %s's */);
  
      IPcheck_connect_succeeded(sptr);
@@ -781,10 +777,10 @@ diff -r 0e1d8ae7ca15 ircd/s_user.c
    }
    else {
      struct Client *acptr = user->server;
-diff -r 0e1d8ae7ca15 ircd/welcome.c
---- /dev/null  Thu Jan 01 00:00:00 1970 +0000
-+++ b/ircd/welcome.c   Sun Feb 15 16:05:56 2009 +0100
-@@ -0,0 +1,373 @@
+diff -r a9b437e961ec ircd/welcome.c
+--- /dev/null
++++ b/ircd/welcome.c
+@@ -0,0 +1,611 @@
 +/*
 + * IRC - Internet Relay Chat, ircd/welcome.c
 + * Copyright (C) 1990 Jarkko Oikarinen and
@@ -848,11 +844,11 @@ diff -r 0e1d8ae7ca15 ircd/welcome.c
 +static int 
 +welcome_make(int name, char *text, char *who, time_t timestamp)
 +{
-+  /* range 0 to 2 * max - 1 */
-+  assert(name >= 0 && name <= 2 * WELCOME_MAX_ENTRIES - 1);
++  /* valid range */
++  assert(WelcomeIsValid(name));
 +
 +  /* 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;
 +
@@ -860,23 +856,278 @@ diff -r 0e1d8ae7ca15 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)
++{
++  /* must be global */
++  assert(!(flags & WELCOME_LOCAL));
++  assert(NULL != sptr);
++  assert(NULL != cptr);
++
++  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;
++}
++
++
++/** Resend a welcome message.
++ * @param[in] cptr Local client that sent us the welcome.
++ * @param[in] nameint Name of the message.
++ * @param[in] namearray Name of the array item.
++ * @return Zero
++ */
++int
++welcome_resend(struct Client *cptr, int nameint, int namearray)
++{
++  /* must be valid */
++  assert(WelcomeIsValid(namearray));
++  assert(NULL != cptr);
++  assert(IsServer(cptr));
++
++  sendcmdto_one(&me, CMD_WELCOME, cptr, "* %d %Tu %s :%s",
++    nameint, WelcomeTS(namearray), WelcomeWho(namearray), WelcomeText(namearray));
++
++  return 0;
++}
++
++
++/** Log a welcome message.
++ * @param[in] sptr Originator of the welcome.
++ * @param[in] msg The message to show.
++ * @param[in] flags Flags to set on welcome.
++ * @return Zero
++ */
++int
++welcome_log(struct Client *sptr, char *msg, unsigned int flags)
++{
++  assert(NULL != sptr);
++  assert(NULL != msg);
++
++  /* 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, "%s %s", get_client_name_and_opername(sptr), 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 logging */
++  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 (WelcomeIsEmpty(namearray))
++    new = 1;
++
++  /* update */
++  welcome_make(namearray, text, who, timestamp);
++
++  /* create msg for log */
++  ircd_snprintf(0, msg, 0, "%s%s%s WELCOME %d \"%s\" %s [%Tu]",
++    new ? "setting" : "changing",
++    (flags & WELCOME_ANNOUNCE) ? " and announcing " : " ",
++    (flags & WELCOME_LOCAL) ? "local" : "global",
++    nameint, WelcomeText(namearray), WelcomeWho(namearray), timestamp);
++
++  /* log it */
++  welcome_log(sptr, msg, 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 logging */
++  int i;                                /* loop variable */
++  int empty = namearray;                /* first empty spot in array after namearray */
++  int end = WELCOME_MAX_ENTRIES -1;     /* last element to check in array */ 
++
++  /* 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));
++
++  /* create msg for log */
++  ircd_snprintf(0, msg, 0, "unsetting %s WELCOME %d \"%s\" %s [%Tu]",
++    (flags & WELCOME_LOCAL) ? "local" : "global",
++    nameint, WelcomeText(namearray), WelcomeWho(namearray), timestamp);
++
++  /* log it */
++  welcome_log(sptr, msg, flags);
++
++  /* update */
++  welcome_make(namearray, "", who, timestamp);
++
++  /* propagate it, but not when inserting */
++  if (!(flags & (WELCOME_LOCAL|WELCOME_INSERT)))
++    welcome_propagate(cptr, sptr, nameint, timestamp, who, "", flags);
++
++  /* correct end for local offset */
++  if (flags & WELCOME_LOCAL)
++    end += WELCOME_MAX_ENTRIES;
++
++  /* move entries up, update timestamp */
++  for (i = namearray; i < end; i++)
++    welcome_make(i, WelcomeText(i+1), WelcomeWho(i+1), timestamp);
++
++  /* clear last entry, update timestamp */
++  welcome_make(end, "", who, timestamp);
++
++  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 logging */
++  int i;                                /* loop variable */
++  int empty = -1;                       /* first empty spot in array after namearray */
++  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));
++
++  /* correct end for local offset */
++  if (flags & WELCOME_LOCAL)
++    end += WELCOME_MAX_ENTRIES;
++
++  /* find first empty spot */
++  for (i = namearray; i <= end; i++) {
++    if (WelcomeIsEmpty(i)) {
++      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, WelcomeText(i-1), WelcomeWho(i-1), timestamp);
++
++  /* correct empty for local offset */
++  if (flags & WELCOME_LOCAL)
++    empty -= WELCOME_MAX_ENTRIES;
++
++  /* create msg for log */
++  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, 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;
++  int max = WELCOME_MAX_ENTRIES;           /* max number of entries */
 +
 +  assert(NULL != cptr);
 +  assert(NULL != sptr);
@@ -885,15 +1136,43 @@ diff -r 0e1d8ae7ca15 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));
++
++  /* if for some reason timestamp is 0, increase it */
++  if (timestamp == 0)
++    timestamp++;
++
++  /* 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) {
++  if (!WelcomeIsValid(namearray)) {
 +    if (IsUser(sptr))
 +      sendcmdto_one(&me, CMD_NOTICE, sptr,
 +        "%C :WELCOME: Invalid message number %s - should between 1 and %d",
-+        sptr, name, WELCOME_MAX_ENTRIES); 
++        sptr, name, max);
++    else
++      protocol_violation(cptr,
++        "WELCOME: Invalid message number %s from %C - should be between 1 and %d",
++        name, sptr, max);
++    return 0;
++  }
++
++  /* source is user, and is myuser or welcome is local, check length of the message */
++  if ((IsUser(sptr)) && ((MyUser(sptr)) || (flags & WELCOME_LOCAL)) && (strlen(text) > WELCOMELEN)) {
++    sendcmdto_one(&me, CMD_NOTICE, sptr,
++      "%C :WELCOME: The message is too long with %d chars - max is %d chars",
++      sptr, strlen(text), WELCOMELEN);
++    ircd_strncpy(text, text, WELCOMELEN);
++    sendcmdto_one(&me, CMD_NOTICE, sptr,
++      "%C :WELCOME: Change or truncate the message to: \"%s\"", sptr, text);
 +    return 0;
 +  }
 +
@@ -901,8 +1180,11 @@ diff -r 0e1d8ae7ca15 ircd/welcome.c
 +  if (flags & WELCOME_LOCAL)
 +    namearray += WELCOME_MAX_ENTRIES;
 +
++  /* must be true by now */
++  assert(WelcomeIsValid(namearray));
++
 +  /* cannot unset welcome that is not set */
-+  if (WelcomeArray[namearray].timestamp == 0 && EmptyString(text)) {
++  if (!WelcomeIsSet(namearray) && EmptyString(text)) {
 +
 +    /* from user, throw error */
 +    if (IsUser(sptr))
@@ -917,113 +1199,71 @@ diff -r 0e1d8ae7ca15 ircd/welcome.c
 +
 +  /* check if there is something to change */
 +  /* we got a record for it */
-+  if (WelcomeArray[namearray].timestamp != 0) {
++  if (WelcomeIsSet(namearray)) {
 +
 +    /* global */
-+    if (namearray < WELCOME_MAX_ENTRIES) {
-+
-+      /* netburst and we got the same or a newer one
-+       *
-+       *   we only use the timestamp for resolving conflicts in net burst
-+       *   outside of netburst, we simply parse whatever we get
-+       *   this way we will not get stuck with a welcome message set by a server
-+       *   running ahead with the time
++    if (!(flags & WELCOME_LOCAL)) {
++
++      /* myuser changes it,
++       *   welcomeTS greater than or equal to timestamp, take welcomeTS+1 as timestamp
++       *   else the change is not accepted upstream because of the older TS
 +       */
-+      if (IsBurstOrBurstAck(cptr) && timestamp <= WelcomeArray[namearray].timestamp)
-+        return 0;
++      if (MyUser(sptr) && WelcomeTS(namearray) >= timestamp)
++        timestamp = WelcomeTS(namearray) +1;
++
++      /* compare timestamps, ignore welcome when:
++       *  we got a newer one
++       *  or when timestamps are the same
++       *  our text is shorter or our text is 'smaller'
++       */
++      if ((timestamp < WelcomeTS(namearray)) ||                 /* we got a newer one */
++           ((timestamp == WelcomeTS(namearray)) &&              /* same timestamp */
++           ((strlen(WelcomeText(namearray)) < strlen(text)) ||  /* our text is shorter */
++           (ircd_strcmp(WelcomeText(namearray), text) < 0)))) { /* our text is 'smaller' */
++        /* burst or burst ack, cptr gets our version from the burst */
++        if (IsBurstOrBurstAck(cptr))
++          return 0;
++        /* sync server */
++        return welcome_resend(cptr, nameint, namearray);
++      }
 +
 +    /* local welcome - we use our idea of the time */
 +    } else
 +      timestamp = TStime();
 +
-+    /* compare new message with old message */
-+    if (ircd_strcmp(text, WelcomeArray[namearray].text) == 0) {
-+      if (IsUser(sptr))
++    /* new global welcome from my user or local welcome
++     *   compare new message with old message
++     */
++    if (IsUser(sptr) && (MyUser(sptr) || (flags & WELCOME_LOCAL))) {
++      if (ircd_strcmp(text, WelcomeText(namearray)) == 0) {
 +        sendcmdto_one(&me, CMD_NOTICE, sptr,
-+          "%C :WELCOME: Cannot change %smessage for %s - nothing to change.",
-+          sptr, (flags & WELCOME_LOCAL) ? "local " : "", name); 
-+      return 0;
++          "%C :WELCOME: Cannot change %s message for %s - nothing to change",
++          sptr, (flags & WELCOME_LOCAL) ? "local" : "global", name); 
++        return 0;
++      }
 +    }
 +  }
 +
-+  /* TODO: rate limited for what? max 10 welcome messages..? */
-+  /* possible timestamp drift - warn ops */
-+  if (timestamp - TStime() > WELCOME_MAX_DRIFT) {
-+    sendto_opmask_butone_ratelimited(0, SNO_NETWORK, &rate,
-+      "Possible timestamp drift from %C; timestamp in WELCOME message is %is ahead of time",
-+      IsServer(sptr) ? sptr : cli_user(sptr)->server, timestamp - TStime());
-+
-+    /* warn remote oper too */
-+    if (IsUser(sptr))
-+      sendcmdto_one(&me, CMD_NOTICE, sptr,
-+        "%C :Possible timestamp drift from %C; timestamp in WELCOME message is %is ahead of time",
-+        sptr, cli_user(sptr)->server, timestamp - TStime()); 
-+  }
++  /* do not insert for last global/local entry and when not set yet */
++  if ((flags & WELCOME_INSERT) && ((!WelcomeIsSet(namearray)) || (nameint == max)))
++    flags &= ~WELCOME_INSERT;
 +
-+  /* 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%sWELCOME %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 " : "",
-+    nameint,
-+    EmptyString(text) ? oldtext : WelcomeArray[namearray].text,
-+    WelcomeArray[namearray].timestamp);
-+
-+  /* log it */
-+  log_write(LS_NETWORK, L_INFO, LOG_NOSNOTICE, "%#C (%s) %s%s%sWELCOME %d \"%s\" [%Tu]",
-+    sptr, WelcomeArray[namearray].who,
-+    EmptyString(text) ? "unsetting" : "changing",
-+    (flags & WELCOME_ANNOUNCE) ? " and announcing " : " ",
-+    (flags & WELCOME_LOCAL) ? "local " : "",
-+    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);
++    /* clear insert flag,
++     * when this flag is set, welcome_unset() assumes it is being called from welcome_insert()
++     * and wont propagate the change
++     */
++    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);
 +}
 +
 +
@@ -1037,8 +1277,10 @@ diff -r 0e1d8ae7ca15 ircd/welcome.c
 +  struct MsgBuf *msgbuf;
 +  int i;
 +
-+  /* range 0 to 2 * max - 1 */
-+  assert(name >= 0 && name <= 2 * WELCOME_MAX_ENTRIES - 1); 
++  /* valid range, set and not empty */
++  assert(WelcomeIsValid(name));
++  assert(WelcomeIsSet(name));
++  assert(!WelcomeIsEmpty(name));
 +
 +  /* TODO: target is $* as if it were a global broadcast
 +   * could make it $servername for local message announcement
@@ -1048,14 +1290,12 @@ diff -r 0e1d8ae7ca15 ircd/welcome.c
 +  /* build msgbuf */
 +  msgbuf = msgq_make(0, ":%C %s $* :[%s] %s", &me, MSG_NOTICE,
 +    name >= WELCOME_MAX_ENTRIES ? cli_name(&me) : feature_str(FEAT_NETWORK),
-+    WelcomeArray[name].text);
++    WelcomeText(name));
 +
 +  /* go over local clients */
 +  for (i = HighestFd; i > 0; --i) {
 +
-+    /* skip unregistered clients - they see the message during login
-+     * skip servers
-+     */
++    /* skip unregistered clients, skip servers */
 +    if (!(acptr = LocalClientArray[i]) || !IsRegistered(acptr) || IsServer(acptr))
 +      continue;
 +
@@ -1077,10 +1317,9 @@ diff -r 0e1d8ae7ca15 ircd/welcome.c
 +
 +  /* loop over global entries - 0 to max - 1*/
 +  for (name = 0; name <= WELCOME_MAX_ENTRIES - 1; name++) {
-+    if (WelcomeArray[name].timestamp != 0)
++    if (WelcomeIsSet(name))
 +      sendcmdto_one(&me, CMD_WELCOME, cptr, "* %d %Tu %s :%s",
-+        name + 1, WelcomeArray[name].timestamp, WelcomeArray[name].who,
-+        WelcomeArray[name].text);
++        name + 1, WelcomeTS(name), WelcomeWho(name), WelcomeText(name));
 +  }
 +}
 +
@@ -1105,14 +1344,13 @@ diff -r 0e1d8ae7ca15 ircd/welcome.c
 +      local = 1;
 +
 +    /* not set or empty - skip */
-+    /* TODO: EmptyString? */
-+    if (WelcomeArray[name].timestamp == 0 || *WelcomeArray[name].text == 0)
++    if (!WelcomeIsSet(name) || WelcomeIsEmpty(name))
 +      continue;
 +
 +    /* got one */
 +    found++;
 +    sendcmdto_one(&me, CMD_NOTICE, sptr, "%C :[%s] %s",
-+      sptr, local ? cli_name(&me) : feature_str(FEAT_NETWORK), WelcomeArray[name].text);
++      sptr, local ? cli_name(&me) : feature_str(FEAT_NETWORK), WelcomeText(name));
 +  }
 +
 +  /* nothing set */
@@ -1135,10 +1373,6 @@ diff -r 0e1d8ae7ca15 ircd/welcome.c
 +
 +  assert(NULL != sptr);
 +
-+  /* send header so the client knows what we are showing */
-+  send_reply(sptr, SND_EXPLICIT | RPL_STATSHEADER,
-+    "W Name Target Opername Timestamp :Message");
-+
 +  /* loop over all entries - range 0 to 2 * max - 1*/
 +  for (name = 0; name <= 2 * WELCOME_MAX_ENTRIES - 1; name++) {
 +
@@ -1147,14 +1381,14 @@ diff -r 0e1d8ae7ca15 ircd/welcome.c
 +      local = 1;
 +
 +    /* not set */
-+    if (WelcomeArray[name].timestamp == 0)
++    if (!WelcomeIsSet(name))
 +      continue;
 +
 +    /* send it */
 +    send_reply(sptr, RPL_STATSWELCOME,
 +      local ? name + 1 - WELCOME_MAX_ENTRIES : name + 1,
 +      local ? cli_name(&me) : "*",
-+      WelcomeArray[name].who, WelcomeArray[name].timestamp,
-+      EmptyString(WelcomeArray[name].text)  ? "<Empty>" : WelcomeArray[name].text);
++      WelcomeWho(name), WelcomeTS(name),
++      WelcomeIsEmpty(name) ? "<Empty>" : WelcomeText(name));
 +  }
 +}