]> jfr.im git - irc/evilnet/x3.git/blobdiff - src/nickserv.c
fixing delpeon bug
[irc/evilnet/x3.git] / src / nickserv.c
index 08faff51c92e8e8fae7ffce68802c8ea964a7352..e189e7778e067b939e6a2bf422aa7f63df38f8b3 100644 (file)
@@ -27,9 +27,7 @@
 #include "sendmail.h"
 #include "timeq.h"
 
-#ifdef HAVE_REGEX_H
-#include <regex.h>
-#endif
+#include <tre/regex.h>
 
 #define NICKSERV_CONF_NAME "services/nickserv"
 
@@ -44,6 +42,7 @@
 #define KEY_VALID_HANDLE_REGEX "valid_handle_regex"
 #define KEY_VALID_ACCOUNT_REGEX "valid_account_regex"
 #define KEY_VALID_NICK_REGEX "valid_nick_regex"
+#define KEY_VALID_FAKEHOST_REGEX "valid_fakehost_regex"
 #define KEY_DB_BACKUP_FREQ "db_backup_freq"
 #define KEY_MODOPER_LEVEL "modoper_level"
 #define KEY_SET_EPITHET_LEVEL "set_epithet_level"
 #define NICKSERV_VALID_CHARS   "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789-_"
 
 #define NICKSERV_FUNC(NAME) MODCMD_FUNC(NAME)
-#define OPTION_FUNC(NAME) int NAME(struct userNode *user, struct handle_info *hi, UNUSED_ARG(unsigned int override), unsigned int argc, char *argv[])
+#define OPTION_FUNC(NAME) int NAME(struct svccmd *cmd, struct userNode *user, struct handle_info *hi, UNUSED_ARG(unsigned int override), unsigned int argc, char *argv[])
 typedef OPTION_FUNC(option_func_t);
 
 DEFINE_LIST(handle_info_list, struct handle_info*);
@@ -176,7 +175,7 @@ static const struct message_entry msgtab[] = {
     { "NSMSG_ATE_FOREIGN_COOKIE", "I ate the cookie for account $b%s$b.  It may now have another." },
     { "NSMSG_USE_RENAME", "You are already authenticated to account $b%s$b -- contact the support staff to rename your account." },
     { "NSMSG_ALREADY_REGISTERING", "You have already used $bREGISTER$b once this session; you may not use it again." },
-    { "NSMSG_REGISTER_BAD_NICKMASK", "Could not recognize $b%s$b as either a current nick or a hostmask." },
+    { "NSMSG_REGISTER_BAD_NICKMASK", "You must provide a hostmask, or online nick to generate one automatically. (or set a default hostmask in the config such as *@*)." },
     { "NSMSG_NICK_NOT_REGISTERED", "Nick $b%s$b has not been registered to any account." },
     { "NSMSG_HANDLE_NOT_FOUND", "Could not find your account -- did you register yet?" },
     { "NSMSG_ALREADY_AUTHED", "You are already authed to account $b%s$b; you must reconnect to auth to a different account." },
@@ -193,7 +192,7 @@ static const struct message_entry msgtab[] = {
     { "NSMSG_STAMPED_AUTH", "You have already authenticated to an account once this session; you may not authenticate to another." },
     { "NSMSG_STAMPED_RESETPASS", "You have already authenticated to an account once this session; you may not reset your password to authenticate again." },
     { "NSMSG_STAMPED_AUTHCOOKIE",  "You have already authenticated to an account once this session; you may not use a cookie to authenticate to another account." },
-    { "NSMSG_TITLE_INVALID", "Titles cannot contain any dots; please choose another." },
+    { "NSMSG_TITLE_INVALID", "Titles may contain only a-z, A-Z, 0-9, and '-'.  Please choose another." },
     { "NSMSG_TITLE_TRUNCATED", "That title combined with the user's account name would result in a truncated host; please choose a shorter title." },
     { "NSMSG_FAKEHOST_INVALID", "Fake hosts must be shorter than %d characters and cannot start with a dot." },
     { "NSMSG_HANDLEINFO_ON", "$bAccount Information for %s$b" },
@@ -365,8 +364,10 @@ static const struct message_entry msgtab[] = {
     { "NSMSG_DENIED_FAKEHOST_WORD", "Access denied because there's a prohibited word in $b%s$b (%s)." },
     { "NSMSG_NOT_VALID_FAKEHOST_LEN", "$b%s$b is not a valid vhost. (can only be 63 characters)" },
     { "NSMSG_NOT_VALID_FAKEHOST_TLD_LEN", "$b%s$b is not a valid vhost. (TLD can only be 4 characters and less)" },
+    { "NSMSG_NOT_VALID_FAKEHOST_REGEX", "$b%s$b is not allowed by the admin, consult the valid vhost regex pattern in the config file under nickserv/valid_fakehost_regex." },
     { "CHECKPASS_YES", "Yes." },
     { "CHECKPASS_NO", "No." },
+    { "NSMSG_DEFCON_NO_NEW_NICKS", "You cannot register new %s at this time, please try again soon" },
     { NULL, NULL }
 };
 
@@ -383,6 +384,7 @@ static struct {
     unsigned int disable_nicks : 1;
     unsigned int valid_handle_regex_set : 1;
     unsigned int valid_nick_regex_set : 1;
+    unsigned int valid_fakehost_regex_set : 1;
     unsigned int autogag_enabled : 1;
     unsigned int email_enabled : 1;
     unsigned int email_required : 1;
@@ -412,6 +414,7 @@ static struct {
     const char *titlehost_suffix;
     regex_t valid_handle_regex;
     regex_t valid_nick_regex;
+    regex_t valid_fakehost_regex;
     dict_t weak_password_dict;
     struct policer_params *auth_policer_params;
     enum reclaim_action reclaim_action;
@@ -456,32 +459,6 @@ register_handle(const char *handle, const char *passwd, UNUSED_ARG(unsigned long
 {
     struct handle_info *hi;
 
-#ifdef WITH_PROTOCOL_BAHAMUT
-    char id_base64[IDLEN + 1];
-    do
-    {
-        /* Assign a unique account ID to the account; note that 0 is
-           an invalid account ID. 1 is therefore the first account ID. */
-        if (!id) {
-            id = 1 + highest_id++;
-        } else {
-            /* Note: highest_id is and must always be the highest ID. */
-            if(id > highest_id) {
-                highest_id = id;
-            }
-        }
-        inttobase64(id_base64, id, IDLEN);
-
-        /* Make sure an account with the same ID doesn't exist. If a
-           duplicate is found, log some details and assign a new one.
-           This should be impossible, but it never hurts to expect it. */
-        if ((hi = dict_find(nickserv_id_dict, id_base64, NULL))) {
-            log_module(NS_LOG, LOG_WARNING, "Duplicated account ID %lu (%s) found belonging to %s while inserting %s.", id, id_base64, hi->handle, handle);
-            id = 0;
-        }
-    } while(!id);
-#endif
-
     hi = calloc(1, sizeof(*hi));
     hi->userlist_style = HI_DEFAULT_STYLE;
     hi->announcements = '?';
@@ -490,11 +467,6 @@ register_handle(const char *handle, const char *passwd, UNUSED_ARG(unsigned long
     hi->infoline = NULL;
     dict_insert(nickserv_handle_dict, hi->handle, hi);
 
-#ifdef WITH_PROTOCOL_BAHAMUT
-    hi->id = id;
-    dict_insert(nickserv_id_dict, strdup(id_base64), hi);
-#endif
-
     return hi;
 }
 
@@ -567,13 +539,6 @@ free_handle_info(void *vhi)
 {
     struct handle_info *hi = vhi;
 
-#ifdef WITH_PROTOCOL_BAHAMUT
-    char id[IDLEN + 1];
-
-    inttobase64(id, hi->id, IDLEN);
-    dict_remove(nickserv_id_dict, id);
-#endif
-
     free_string_list(hi->masks);
     free_string_list(hi->ignores);
     assert(!hi->users);
@@ -793,7 +758,7 @@ smart_get_handle_info(struct userNode *service, struct userNode *user, const cha
 }
 
 int
-oper_outranks(struct userNode *user, struct handle_info *hi) {
+oper_outranks(struct svccmd *cmd, struct userNode *user, struct handle_info *hi) {
     if (user->handle_info->opserv_level > hi->opserv_level)
         return 1;
     if (user->handle_info->opserv_level == hi->opserv_level) {
@@ -805,12 +770,12 @@ oper_outranks(struct userNode *user, struct handle_info *hi) {
             return 1;
         }
     }
-    send_message(user, nickserv, "MSG_USER_OUTRANKED", hi->handle);
+    reply("MSG_USER_OUTRANKED", hi->handle);
     return 0;
 }
 
-static struct handle_info *
-get_victim_oper(struct userNode *user, const char *target)
+struct handle_info *
+get_victim_oper(struct svccmd *cmd, struct userNode *user, const char *target)
 {
     struct handle_info *hi;
     if (!(hi = smart_get_handle_info(nickserv, user, target)))
@@ -819,7 +784,7 @@ get_victim_oper(struct userNode *user, const char *target)
        send_message(user, nickserv, "MSG_OPER_SUSPENDED");
        return 0;
     }
-    return oper_outranks(user, hi) ? hi : NULL;
+    return oper_outranks(cmd, user, hi) ? hi : NULL;
 }
 
 static int
@@ -924,11 +889,28 @@ reg_handle_rename_func(handle_rename_func_t func)
 static char *
 generate_fakehost(struct handle_info *handle)
 {
+    struct userNode *target;
     extern const char *hidden_host_suffix;
     static char buffer[HOSTLEN+1];
+    char *data;
+    int style = 1;
 
     if (!handle->fakehost) {
-        snprintf(buffer, sizeof(buffer), "%s.%s", handle->handle, hidden_host_suffix);
+        data = conf_get_data("server/hidden_host_type", RECDB_QSTRING);
+        if (data)
+            style = atoi(data);
+
+        if (style == 1)
+            snprintf(buffer, sizeof(buffer), "%s.%s", handle->handle, hidden_host_suffix);
+        else if (style == 2) {
+            /* Due to the way fakehost is coded theres no way i can
+               get the exact user, so for now ill just take the first
+               authed user. */
+            for (target = handle->users; target; target = target->next_authed)
+               break;
+
+            snprintf(buffer, sizeof(buffer), "%s", target->crypthost);
+        }
         return buffer;
     } else if (handle->fakehost[0] == '.') {
         /* A leading dot indicates the stored value is actually a title. */
@@ -1002,10 +984,8 @@ set_user_handle_info(struct userNode *user, struct handle_info *hi, int stamp)
     if (hi && !hi->users && !hi->opserv_level)
         HANDLE_CLEAR_FLAG(hi, HELPING);
 
-    if (GetUserH(user->nick)) {
-        for (n=0; n<auth_func_used; n++)
-            auth_func_list[n](user, old_info);
-    } else
+    /* Call auth handlers */
+    if (!GetUserH(user->nick))
       user->loc = 1;
 
     if (hi) {
@@ -1018,26 +998,28 @@ set_user_handle_info(struct userNode *user, struct handle_info *hi, int stamp)
                 send_message(other, nickserv, "NSMSG_CLONE_AUTH", user->nick, user->ident, user->hostname);
         }
 
+        /* Add this auth to users list of current auths */
        user->next_authed = hi->users;
        hi->users = user;
        hi->lastseen = now;
+        /* Add to helpers list */
        if (IsHelper(user))
             userList_append(&curr_helpers, user);
 
+        /* Set the fakehost */
         if (hi->fakehost || old_info)
             apply_fakehost(hi);
 
         if (stamp) {
-#ifdef WITH_PROTOCOL_BAHAMUT
-            /* Stamp users with their account ID. */
-            char id[IDLEN + 1];
-            inttobase64(id, hi->id, IDLEN);
-#elif WITH_PROTOCOL_P10
+#ifdef WITH_PROTOCOL_P10
             /* Stamp users with their account name. */
             char *id = hi->handle;
 #else
             const char *id = "???";
 #endif
+            /* Mark all the nicks registered to this
+             * account as registered nicks 
+             *  -  Why not just this one? -rubin */
             if (!nickserv_conf.disable_nicks) {
                 struct nick_info *ni;
                 for (ni = hi->nicks; ni; ni = ni->next) {
@@ -1047,15 +1029,23 @@ set_user_handle_info(struct userNode *user, struct handle_info *hi, int stamp)
                     }
                 }
             }
+            /* send the account to the ircd */
             StampUser(user, id, hi->registered);
         }
 
+        /* Stop trying to kick this user off their nick */
         if ((ni = get_nick_info(user->nick)) && (ni->owner == hi))
             timeq_del(0, nickserv_reclaim_p, user, TIMEQ_IGNORE_WHEN);
     } else {
         /* We cannot clear the user's account ID, unfortunately. */
        user->next_authed = NULL;
     }
+
+    /* Call auth handlers */
+    if (GetUserH(user->nick)) {
+        for (n=0; n<auth_func_used; n++)
+            auth_func_list[n](user, old_info);
+    }
 }
 
 static struct handle_info*
@@ -1277,6 +1267,11 @@ static NICKSERV_FUNC(cmd_register)
     char syncpass[MD5_CRYPT_LENGTH];
     int no_auth, weblink;
 
+    if (checkDefCon(DEFCON_NO_NEW_NICKS) && !IsOper(user)) {
+        reply("NSMSG_DEFCON_NO_NEW_NICKS", nickserv_conf.disable_nicks ? "accounts" : "nicknames");
+        return 0;
+    }
+
     if (!IsOper(user) && !dict_size(nickserv_handle_dict)) {
        /* Require the first handle registered to belong to someone +o. */
        reply("NSMSG_REQUIRE_OPER");
@@ -1404,75 +1399,96 @@ static NICKSERV_FUNC(cmd_register)
 
 static NICKSERV_FUNC(cmd_oregister)
 {
-    char *mask;
-    struct userNode *settee;
+    struct userNode *settee = NULL;
     struct handle_info *hi;
+    char* account = NULL;
+    char* pass = NULL;
+    char* email = NULL;
+    char* mask = NULL;
+    char* nick = NULL;
 
-    NICKSERV_MIN_PARMS(nickserv_conf.email_required ? 4 : 3);
-
-    if (!is_valid_handle(argv[1])) {
-        reply("NSMSG_BAD_HANDLE", argv[1]);
-        return 0;
-    }
-
+    NICKSERV_MIN_PARMS(3);
+   
+    account = argv[1];
+    pass = argv[2];
     if (nickserv_conf.email_required) {
-        if (!valid_email(argv[4])) {
-            reply("NSMSG_BAD_EMAIL_ADDR");
-            return 0;
+        NICKSERV_MIN_PARMS(4);
+        email = argv[3];
+        if (argc >= 5) {/* take: "acct pass email mask nick" or "acct pass email mask" or "acct pass email nick" */
+            if (strchr(argv[4], '@') || argc >= 6) /* If @, its mask not nick */
+                mask = argv[4];
+            else
+                nick = argv[4];
+        }
+        if (argc >= 6) {
+            nick = argv[5];
         }
     }
-
-    if (strchr(argv[3], '@')) {
-       mask = canonicalize_hostmask(strdup(argv[3]));
-       if (argc > 4) {
-           settee = GetUserH(nickserv_conf.email_required ? argv[5] : argv[4]);
-           if (!settee) {
-               reply("MSG_NICK_UNKNOWN", nickserv_conf.email_required ? argv[5] : argv[4]);
-                free(mask);
-               return 0;
-           }
-       } else {
-           settee = NULL;
-       }
-    } else if ((settee = GetUserH(argv[3]))) {
-       mask = generate_hostmask(settee, GENMASK_OMITNICK|GENMASK_NO_HIDING|GENMASK_ANY_IDENT);
-    } else {
-       reply("NSMSG_REGISTER_BAD_NICKMASK", argv[3]);
-       return 0;
+    else {
+        if (argc >= 4) {/* take: "account pass mask nick" or "account pass mask" or "account pass nick" */
+            if (strchr(argv[3], '@') || argc >= 5) /* If @, its mask not nick */
+                mask = argv[3];
+            else
+                nick = argv[3];
+        }
+        if (argc >= 5) {
+            nick = argv[4];
+        }
     }
+    /* If they passed a nick, look for that user.. */
+    if (nick && !(settee = GetUserH(nick))) {
+        reply("MSG_NICK_UNKNOWN", argv[4]);
+        return 0;
+    }
+    /* If the setee is already authed, we cant add a 2nd account for them.. */
     if (settee && settee->handle_info) {
         reply("NSMSG_USER_PREV_AUTH", settee->nick);
-        free(mask);
         return 0;
     }
-    if (!(hi = nickserv_register(user, settee, argv[1], argv[2], 0))) {
-        if (nickserv_conf.email_required) {
-            nickserv_set_email_addr(hi, argv[4]);
-            if (nickserv_conf.sync_log)
-                SyncLog("REGISTER %s %s %s %s", hi->handle, hi->passwd, argv[4], user->info);
+    /* If there is no default mask in the conf, and they didn't pass a mask, 
+     * but we did find a user by nick, generate the mask */
+    if (!mask) {
+        if (nickserv_conf.default_hostmask)
+            mask = "*@*";
+        else if (settee)
+            mask = generate_hostmask(settee, GENMASK_OMITNICK|GENMASK_NO_HIDING|GENMASK_ANY_IDENT);
+        else {
+            reply("NSMSG_REGISTER_BAD_NICKMASK");
+            return 0;
         }
-        free(mask);
-        return 0;
     }
-    string_list_append(hi->masks, mask);
+
+    if (!(hi = nickserv_register(user, settee, account, pass, 0))) {
+        return 0; /* error reply handled by above */
+    }
+    if (email) {
+        nickserv_set_email_addr(hi, email);
+    }
+    if (mask) {
+        char* mask_canonicalized = canonicalize_hostmask(strdup(mask));
+        string_list_append(hi->masks, mask_canonicalized);
+    }
+
+    if (nickserv_conf.sync_log)
+        SyncLog("REGISTER %s %s %s %s", hi->handle, hi->passwd, email ? email : "@", user->info); /* Send just @ for email if none */
     return 1;
 }
 
 static int
-nickserv_ignore(struct userNode *user, struct handle_info *hi, const char *mask)
+nickserv_ignore(struct svccmd *cmd, struct userNode *user, struct handle_info *hi, char *mask)
 {
     unsigned int i;
     struct userNode *target;
-    char *new_mask = pretty_mask(strdup(mask));
+    char *new_mask = strdup(pretty_mask(mask));
     for (i=0; i<hi->ignores->used; i++) {
         if (!irccasecmp(new_mask, hi->ignores->list[i])) {
-            send_message(user, nickserv, "NSMSG_ADDIGNORE_ALREADY", new_mask);
-/*            free(new_mask); i hate glibc */
+            reply("NSMSG_ADDIGNORE_ALREADY", new_mask);
+            free(new_mask);
             return 0;
         }
     }
     string_list_append(hi->ignores, new_mask);
-    send_message(user, nickserv, "NSMSG_ADDIGNORE_SUCCESS", new_mask);
+    reply("NSMSG_ADDIGNORE_SUCCESS", new_mask);
 
     for (target = hi->users; target; target = target->next_authed) {
         irc_silence(target, new_mask, 1);
@@ -1484,7 +1500,7 @@ static NICKSERV_FUNC(cmd_addignore)
 {
     NICKSERV_MIN_PARMS(2);
 
-    return nickserv_ignore(user, user->handle_info, argv[1]);
+    return nickserv_ignore(cmd, user, user->handle_info, argv[1]);
 }
 
 static NICKSERV_FUNC(cmd_oaddignore)
@@ -1492,47 +1508,48 @@ static NICKSERV_FUNC(cmd_oaddignore)
     struct handle_info *hi;
 
     NICKSERV_MIN_PARMS(3);
-    if (!(hi = get_victim_oper(user, argv[1])))
+    if (!(hi = get_victim_oper(cmd, user, argv[1])))
         return 0;
 
-    return nickserv_ignore(user, hi, argv[2]);
+    return nickserv_ignore(cmd, user, hi, argv[2]);
 }
 
 static int
-nickserv_delignore(struct userNode *user, struct handle_info *hi, const char *del_mask)
+nickserv_delignore(struct svccmd *cmd, struct userNode *user, struct handle_info *hi, char *del_mask)
 {
     unsigned int i;
     struct userNode *target;
-    char *dmask = pretty_mask(strdup(del_mask));
+    char *pmask = strdup(pretty_mask(del_mask));
     for (i=0; i<hi->ignores->used; i++) {
-       if (!strcmp(dmask, hi->ignores->list[i])) {
+       if (!strcmp(pmask, hi->ignores->list[i]) || !strcmp(del_mask, hi->ignores->list[i])) {
            char *old_mask = hi->ignores->list[i];
            hi->ignores->list[i] = hi->ignores->list[--hi->ignores->used];
-           send_message(user, nickserv, "NSMSG_DELMASK_SUCCESS", old_mask);
+           reply("NSMSG_DELMASK_SUCCESS", old_mask);
             for (target = hi->users; target; target = target->next_authed) {
-                irc_silence(user, old_mask, 0);
+                irc_silence(target, old_mask, 0);
             }
-/*         free(old_mask); i hate glibc */
+           free(old_mask);
+            free(pmask);
            return 1;
        }
     }
-    send_message(user, nickserv, "NSMSG_DELMASK_NOT_FOUND");
+    reply("NSMSG_DELMASK_NOT_FOUND");
     return 0;
 }
 
 static NICKSERV_FUNC(cmd_delignore)
 {
     NICKSERV_MIN_PARMS(2);
-    return nickserv_delignore(user, user->handle_info, argv[1]);
+    return nickserv_delignore(cmd, user, user->handle_info, argv[1]);
 }
 
 static NICKSERV_FUNC(cmd_odelignore)
 {
     struct handle_info *hi;
     NICKSERV_MIN_PARMS(3);
-    if (!(hi = get_victim_oper(user, argv[1])))
+    if (!(hi = get_victim_oper(cmd, user, argv[1])))
         return 0;
-    return nickserv_delignore(user, hi, argv[2]);
+    return nickserv_delignore(cmd, user, hi, argv[2]);
 }
 
 static NICKSERV_FUNC(cmd_handleinfo)
@@ -1555,9 +1572,6 @@ static NICKSERV_FUNC(cmd_handleinfo)
     nsmsg_none = handle_find_message(hi, "MSG_NONE");
     reply("NSMSG_HANDLEINFO_ON", hi->handle);
     reply("MSG_BAR");
-#ifdef WITH_PROTOCOL_BAHAMUT
-    reply("NSMSG_HANDLEINFO_ID", hi->id);
-#endif
     reply("NSMSG_HANDLEINFO_REGGED", ctime(&hi->registered));
 
     if (!hi->users) {
@@ -1575,7 +1589,7 @@ static NICKSERV_FUNC(cmd_handleinfo)
         struct do_not_register *dnr;
         if ((dnr = chanserv_is_dnr(NULL, hi)))
             reply("NSMSG_HANDLEINFO_DNR", dnr->setter, dnr->reason);
-        if (!oper_outranks(user, hi))
+        if (!oper_outranks(cmd, user, hi))
             return 1;
     } else if (hi != user->handle_info) {
         reply("NSMSG_HANDLEINFO_END");
@@ -1780,11 +1794,11 @@ static NICKSERV_FUNC(cmd_rename_handle)
 {
     struct handle_info *hi;
     struct userNode *uNode;
-    char msgbuf[MAXLEN], *old_handle;
+    char *old_handle;
     unsigned int nn;
 
     NICKSERV_MIN_PARMS(3);
-    if (!(hi = get_victim_oper(user, argv[1])))
+    if (!(hi = get_victim_oper(cmd, user, argv[1])))
         return 0;
     if (!is_valid_handle(argv[2])) {
         reply("NSMSG_FAIL_RENAME", argv[1], argv[2]);
@@ -1805,8 +1819,6 @@ static NICKSERV_FUNC(cmd_rename_handle)
     dict_insert(nickserv_handle_dict, hi->handle, hi);
     for (nn=0; nn<rf_list_used; nn++)
         rf_list[nn](hi, old_handle);
-    snprintf(msgbuf, sizeof(msgbuf), "%s renamed account %s to %s.", user->handle_info->handle, old_handle, hi->handle);
-
 
     if (nickserv_conf.sync_log) {
         for (uNode = hi->users; uNode; uNode = uNode->next_authed)
@@ -1816,7 +1828,9 @@ static NICKSERV_FUNC(cmd_rename_handle)
     }
 
     reply("NSMSG_HANDLE_CHANGED", old_handle, hi->handle);
-    global_message(MESSAGE_RECIPIENT_STAFF, msgbuf);
+    global_message_args(MESSAGE_RECIPIENT_OPERS, "NSMSG_ACCOUNT_RENAMED",
+                        user->handle_info->handle, old_handle, hi->handle);
+
     free(old_handle);
     return 1;
 }
@@ -2158,7 +2172,7 @@ static NICKSERV_FUNC(cmd_odelcookie)
 
     NICKSERV_MIN_PARMS(2);
 
-    if (!(hi = get_victim_oper(user, argv[1])))
+    if (!(hi = get_victim_oper(cmd, user, argv[1])))
         return 0;
 
     if (!hi->cookie) {
@@ -2166,6 +2180,32 @@ static NICKSERV_FUNC(cmd_odelcookie)
         return 0;
     }
 
+    switch (hi->cookie->type) {
+    case ACTIVATION:
+        safestrncpy(hi->passwd, hi->cookie->data, sizeof(hi->passwd));
+        if (nickserv_conf.sync_log)
+          SyncLog("ACCOUNTACC %s", hi->handle);
+        break;
+    case PASSWORD_CHANGE:
+        safestrncpy(hi->passwd, hi->cookie->data, sizeof(hi->passwd));
+        if (nickserv_conf.sync_log)
+          SyncLog("PASSCHANGE %s %s", hi->handle, hi->passwd);
+        break;
+    case EMAIL_CHANGE:
+        if (!hi->email_addr && nickserv_conf.sync_log) {
+          if (nickserv_conf.sync_log)
+            SyncLog("REGISTER %s %s %s %s", hi->handle, hi->passwd, hi->cookie->data, user->info);
+        }
+        nickserv_set_email_addr(hi, hi->cookie->data);
+        if (nickserv_conf.sync_log)
+          SyncLog("EMAILCHANGE %s %s", hi->handle, hi->cookie->data);
+        break;
+    default:
+        reply("NSMSG_BAD_COOKIE_TYPE", hi->cookie->type);
+        log_module(NS_LOG, LOG_ERROR, "Bad cookie type %d for account %s.", hi->cookie->type, hi->handle);
+        break;
+    }
+
     nickserv_eat_cookie(hi->cookie);
     reply("NSMSG_ATE_FOREIGN_COOKIE", hi->handle);
 
@@ -2366,19 +2406,19 @@ static NICKSERV_FUNC(cmd_pass)
 }
 
 static int
-nickserv_addmask(struct userNode *user, struct handle_info *hi, const char *mask)
+nickserv_addmask(struct svccmd *cmd, struct userNode *user, struct handle_info *hi, const char *mask)
 {
     unsigned int i;
     char *new_mask = canonicalize_hostmask(strdup(mask));
     for (i=0; i<hi->masks->used; i++) {
         if (!irccasecmp(new_mask, hi->masks->list[i])) {
-            send_message(user, nickserv, "NSMSG_ADDMASK_ALREADY", new_mask);
+            reply("NSMSG_ADDMASK_ALREADY", new_mask);
             free(new_mask);
             return 0;
         }
     }
     string_list_append(hi->masks, new_mask);
-    send_message(user, nickserv, "NSMSG_ADDMASK_SUCCESS", new_mask);
+    reply("NSMSG_ADDMASK_SUCCESS", new_mask);
     return 1;
 }
 
@@ -2386,7 +2426,7 @@ static NICKSERV_FUNC(cmd_addmask)
 {
     if (argc < 2) {
         char *mask = generate_hostmask(user, GENMASK_OMITNICK|GENMASK_NO_HIDING|GENMASK_ANY_IDENT);
-        int res = nickserv_addmask(user, user->handle_info, mask);
+        int res = nickserv_addmask(cmd, user, user->handle_info, mask);
         free(mask);
         return res;
     } else {
@@ -2394,7 +2434,7 @@ static NICKSERV_FUNC(cmd_addmask)
             reply("NSMSG_MASK_INVALID", argv[1]);
             return 0;
         }
-        return nickserv_addmask(user, user->handle_info, argv[1]);
+        return nickserv_addmask(cmd, user, user->handle_info, argv[1]);
     }
 }
 
@@ -2403,45 +2443,45 @@ static NICKSERV_FUNC(cmd_oaddmask)
     struct handle_info *hi;
 
     NICKSERV_MIN_PARMS(3);
-    if (!(hi = get_victim_oper(user, argv[1])))
+    if (!(hi = get_victim_oper(cmd, user, argv[1])))
         return 0;
-    return nickserv_addmask(user, hi, argv[2]);
+    return nickserv_addmask(cmd, user, hi, argv[2]);
 }
 
 static int
-nickserv_delmask(struct userNode *user, struct handle_info *hi, const char *del_mask)
+nickserv_delmask(struct svccmd *cmd, struct userNode *user, struct handle_info *hi, const char *del_mask)
 {
     unsigned int i;
     for (i=0; i<hi->masks->used; i++) {
        if (!strcmp(del_mask, hi->masks->list[i])) {
            char *old_mask = hi->masks->list[i];
            if (hi->masks->used == 1) {
-               send_message(user, nickserv, "NSMSG_DELMASK_NOTLAST");
+               reply("NSMSG_DELMASK_NOTLAST");
                return 0;
            }
            hi->masks->list[i] = hi->masks->list[--hi->masks->used];
-           send_message(user, nickserv, "NSMSG_DELMASK_SUCCESS", old_mask);
+           reply("NSMSG_DELMASK_SUCCESS", old_mask);
            free(old_mask);
            return 1;
        }
     }
-    send_message(user, nickserv, "NSMSG_DELMASK_NOT_FOUND");
+    reply("NSMSG_DELMASK_NOT_FOUND");
     return 0;
 }
 
 static NICKSERV_FUNC(cmd_delmask)
 {
     NICKSERV_MIN_PARMS(2);
-    return nickserv_delmask(user, user->handle_info, argv[1]);
+    return nickserv_delmask(cmd, user, user->handle_info, argv[1]);
 }
 
 static NICKSERV_FUNC(cmd_odelmask)
 {
     struct handle_info *hi;
     NICKSERV_MIN_PARMS(3);
-    if (!(hi = get_victim_oper(user, argv[1])))
+    if (!(hi = get_victim_oper(cmd, user, argv[1])))
         return 0;
-    return nickserv_delmask(user, hi, argv[2]);
+    return nickserv_delmask(cmd, user, hi, argv[2]);
 }
 
 int
@@ -2519,7 +2559,7 @@ nickserv_apply_flags(struct userNode *user, struct handle_info *hi, const char *
 }
 
 static void
-set_list(struct userNode *user, struct handle_info *hi, int override)
+set_list(struct svccmd *cmd, struct userNode *user, struct handle_info *hi, int override)
 {
     option_func_t *opt;
     unsigned int i;
@@ -2529,14 +2569,14 @@ set_list(struct userNode *user, struct handle_info *hi, int override)
         "FAKEHOST", "TITLE", "EPITHET", "ADVANCED"
     };
 
-    send_message(user, nickserv, "NSMSG_SETTING_LIST");
-    send_message(user, nickserv, "NSMSG_SETTING_LIST_HEADER");
+    reply("NSMSG_SETTING_LIST");
+    reply("NSMSG_SETTING_LIST_HEADER");
 
     /* Do this so options are presented in a consistent order. */
     for (i = 0; i < ArrayLength(set_display); ++i)
        if ((opt = dict_find(nickserv_opt_dict, set_display[i], NULL)))
-           opt(user, hi, override, 0, NULL);
-    send_message(user, nickserv, "NSMSG_SETTING_LIST_END");
+           opt(cmd, user, hi, override, 0, NULL);
+    reply("NSMSG_SETTING_LIST_END");
 }
 
 static NICKSERV_FUNC(cmd_set)
@@ -2546,14 +2586,14 @@ static NICKSERV_FUNC(cmd_set)
 
     hi = user->handle_info;
     if (argc < 2) {
-       set_list(user, hi, 0);
+       set_list(cmd, user, hi, 0);
        return 1;
     }
     if (!(opt = dict_find(nickserv_opt_dict, argv[1], NULL))) {
        reply("NSMSG_INVALID_OPTION", argv[1]);
         return 0;
     }
-    return opt(user, hi, 0, argc-1, argv+1);
+    return opt(cmd, user, hi, 0, argc-1, argv+1);
 }
 
 static NICKSERV_FUNC(cmd_oset)
@@ -2563,11 +2603,11 @@ static NICKSERV_FUNC(cmd_oset)
 
     NICKSERV_MIN_PARMS(2);
 
-    if (!(hi = get_victim_oper(user, argv[1])))
+    if (!(hi = get_victim_oper(cmd, user, argv[1])))
         return 0;
 
     if (argc < 3) {
-       set_list(user, hi, 0);
+       set_list(cmd, user, hi, 0);
        return 1;
     }
 
@@ -2576,7 +2616,7 @@ static NICKSERV_FUNC(cmd_oset)
         return 0;
     }
 
-    return opt(user, hi, 1, argc-2, argv+2);
+    return opt(cmd, user, hi, 1, argc-2, argv+2);
 }
 
 static OPTION_FUNC(opt_info)
@@ -2592,7 +2632,7 @@ static OPTION_FUNC(opt_info)
     }
 
     info = hi->infoline ? hi->infoline : user_find_message(user, "MSG_NONE");
-    send_message(user, nickserv, "NSMSG_SET_INFO", info);
+    reply("NSMSG_SET_INFO", info);
     return 1;
 }
 
@@ -2606,7 +2646,7 @@ static OPTION_FUNC(opt_width)
     else if (hi->screen_width > MAX_LINE_SIZE)
         hi->screen_width = MAX_LINE_SIZE;
 
-    send_message(user, nickserv, "NSMSG_SET_WIDTH", hi->screen_width);
+    reply("NSMSG_SET_WIDTH", hi->screen_width);
     return 1;
 }
 
@@ -2620,7 +2660,7 @@ static OPTION_FUNC(opt_tablewidth)
     else if (hi->screen_width > MAX_LINE_SIZE)
         hi->table_width = MAX_LINE_SIZE;
 
-    send_message(user, nickserv, "NSMSG_SET_TABLEWIDTH", hi->table_width);
+    reply("NSMSG_SET_TABLEWIDTH", hi->table_width);
     return 1;
 }
 
@@ -2632,12 +2672,12 @@ static OPTION_FUNC(opt_color)
         else if (disabled_string(argv[1]))
            HANDLE_CLEAR_FLAG(hi, MIRC_COLOR);
        else {
-           send_message(user, nickserv, "MSG_INVALID_BINARY", argv[1]);
+           reply("MSG_INVALID_BINARY", argv[1]);
            return 0;
        }
     }
 
-    send_message(user, nickserv, "NSMSG_SET_COLOR", user_find_message(user, HANDLE_FLAGGED(hi, MIRC_COLOR) ? "MSG_ON" : "MSG_OFF"));
+    reply("NSMSG_SET_COLOR", user_find_message(user, HANDLE_FLAGGED(hi, MIRC_COLOR) ? "MSG_ON" : "MSG_OFF"));
     return 1;
 }
 
@@ -2649,12 +2689,12 @@ static OPTION_FUNC(opt_privmsg)
         else if (disabled_string(argv[1]))
            HANDLE_CLEAR_FLAG(hi, USE_PRIVMSG);
        else {
-           send_message(user, nickserv, "MSG_INVALID_BINARY", argv[1]);
+           reply("MSG_INVALID_BINARY", argv[1]);
            return 0;
        }
     }
 
-    send_message(user, nickserv, "NSMSG_SET_PRIVMSG", user_find_message(user, HANDLE_FLAGGED(hi, USE_PRIVMSG) ? "MSG_ON" : "MSG_OFF"));
+    reply("NSMSG_SET_PRIVMSG", user_find_message(user, HANDLE_FLAGGED(hi, USE_PRIVMSG) ? "MSG_ON" : "MSG_OFF"));
     return 1;
 }
 
@@ -2666,12 +2706,12 @@ static OPTION_FUNC(opt_autohide)
         else if (disabled_string(argv[1]))
            HANDLE_CLEAR_FLAG(hi, AUTOHIDE);
        else {
-           send_message(user, nickserv, "MSG_INVALID_BINARY", argv[1]);
+           reply("MSG_INVALID_BINARY", argv[1]);
            return 0;
        }
     }
 
-    send_message(user, nickserv, "NSMSG_SET_AUTOHIDE", user_find_message(user, HANDLE_FLAGGED(hi, AUTOHIDE) ? "MSG_ON" : "MSG_OFF"));
+    reply("NSMSG_SET_AUTOHIDE", user_find_message(user, HANDLE_FLAGGED(hi, AUTOHIDE) ? "MSG_ON" : "MSG_OFF"));
     return 1;
 }
 
@@ -2705,7 +2745,7 @@ static OPTION_FUNC(opt_style)
         style = "Normal";
     }
 
-    send_message(user, nickserv, "NSMSG_SET_STYLE", style);
+    reply("NSMSG_SET_STYLE", style);
     return 1;
 }
 
@@ -2721,7 +2761,7 @@ static OPTION_FUNC(opt_announcements)
         else if (!strcmp(argv[1], "?") || !irccasecmp(argv[1], "default"))
             hi->announcements = '?';
         else {
-            send_message(user, nickserv, "NSMSG_INVALID_ANNOUNCE", argv[1]);
+            reply("NSMSG_INVALID_ANNOUNCE", argv[1]);
             return 0;
         }
     }
@@ -2732,14 +2772,14 @@ static OPTION_FUNC(opt_announcements)
     case '?': choice = "default"; break;
     default: choice = "unknown"; break;
     }
-    send_message(user, nickserv, "NSMSG_SET_ANNOUNCEMENTS", choice);
+    reply("NSMSG_SET_ANNOUNCEMENTS", choice);
     return 1;
 }
 
 static OPTION_FUNC(opt_password)
 {
     if (!override) {
-       send_message(user, nickserv, "NSMSG_USE_CMD_PASS");
+       reply("NSMSG_USE_CMD_PASS");
        return 0;
     }
 
@@ -2749,7 +2789,7 @@ static OPTION_FUNC(opt_password)
     if (nickserv_conf.sync_log)
         SyncLog("PASSCHANGE %s %s", hi->handle, hi->passwd);
 
-    send_message(user, nickserv, "NSMSG_SET_PASSWORD", "***");
+    reply("NSMSG_SET_PASSWORD", "***");
     return 1;
 }
 
@@ -2759,7 +2799,7 @@ static OPTION_FUNC(opt_flags)
     unsigned int ii, flen;
 
     if (!override) {
-       send_message(user, nickserv, "MSG_SETTING_PRIVILEGED", argv[0]);
+       reply("MSG_SETTING_PRIVILEGED", argv[0]);
        return 0;
     }
 
@@ -2771,9 +2811,9 @@ static OPTION_FUNC(opt_flags)
             flags[flen++] = handle_flags[ii];
     flags[flen] = '\0';
     if (hi->flags)
-        send_message(user, nickserv, "NSMSG_SET_FLAGS", flags);
+        reply("NSMSG_SET_FLAGS", flags);
     else
-        send_message(user, nickserv, "NSMSG_SET_FLAGS", user_find_message(user, "MSG_NONE"));
+        reply("NSMSG_SET_FLAGS", user_find_message(user, "MSG_NONE"));
     return 1;
 }
 
@@ -2782,25 +2822,25 @@ static OPTION_FUNC(opt_email)
     if (argc > 1) {
         const char *str;
         if (!valid_email(argv[1])) {
-            send_message(user, nickserv, "NSMSG_BAD_EMAIL_ADDR");
+            reply("NSMSG_BAD_EMAIL_ADDR");
             return 0;
         }
         if ((str = sendmail_prohibited_address(argv[1]))) {
-            send_message(user, nickserv, "NSMSG_EMAIL_PROHIBITED", argv[1], str);
+            reply("NSMSG_EMAIL_PROHIBITED", argv[1], str);
             return 0;
         }
         if (hi->email_addr && !irccasecmp(hi->email_addr, argv[1]))
-            send_message(user, nickserv, "NSMSG_EMAIL_SAME");
+            reply("NSMSG_EMAIL_SAME");
         else if (!override)
                 nickserv_make_cookie(user, hi, EMAIL_CHANGE, argv[1], 0);
         else {
             nickserv_set_email_addr(hi, argv[1]);
             if (hi->cookie)
                 nickserv_eat_cookie(hi->cookie);
-            send_message(user, nickserv, "NSMSG_SET_EMAIL", visible_email_addr(user, hi));
+            reply("NSMSG_SET_EMAIL", visible_email_addr(user, hi));
         }
     } else
-        send_message(user, nickserv, "NSMSG_SET_EMAIL", visible_email_addr(user, hi));
+        reply("NSMSG_SET_EMAIL", visible_email_addr(user, hi));
     return 1;
 }
 
@@ -2810,13 +2850,13 @@ static OPTION_FUNC(opt_maxlogins)
     if (argc > 1) {
         maxlogins = strtoul(argv[1], NULL, 0);
         if ((maxlogins > nickserv_conf.hard_maxlogins) && !override) {
-            send_message(user, nickserv, "NSMSG_BAD_MAX_LOGINS", nickserv_conf.hard_maxlogins);
+            reply("NSMSG_BAD_MAX_LOGINS", nickserv_conf.hard_maxlogins);
             return 0;
         }
         hi->maxlogins = maxlogins;
     }
     maxlogins = hi->maxlogins ? hi->maxlogins : nickserv_conf.default_maxlogins;
-    send_message(user, nickserv, "NSMSG_SET_MAXLOGINS", maxlogins);
+    reply("NSMSG_SET_MAXLOGINS", maxlogins);
     return 1;
 }
 
@@ -2828,12 +2868,12 @@ static OPTION_FUNC(opt_advanced)
         else if (disabled_string(argv[1]))
            HANDLE_CLEAR_FLAG(hi, ADVANCED);
        else {
-           send_message(user, nickserv, "MSG_INVALID_BINARY", argv[1]);
+           reply("MSG_INVALID_BINARY", argv[1]);
            return 0;
        }
     }
 
-    send_message(user, nickserv, "NSMSG_SET_ADVANCED", user_find_message(user, HANDLE_FLAGGED(hi, ADVANCED) ? "MSG_ON" : "MSG_OFF"));
+    reply("NSMSG_SET_ADVANCED", user_find_message(user, HANDLE_FLAGGED(hi, ADVANCED) ? "MSG_ON" : "MSG_OFF"));
     return 1;
 }
 
@@ -2843,10 +2883,10 @@ static OPTION_FUNC(opt_language)
     if (argc > 1) {
         lang = language_find(argv[1]);
         if (irccasecmp(lang->name, argv[1]))
-            send_message(user, nickserv, "NSMSG_LANGUAGE_NOT_FOUND", argv[1], lang->name);
+            reply("NSMSG_LANGUAGE_NOT_FOUND", argv[1], lang->name);
         hi->language = lang;
     }
-    send_message(user, nickserv, "NSMSG_SET_LANGUAGE", hi->language->name);
+    reply("NSMSG_SET_LANGUAGE", hi->language->name);
     return 1;
 }
 
@@ -2883,12 +2923,12 @@ static OPTION_FUNC(opt_level)
     int res;
 
     if (!override) {
-       send_message(user, nickserv, "MSG_SETTING_PRIVILEGED", argv[0]);
+       reply("MSG_SETTING_PRIVILEGED", argv[0]);
        return 0;
     }
 
     res = (argc > 1) ? oper_try_set_access(user, nickserv, hi, strtoul(argv[1], NULL, 0)) : 0;
-    send_message(user, nickserv, "NSMSG_SET_LEVEL", hi->opserv_level);
+    reply("NSMSG_SET_LEVEL", hi->opserv_level);
     return res;
 }
 
@@ -2899,7 +2939,7 @@ static OPTION_FUNC(opt_epithet)
         struct userNode *target, *next_un;
 
         if (!override) {
-            send_message(user, nickserv, "MSG_SETTING_PRIVILEGED", argv[0]);
+            reply("MSG_SETTING_PRIVILEGED", argv[0]);
             return 0;
         }
 
@@ -2920,37 +2960,47 @@ static OPTION_FUNC(opt_epithet)
     }
 
     if (hi->epithet)
-        send_message(user, nickserv, "NSMSG_SET_EPITHET", hi->epithet);
+        reply("NSMSG_SET_EPITHET", hi->epithet);
     else
-        send_message(user, nickserv, "NSMSG_SET_EPITHET", user_find_message(user, "MSG_NONE"));
+        reply("NSMSG_SET_EPITHET", user_find_message(user, "MSG_NONE"));
     return 1;
 }
 
 static OPTION_FUNC(opt_title)
 {
-    const char *title;
+    char *title;
+    const char *none;
+    char *sptr;
 
     if ((argc > 1) && oper_has_access(user, nickserv, nickserv_conf.set_title_level, 0)) {
         if (!override) {
-            send_message(user, nickserv, "MSG_SETTING_PRIVILEGED", argv[0]);
+            reply("MSG_SETTING_PRIVILEGED", argv[0]);
             return 0;
         }
 
         title = argv[1];
-        if (strchr(title, '.')) {
-            send_message(user, nickserv, "NSMSG_TITLE_INVALID");
-            return 0;
-        }
-        if ((strlen(user->handle_info->handle) + strlen(title) +
-             strlen(nickserv_conf.titlehost_suffix) + 2) > HOSTLEN) {
-            send_message(user, nickserv, "NSMSG_TITLE_TRUNCATED");
-            return 0;
-        }
-
-        free(hi->fakehost);
-        if (!strcmp(title, "*")) {
+        if(!strcmp(title, "*")) {
+            free(hi->fakehost);
             hi->fakehost = NULL;
-        } else {
+        }
+        else {
+            if (strchr(title, '.')) {
+                reply("NSMSG_TITLE_INVALID");
+                return 0;
+            }
+            /* Alphanumeric titles only. */
+            for(sptr = title; *sptr; sptr++) {
+                if(!isalnum(*sptr) && *sptr != '-') {
+                    reply("NSMSG_TITLE_INVALID");
+                    return 0;
+                }
+            }
+            if ((strlen(user->handle_info->handle) + strlen(title) +
+                 strlen(nickserv_conf.titlehost_suffix) + 2) > HOSTLEN) {
+                reply("NSMSG_TITLE_TRUNCATED");
+                return 0;
+            }
+            free(hi->fakehost);
             hi->fakehost = malloc(strlen(title)+2);
             hi->fakehost[0] = '.';
             strcpy(hi->fakehost+1, title);
@@ -2958,46 +3008,69 @@ static OPTION_FUNC(opt_title)
         apply_fakehost(hi);
     } else if (hi->fakehost && (hi->fakehost[0] == '.'))
         title = hi->fakehost + 1;
-    else
-        title = NULL;
+    else {
+        /* If theres no title set then the default title will therefore
+           be the first part of hidden_host in x3.conf, so for
+           consistency with opt_fakehost we will print this here.
+           This isnt actually used in P10, its just handled to keep from crashing... */
+        char *hs, *hidden_suffix, *rest;
+
+        hs = conf_get_data("server/hidden_host", RECDB_QSTRING);
+        hidden_suffix = strdup(hs);
+
+        /* Yes we do this twice */
+        if((rest = strchr(hidden_suffix, '.')))
+        {
+            *rest = '\0';
+            title = hidden_suffix;
+        }
+        else
+        {
+            /* A lame default if someone configured hidden_host to something lame */
+            title = strdup("users");
+            free(hidden_suffix);
+        }
+
+    }
+
     if (!title)
-        title = user_find_message(user, "MSG_NONE");
-    send_message(user, nickserv, "NSMSG_SET_TITLE", title);
+        none = user_find_message(user, "MSG_NONE");
+    send_message(user, nickserv, "NSMSG_SET_TITLE", title ? title : none);
     return 1;
 }
 
 int 
-check_vhost(char *vhost, struct userNode *user) 
+check_vhost(char *vhost, struct userNode *user, struct svccmd *cmd
 {
-    unsigned int y, depth;
-    char *hostname;
+    unsigned int y;
 
     // check for a dot in the vhost
     if(strchr(vhost, '.') == NULL) {
-        send_message(user, nickserv, "NSMSG_NOT_VALID_FAKEHOST_DOT", vhost);
+        reply("NSMSG_NOT_VALID_FAKEHOST_DOT", vhost);
         return 0;  
     }
 
     // check for a @ in the vhost
     if(strchr(vhost, '@') != NULL) {
-        send_message(user, nickserv, "NSMSG_NOT_VALID_FAKEHOST_AT", vhost);
+        reply("NSMSG_NOT_VALID_FAKEHOST_AT", vhost);
         return 0;  
     }
 
     // check for denied words, inspired by monk at paki.sex
     for(y = 0; y < nickserv_conf.denied_fakehost_words->used; y++) {
         if(strstr(vhost, nickserv_conf.denied_fakehost_words->list[y]) != NULL) {
-            send_message(user, nickserv, "NSMSG_DENIED_FAKEHOST_WORD", vhost, nickserv_conf.denied_fakehost_words->list[y]);
+            reply("NSMSG_DENIED_FAKEHOST_WORD", vhost, nickserv_conf.denied_fakehost_words->list[y]);
             return 0;
         }
     } 
 
    // check for ircu's HOSTLEN length.
    if(strlen(vhost) >= HOSTLEN) {
-       send_message(user, nickserv, "NSMSG_NOT_VALID_FAKEHOST_LEN", vhost);
+       reply("NSMSG_NOT_VALID_FAKEHOST_LEN", vhost);
        return 0;
    }
 
+   /* This can be handled by the regex now if desired.
    if (vhost[strspn(vhost, "0123456789.")]) {
        hostname = vhost + strlen(vhost);
        for (depth = 1; depth && (hostname > vhost); depth--) {
@@ -3005,13 +3078,28 @@ check_vhost(char *vhost, struct userNode *user)
            while ((hostname > vhost) && (*hostname != '.')) hostname--;
        }
 
-       if (*hostname == '.') hostname++; /* advance past last dot we saw */
+       if (*hostname == '.') hostname++; * advance past last dot we saw *
        if(strlen(hostname) > 4) {
-           send_message(user, nickserv, "NSMSG_NOT_VALID_FAKEHOST_TLD_LEN", vhost);
+           reply("NSMSG_NOT_VALID_FAKEHOST_TLD_LEN", vhost);
+           return 0;
+       }
+   }
+   */
+   /* test either regex or as valid handle */
+   if (nickserv_conf.valid_fakehost_regex_set) {
+       int err = regexec(&nickserv_conf.valid_fakehost_regex, vhost, 0, 0, 0);
+       if (err) {
+           char buff[256];
+           buff[regerror(err, &nickserv_conf.valid_fakehost_regex, buff, sizeof(buff))] = 0;
+           log_module(NS_LOG, LOG_INFO, "regexec error: %s (%d)", buff, err);
+       }
+       if(err == REG_NOMATCH) {
+           reply("NSMSG_NOT_VALID_FAKEHOST_REGEX", vhost);
            return 0;
        }
    }
 
+
    return 1;
 }
 
@@ -3021,39 +3109,46 @@ static OPTION_FUNC(opt_fakehost)
 
     if ((argc > 1) && oper_has_access(user, nickserv, nickserv_conf.set_fakehost_level, 0)) {
         if (!override) {
-            send_message(user, nickserv, "MSG_SETTING_PRIVILEGED", argv[0]);
+            reply("MSG_SETTING_PRIVILEGED", argv[0]);
             return 0;
         }
 
         fake = argv[1];
         if ((strlen(fake) > HOSTLEN) || (fake[0] == '.')) {
-            send_message(user, nickserv, "NSMSG_FAKEHOST_INVALID", HOSTLEN);
+            reply("NSMSG_FAKEHOST_INVALID", HOSTLEN);
             return 0;
         }
-        free(hi->fakehost);
         if (!strcmp(fake, "*")) {
-            hi->fakehost = NULL;
-        } else {
-            if (!check_vhost(argv[1], user)) 
-                return 0;
-
+            if(hi->fakehost) {
+                free(hi->fakehost);
+                hi->fakehost = NULL;
+            }
+        } 
+        else if (!check_vhost(argv[1], user, cmd))  {
+            /* check_vhost takes care of error reply */
+            return 0;
+        }
+        else {
+            if(hi->fakehost)
+                free(hi->fakehost);
             hi->fakehost = strdup(fake);
         }
-        fake = hi->fakehost;
         apply_fakehost(hi);
-    } else {
+        fake = hi->fakehost;
+    } else
         fake = generate_fakehost(hi);
-    }
+
+    /* Tell them we set the host */
     if (!fake)
         fake = user_find_message(user, "MSG_NONE");
-    send_message(user, nickserv, "NSMSG_SET_FAKEHOST", fake);
+    reply("NSMSG_SET_FAKEHOST", fake);
     return 1;
 }
 
 static OPTION_FUNC(opt_note)
 {
     if (!override) {
-        send_message(user, nickserv, "MSG_SETTING_PRIVILEGED", argv[0]);
+        reply("MSG_SETTING_PRIVILEGED", argv[0]);
         return 0;
     }
 
@@ -3071,7 +3166,7 @@ static OPTION_FUNC(opt_note)
         }
     }
 
-    send_message(user, nickserv, "NSMSG_SET_NOTE", hi->note->note);
+    reply("NSMSG_SET_NOTE", hi->note->note);
     return 1;
 }
 
@@ -3175,7 +3270,7 @@ static NICKSERV_FUNC(cmd_ounregister)
     struct handle_info *hi;
 
     NICKSERV_MIN_PARMS(2);
-    if (!(hi = get_victim_oper(user, argv[1])))
+    if (!(hi = get_victim_oper(cmd, user, argv[1])))
         return 0;
     nickserv_unregister_handle(hi, user, cmd->parent->bot);
     return 1;
@@ -3241,9 +3336,6 @@ nickserv_saxdb_write(struct saxdb_context *ctx) {
 
     for (it = dict_first(nickserv_handle_dict); it; it = iter_next(it)) {
         hi = iter_data(it);
-#ifdef WITH_PROTOCOL_BAHAMUT
-        assert(hi->id);
-#endif
         saxdb_start_record(ctx, iter_key(it), 0);
         if (hi->announcements != '?') {
             flags[0] = hi->announcements;
@@ -3294,9 +3386,6 @@ nickserv_saxdb_write(struct saxdb_context *ctx) {
             flags[flen] = 0;
             saxdb_write_string(ctx, KEY_FLAGS, flags);
         }
-#ifdef WITH_PROTOCOL_BAHAMUT
-        saxdb_write_int(ctx, KEY_ID, hi->id);
-#endif
         if (hi->infoline)
             saxdb_write_string(ctx, KEY_INFO, hi->infoline);
         if (hi->last_quit_host[0])
@@ -3361,13 +3450,12 @@ static NICKSERV_FUNC(cmd_merge)
     struct userNode *last_user;
     struct userData *cList, *cListNext;
     unsigned int ii, jj, n;
-    char buffer[MAXLEN];
 
     NICKSERV_MIN_PARMS(3);
 
-    if (!(hi_from = get_victim_oper(user, argv[1])))
+    if (!(hi_from = get_victim_oper(cmd, user, argv[1])))
         return 0;
-    if (!(hi_to = get_victim_oper(user, argv[2])))
+    if (!(hi_to = get_victim_oper(cmd, user, argv[2])))
         return 0;
     if (hi_to == hi_from) {
         reply("NSMSG_CANNOT_MERGE_SELF", hi_to->handle);
@@ -3472,9 +3560,9 @@ static NICKSERV_FUNC(cmd_merge)
         hi_to->fakehost = strdup(hi_from->fakehost);
 
     /* Notify of success. */
-    sprintf(buffer, "%s (%s) merged account %s into %s.", user->nick, user->handle_info->handle, hi_from->handle, hi_to->handle);
     reply("NSMSG_HANDLES_MERGED", hi_from->handle, hi_to->handle);
-    global_message(MESSAGE_RECIPIENT_STAFF, buffer);
+    global_message_args(MESSAGE_RECIPIENT_OPERS, "NSMSG_ACCOUNT_MERGED", user->nick,
+                        user->handle_info->handle, hi_from->handle, hi_to->handle);
 
     /* Unregister the "from" handle. */
     nickserv_unregister_handle(hi_from, NULL, cmd->parent->bot);
@@ -3504,7 +3592,7 @@ struct discrim_apply_info {
 };
 
 static struct nickserv_discrim *
-nickserv_discrim_create(struct userNode *user, unsigned int argc, char *argv[])
+nickserv_discrim_create(struct svccmd *cmd, struct userNode *user, unsigned int argc, char *argv[])
 {
     unsigned int i;
     struct nickserv_discrim *discrim;
@@ -3520,7 +3608,7 @@ nickserv_discrim_create(struct userNode *user, unsigned int argc, char *argv[])
 
     for (i=0; i<argc; i++) {
         if (i == argc - 1) {
-            send_message(user, nickserv, "MSG_MISSING_PARAMS", argv[i]);
+            reply("MSG_MISSING_PARAMS", argv[i]);
             goto fail;
         }
         if (!irccasecmp(argv[i], "limit")) {
@@ -3544,7 +3632,7 @@ nickserv_discrim_create(struct userNode *user, unsigned int argc, char *argv[])
                     discrim->max_registered = now - ParseInterval(cmp+1) - 1;
                 }
             } else {
-                send_message(user, nickserv, "MSG_INVALID_CRITERIA", cmp);
+                reply("MSG_INVALID_CRITERIA", cmp);
             }
         } else if (!irccasecmp(argv[i], "seen")) {
             discrim->lastseen = now - ParseInterval(argv[++i]);
@@ -3554,25 +3642,25 @@ nickserv_discrim_create(struct userNode *user, unsigned int argc, char *argv[])
             i++;
             if (!irccasecmp(argv[i], "exact")) {
                 if (i == argc - 1) {
-                    send_message(user, nickserv, "MSG_MISSING_PARAMS", argv[i]);
+                    reply("MSG_MISSING_PARAMS", argv[i]);
                     goto fail;
                 }
                 discrim->hostmask_type = EXACT;
             } else if (!irccasecmp(argv[i], "subset")) {
                 if (i == argc - 1) {
-                    send_message(user, nickserv, "MSG_MISSING_PARAMS", argv[i]);
+                    reply("MSG_MISSING_PARAMS", argv[i]);
                     goto fail;
                 }
                 discrim->hostmask_type = SUBSET;
             } else if (!irccasecmp(argv[i], "superset")) {
                 if (i == argc - 1) {
-                    send_message(user, nickserv, "MSG_MISSING_PARAMS", argv[i]);
+                    reply("MSG_MISSING_PARAMS", argv[i]);
                     goto fail;
                 }
                 discrim->hostmask_type = SUPERSET;
            } else if (!irccasecmp(argv[i], "lastquit") || !irccasecmp(argv[i], "lastauth")) {
               if (i == argc - 1) {
-                  send_message(user, nickserv, "MSG_MISSING_PARAMS", argv[i]);
+                  reply("MSG_MISSING_PARAMS", argv[i]);
                   goto fail;
               }
               discrim->hostmask_type = LASTQUIT;
@@ -3589,7 +3677,7 @@ nickserv_discrim_create(struct userNode *user, unsigned int argc, char *argv[])
             }
         } else if (!irccasecmp(argv[i], "email")) {
             if (user->handle_info->opserv_level < nickserv_conf.email_search_level) {
-                send_message(user, nickserv, "MSG_NO_SEARCH_ACCESS", "email");
+                reply("MSG_NO_SEARCH_ACCESS", "email");
                 goto fail;
             } else if (!irccasecmp(argv[++i], "*")) {
                 discrim->emailmask = 0;
@@ -3614,10 +3702,10 @@ nickserv_discrim_create(struct userNode *user, unsigned int argc, char *argv[])
                     discrim->min_level = strtoul(cmp+1, NULL, 0) + 1;
                 }
             } else {
-                send_message(user, nickserv, "MSG_INVALID_CRITERIA", cmp);
+                reply("MSG_INVALID_CRITERIA", cmp);
             }
         } else {
-            send_message(user, nickserv, "MSG_INVALID_CRITERIA", argv[i]);
+            reply("MSG_INVALID_CRITERIA", argv[i]);
             goto fail;
         }
     }
@@ -3731,7 +3819,7 @@ nickserv_show_oper_accounts(struct userNode *user, struct svccmd *cmd)
     qsort(hil.list, hil.used, sizeof(hil.list[0]), nickserv_sort_accounts_by_access);
     tbl.length = hil.used + 1;
     tbl.width = 2;
-    tbl.flags = TABLE_NO_FREE;
+    tbl.flags = TABLE_NO_FREE | TABLE_REPEAT_ROWS | TABLE_REPEAT_HEADERS;
     tbl.contents = malloc(tbl.length * sizeof(tbl.contents[0]));
     tbl.contents[0] = ary = malloc(tbl.width * sizeof(ary[0]));
     ary[0] = "Account";
@@ -3743,7 +3831,7 @@ nickserv_show_oper_accounts(struct userNode *user, struct svccmd *cmd)
         tbl.contents[++ii] = ary;
     }
     table_send(cmd->parent->bot, user->nick, 0, NULL, tbl);
-    reply("MSG_MATCH_COUNT", hil.used);
+    /*reply("MSG_MATCH_COUNT", hil.used); */
     for (ii = 0; ii < hil.used; ii++)
         free(tbl.contents[ii]);
     free(tbl.contents);
@@ -3775,7 +3863,7 @@ static NICKSERV_FUNC(cmd_search)
     if (subcmd && !svccmd_can_invoke(user, nickserv, subcmd, NULL, SVCCMD_NOISY))
         return 0;
 
-    discrim = nickserv_discrim_create(user, argc-2, argv+2);
+    discrim = nickserv_discrim_create(cmd, user, argc-2, argv+2);
     if (!discrim)
         return 0;
 
@@ -4090,6 +4178,16 @@ nickserv_conf_read(void)
     } else {
         nickserv_conf.valid_nick_regex_set = 0;
     }
+    str = database_get_data(conf_node, KEY_VALID_FAKEHOST_REGEX, RECDB_QSTRING);
+    if (nickserv_conf.valid_fakehost_regex_set)
+        regfree(&nickserv_conf.valid_fakehost_regex);
+    if (str) {
+        int err = regcomp(&nickserv_conf.valid_fakehost_regex, str, REG_EXTENDED|REG_ICASE|REG_NOSUB);
+        nickserv_conf.valid_fakehost_regex_set = !err;
+        if (err) log_module(NS_LOG, LOG_ERROR, "Bad valid_fakehost_regex (error %d)", err);
+    } else {
+        nickserv_conf.valid_fakehost_regex_set = 0;
+    }
     str = database_get_data(conf_node, KEY_NICKS_PER_HANDLE, RECDB_QSTRING);
     if (!str)
         str = database_get_data(conf_node, KEY_NICKS_PER_ACCOUNT, RECDB_QSTRING);