make some functions a bit shorter.
[apps/madmutt.git] / alias.c
diff --git a/alias.c b/alias.c
index a41c8d8..7282a8c 100644 (file)
--- a/alias.c
+++ b/alias.c
 # include "config.h"
 #endif
 
+#include <string.h>
+#include <ctype.h>
+
+#include <lib-lib/mem.h>
+#include <lib-lib/ascii.h>
+#include <lib-lib/str.h>
+#include <lib-lib/file.h>
+#include <lib-lib/macros.h>
+#include <lib-lib/mapping.h>
+#include <lib-lib/debug.h>
+#include <lib-lib/rx.h>
+
+#include <lib-sys/unix.h>
+
+#include <lib-ui/curses.h>
+#include <lib-ui/enter.h>
+#include <lib-ui/menu.h>
+
 #include "mutt.h"
-#include "mutt_curses.h"
 #include "mutt_idna.h"
+#include "sort.h"
+
+#define RSORT(x) (SortAlias & SORT_REVERSE) ? -x : x
+
+static struct mapping_t AliasHelp[] = {
+    {N_("Exit"), OP_EXIT},
+    {N_("Del"), OP_DELETE},
+    {N_("Undel"), OP_UNDELETE},
+    {N_("Select"), OP_GENERIC_SELECT_ENTRY},
+    {N_("Help"), OP_HELP},
+    {NULL, OP_NULL}
+};
+
+void alias_wipe(alias_t *a) {
+    p_delete(&a->name);
+    address_delete(&a->addr);
+    alias_delete(&a->next);
+}
 
-#include "lib/mem.h"
-#include "lib/intl.h"
-#include "lib/str.h"
-#include "lib/rx.h"
+address_t *alias_lookup(alias_t *list, const char *s)
+{
+    while (list) {
+        if (!m_strcasecmp(s, list->name))
+            return list->addr;
+        list = list->next;
+    }
 
-#include <string.h>
-#include <ctype.h>
+    return NULL;
+}
+
+static int string_is_address(const char *str, const char *u, const char *d)
+{
+    char buf[LONG_STRING];
+    snprintf(buf, sizeof (buf), "%s@%s", NONULL(u), NONULL(d));
+    return !ascii_strcasecmp(str, buf);
+}
+
+/* returns TRUE if the given address belongs to the user. */
+int mutt_addr_is_user(address_t *addr)
+{
+    /* NULL address is assumed to be the user. */
+    if (!addr)
+        return 1;
+
+    if (!addr->mailbox)
+        return 0;
+
+    if (!ascii_strcasecmp(addr->mailbox, Username)
+    ||  string_is_address(addr->mailbox, Username, Hostname)
+    ||  string_is_address(addr->mailbox, Username, mutt_fqdn(0))
+    ||  string_is_address(addr->mailbox, Username, mutt_fqdn(1))
+    ||  (From && !ascii_strcasecmp(From->mailbox, addr->mailbox)))
+    {
+        return 1;
+    }
+
+    return rx_list_match(Alternates, addr->mailbox)
+        && !rx_list_match(UnAlternates, addr->mailbox);
+}
+
+address_t *mutt_get_address(ENVELOPE *env, const char **pfxp)
+{
+#define RETURN(s, adr)  do { if (pfxp) *pfxp = s; return adr; } while (0)
+
+    if (mutt_addr_is_user(env->from)) {
+        if (env->to && !mutt_is_mail_list(env->to)) {
+            RETURN("To", env->to);
+        } else {
+            RETURN("Cc", env->cc);
+        }
+    } else {
+        if (env->reply_to && !mutt_is_mail_list(env->reply_to)) {
+            RETURN("Reply-To", env->reply_to);
+        } else {
+            RETURN("From", env->from);
+        }
+    }
+
+#undef RETURN
+}
+
+/* Only characters which are non-special to both the RFC 822 and the mutt
+   configuration parser are permitted.                                      */
+static int alias_sanitize(const char *s, char *d)
+{
+    int rv = 0;
+
+    while (*s) {
+        if (isalnum((unsigned char)(*s)) || strchr("-_+=.", *s)) {
+            if (d)
+                *d++ = *s;
+        } else {
+            if (!d)
+                return -1;
+            *d++ = '_';
+            rv = -1;
+        }
+        s++;
+    }
+
+    *d = '\0';
+    return rv;
+}
+
+/* 
+ * if someone has an address like
+ *     From: Michael `/bin/rm -f ~` Elkins <me@mutt.org>
+ * and the user creates an alias for this, Mutt could wind up executing
+ * the backtics because it writes aliases like
+ *     alias me Michael `/bin/rm -f ~` Elkins <me@mutt.org>
+ * To avoid this problem, use a backslash (\) to quote any backtics.  We also
+ * need to quote backslashes as well, since you could defeat the above by
+ * doing
+ *     From: Michael \`/bin/rm -f ~\` Elkins <me@mutt.org>
+ * since that would get aliased as
+ *     alias me Michael \\`/bin/rm -f ~\\` Elkins <me@mutt.org>
+ * which still gets evaluated because the double backslash is not a quote.
+ * 
+ * Additionally, we need to quote ' and " characters - otherwise, mutt will
+ * interpret them on the wrong parsing step.
+ * 
+ * $ wants to be quoted since it may indicate the start of an environment
+ * variable.
+ */
+static void write_safe_address(FILE *fp, const char *s)
+{
+    while (*s) {
+        if (strchr("\\`'\"$", *s)) {
+            fputc('\\', fp);
+        }
+        fputc(*s++, fp);
+    }
+    fputc('\n', fp);
+}
 
-ADDRESS *mutt_lookup_alias (const char *s)
+void mutt_create_alias(ENVELOPE *cur, address_t *iadr)
 {
-  ALIAS *t = Aliases;
+    char buf[LONG_STRING], prompt[SHORT_STRING];
+    address_t *adr = iadr;
+    alias_t *new;
+    FILE *rc;
+
+    if (cur) {
+        adr = mutt_get_address(cur, NULL);
+    }
+
+    if (adr && adr->mailbox) {
+        const char *p = m_strchrnul(adr->mailbox, '@');
+        m_strncpy(buf, sizeof(buf), adr->mailbox, p - adr->mailbox);
+    } else {
+        buf[0] = '\0';
+    }
+
+    /* Don't suggest a bad alias name in the event of a strange local part. */
+    alias_sanitize(buf, buf);
+
+    /* add a new alias */
+    if (mutt_get_field(_("Alias as: "), buf, sizeof(buf), 0) || !buf[0])
+        return;
+
+    /* check to see if the user already has an alias defined */
+    if (alias_lookup(Aliases, buf)) {
+        mutt_error _("You already have an alias defined with that name!");
+        return;
+    }
+
+    alias_sanitize(buf, buf);
+    new = alias_new();
+    new->name = m_strdup(buf);
+
+    mutt_addrlist_to_local(adr);
+    if (adr) {
+        m_strcpy(buf, sizeof(buf), adr->mailbox);
+    } else {
+        buf[0] = 0;
+    }
+
+    mutt_addrlist_to_idna(adr, NULL);
+
+    do {
+        const char *err = NULL;
+
+        if (mutt_get_field(_("Address: "), buf, sizeof(buf), 0) || !buf[0]) {
+            alias_delete(&new);
+            return;
+        }
+
+        new->addr = rfc822_parse_adrlist(new->addr, buf);
+        if (!new->addr)
+            BEEP();
+
+        if (mutt_addrlist_to_idna(new->addr, &err)) {
+            mutt_error(_("Error: '%s' is a bad IDN."), err);
+            mutt_sleep(1);
+            continue;
+        }
+    } while (!new->addr);
+
+    if (adr && adr->personal && !mutt_is_mail_list(adr)) {
+        m_strcpy(buf, sizeof(buf), adr->personal);
+    } else {
+        buf[0] = '\0';
+    }
+
+    if (mutt_get_field(_("Personal name: "), buf, sizeof(buf), 0)) {
+        alias_delete(&new);
+        return;
+    }
+    new->addr->personal = m_strdup(buf);
+
+    buf[0] = '\0';
+    rfc822_write_address(buf, sizeof(buf), new->addr, 1);
+    snprintf(prompt, sizeof(prompt), _("[%s = %s] Accept?"), new->name, buf);
+    if (mutt_yesorno(prompt, M_YES) != M_YES) {
+        alias_delete(&new);
+        return;
+    }
 
-  for (; t; t = t->next)
-    if (!safe_strcasecmp (s, t->name))
-      return (t->addr);
-  return (NULL);                /* no such alias */
+    alias_list_push(&Aliases, new);
+
+    m_strcpy(buf, sizeof(buf), NONULL(AliasFile));
+    if (mutt_get_field(_("Save to file: "), buf, sizeof(buf), M_FILE)) {
+        return;
+    }
+
+    mutt_expand_path(buf, sizeof (buf));
+    rc = safe_fopen (buf, "a");
+
+    if (rc) {
+        if (alias_sanitize(new->name, NULL)) {
+            mutt_quote_filename(buf, sizeof(buf), new->name);
+            fprintf(rc, "alias %s ", buf);
+        } else {
+            fprintf(rc, "alias %s ", new->name);
+        }
+
+        buf[0] = '\0';
+        rfc822_write_address(buf, sizeof(buf), new->addr, 0);
+        write_safe_address(rc, buf);
+        fclose(rc);
+        mutt_message _("Alias added.");
+    } else {
+        mutt_perror(buf);
+    }
 }
 
-static ADDRESS *mutt_expand_aliases_r (ADDRESS * a, LIST ** expn)
+/************* READ MARK *********************/
+
+static address_t *mutt_expand_aliases_r (address_t * a, LIST ** expn)
 {
-  ADDRESS *head = NULL, *last = NULL, *t, *w;
+  address_t *head = NULL, *last = NULL, *t, *w;
   LIST *u;
   char i;
   const char *fqdn;
@@ -43,27 +289,24 @@ static ADDRESS *mutt_expand_aliases_r (ADDRESS * a, LIST ** expn)
   while (a) {
     if (!a->group && !a->personal && a->mailbox
         && strchr (a->mailbox, '@') == NULL) {
-      t = mutt_lookup_alias (a->mailbox);
+      t = alias_lookup(Aliases, a->mailbox);
 
       if (t) {
         i = 0;
         for (u = *expn; u; u = u->next) {
-          if (safe_strcmp (a->mailbox, u->data) == 0) { /* alias already found */
-            dprint (1,
-                    (debugfile,
-                     "mutt_expand_aliases_r(): loop in alias found for '%s'\n",
-                     a->mailbox));
+          if (m_strcmp(a->mailbox, u->data) == 0) { /* alias already found */
+            debug_print(1, ("loop in alias found for '%s'\n", a->mailbox));
             i = 1;
             break;
           }
         }
 
         if (!i) {
-          u = safe_malloc (sizeof (LIST));
-          u->data = safe_strdup (a->mailbox);
+          u = p_new(LIST, 1);
+          u->data = m_strdup(a->mailbox);
           u->next = *expn;
           *expn = u;
-          w = rfc822_cpy_adr (t);
+          w = address_list_dup (t);
           w = mutt_expand_aliases_r (w, expn);
           if (head)
             last->next = w;
@@ -75,7 +318,7 @@ static ADDRESS *mutt_expand_aliases_r (ADDRESS * a, LIST ** expn)
         t = a;
         a = a->next;
         t->next = NULL;
-        rfc822_free_address (&t);
+        address_delete (&t);
         continue;
       }
       else {
@@ -84,12 +327,8 @@ static ADDRESS *mutt_expand_aliases_r (ADDRESS * a, LIST ** expn)
         if (pw) {
           char namebuf[STRING];
 
-          mutt_gecos_name (namebuf, sizeof (namebuf), pw);
-          str_replace (&a->personal, namebuf);
-
-#ifdef EXACT_ADDRESS
-          FREE (&a->val);
-#endif
+          mutt_gecos_name(namebuf, sizeof (namebuf), pw, GecosMask.rx);
+          m_strreplace(&a->personal, namebuf);
         }
       }
     }
@@ -112,9 +351,9 @@ static ADDRESS *mutt_expand_aliases_r (ADDRESS * a, LIST ** expn)
   return (head);
 }
 
-ADDRESS *mutt_expand_aliases (ADDRESS * a)
+address_t *mutt_expand_aliases (address_t * a)
 {
-  ADDRESS *t;
+  address_t *t;
   LIST *expn = NULL;            /* previously expanded aliases to avoid loops */
 
   t = mutt_expand_aliases_r (a, &expn);
@@ -133,234 +372,14 @@ void mutt_expand_aliases_env (ENVELOPE * env)
 }
 
 
-/* 
- * if someone has an address like
- *     From: Michael `/bin/rm -f ~` Elkins <me@mutt.org>
- * and the user creates an alias for this, Mutt could wind up executing
- * the backtics because it writes aliases like
- *     alias me Michael `/bin/rm -f ~` Elkins <me@mutt.org>
- * To avoid this problem, use a backslash (\) to quote any backtics.  We also
- * need to quote backslashes as well, since you could defeat the above by
- * doing
- *     From: Michael \`/bin/rm -f ~\` Elkins <me@mutt.org>
- * since that would get aliased as
- *     alias me Michael \\`/bin/rm -f ~\\` Elkins <me@mutt.org>
- * which still gets evaluated because the double backslash is not a quote.
- * 
- * Additionally, we need to quote ' and " characters - otherwise, mutt will
- * interpret them on the wrong parsing step.
- * 
- * $ wants to be quoted since it may indicate the start of an environment
- * variable.
- */
-
-static void write_safe_address (FILE * fp, char *s)
-{
-  while (*s) {
-    if (*s == '\\' || *s == '`' || *s == '\'' || *s == '"' || *s == '$')
-      fputc ('\\', fp);
-    fputc (*s, fp);
-    s++;
-  }
-}
-
-ADDRESS *mutt_get_address (ENVELOPE * env, char **pfxp)
-{
-  ADDRESS *adr;
-  char *pfx = NULL;
-
-  if (mutt_addr_is_user (env->from)) {
-    if (env->to && !mutt_is_mail_list (env->to)) {
-      pfx = "To";
-      adr = env->to;
-    }
-    else {
-      pfx = "Cc";
-      adr = env->cc;
-    }
-  }
-  else if (env->reply_to && !mutt_is_mail_list (env->reply_to)) {
-    pfx = "Reply-To";
-    adr = env->reply_to;
-  }
-  else {
-    adr = env->from;
-    pfx = "From";
-  }
-
-  if (pfxp)
-    *pfxp = pfx;
-
-  return adr;
-}
-
-void mutt_create_alias (ENVELOPE * cur, ADDRESS * iadr)
-{
-  ALIAS *new, *t;
-  char buf[LONG_STRING], prompt[SHORT_STRING], *pc;
-  char *err = NULL;
-  char fixed[LONG_STRING];
-  FILE *rc;
-  ADDRESS *adr = NULL;
-
-  if (cur) {
-    adr = mutt_get_address (cur, NULL);
-  }
-  else if (iadr) {
-    adr = iadr;
-  }
-
-  if (adr && adr->mailbox) {
-    strfcpy (buf, adr->mailbox, sizeof (buf));
-    if ((pc = strchr (buf, '@')))
-      *pc = 0;
-  }
-  else
-    buf[0] = '\0';
-
-  /* Don't suggest a bad alias name in the event of a strange local part. */
-  mutt_check_alias_name (buf, buf);
-
-retry_name:
-  /* add a new alias */
-  if (mutt_get_field (_("Alias as: "), buf, sizeof (buf), 0) != 0 || !buf[0])
-    return;
-
-  /* check to see if the user already has an alias defined */
-  if (mutt_lookup_alias (buf)) {
-    mutt_error _("You already have an alias defined with that name!");
-
-    return;
-  }
-
-  if (mutt_check_alias_name (buf, fixed)) {
-    switch (mutt_yesorno
-            (_("Warning: This alias name may not work.  Fix it?"), M_YES)) {
-    case M_YES:
-      strfcpy (buf, fixed, sizeof (buf));
-      goto retry_name;
-    case -1:
-      return;
-    }
-  }
-
-  new = safe_calloc (1, sizeof (ALIAS));
-  new->self = new;
-  new->name = safe_strdup (buf);
-
-  mutt_addrlist_to_local (adr);
-
-  if (adr)
-    strfcpy (buf, adr->mailbox, sizeof (buf));
-  else
-    buf[0] = 0;
-
-  mutt_addrlist_to_idna (adr, NULL);
-
-  do {
-    if (mutt_get_field (_("Address: "), buf, sizeof (buf), 0) != 0 || !buf[0]) {
-      mutt_free_alias (&new);
-      return;
-    }
-
-    if ((new->addr = rfc822_parse_adrlist (new->addr, buf)) == NULL)
-      BEEP ();
-    if (mutt_addrlist_to_idna (new->addr, &err)) {
-      mutt_error (_("Error: '%s' is a bad IDN."), err);
-      mutt_sleep (2);
-      continue;
-    }
-  }
-  while (new->addr == NULL);
-
-  if (adr && adr->personal && !mutt_is_mail_list (adr))
-    strfcpy (buf, adr->personal, sizeof (buf));
-  else
-    buf[0] = 0;
-
-  if (mutt_get_field (_("Personal name: "), buf, sizeof (buf), 0) != 0) {
-    mutt_free_alias (&new);
-    return;
-  }
-  new->addr->personal = safe_strdup (buf);
-
-  buf[0] = 0;
-  rfc822_write_address (buf, sizeof (buf), new->addr, 1);
-  snprintf (prompt, sizeof (prompt), _("[%s = %s] Accept?"), new->name, buf);
-  if (mutt_yesorno (prompt, M_YES) != M_YES) {
-    mutt_free_alias (&new);
-    return;
-  }
-
-  if ((t = Aliases)) {
-    while (t->next)
-      t = t->next;
-    t->next = new;
-  }
-  else
-    Aliases = new;
-
-  strfcpy (buf, NONULL (AliasFile), sizeof (buf));
-  if (mutt_get_field (_("Save to file: "), buf, sizeof (buf), M_FILE) != 0)
-    return;
-  mutt_expand_path (buf, sizeof (buf));
-  if ((rc = safe_fopen (buf, "a"))) {
-    if (mutt_check_alias_name (new->name, NULL))
-      mutt_quote_filename (buf, sizeof (buf), new->name);
-    else
-      strfcpy (buf, new->name, sizeof (buf));
-    fprintf (rc, "alias %s ", buf);
-    buf[0] = 0;
-    rfc822_write_address (buf, sizeof (buf), new->addr, 0);
-    write_safe_address (rc, buf);
-    fputc ('\n', rc);
-    fclose (rc);
-    mutt_message _("Alias added.");
-  }
-  else
-    mutt_perror (buf);
-}
-
-/* 
- * Sanity-check an alias name:  Only characters which are non-special to both
- * the RFC 822 and the mutt configuration parser are permitted.
- */
-
-static int check_alias_name_char (char c)
-{
-  return (c == '-' || c == '_' || c == '+' || c == '=' || c == '.' ||
-          isalnum ((unsigned char) c));
-}
-
-int mutt_check_alias_name (const char *s, char *d)
-{
-  int rv = 0;
-
-  for (; *s; s++) {
-    if (!check_alias_name_char (*s)) {
-      if (!d)
-        return -1;
-      else {
-        *d++ = '_';
-        rv = -1;
-      }
-    }
-    else if (d)
-      *d++ = *s;
-  }
-  if (d)
-    *d++ = *s;
-  return rv;
-}
-
 /*
  * This routine looks to see if the user has an alias defined for the given
  * address.
  */
-ADDRESS *alias_reverse_lookup (ADDRESS * a)
+address_t *alias_reverse_lookup (address_t * a)
 {
-  ALIAS *t = Aliases;
-  ADDRESS *ap;
+  alias_t *t = Aliases;
+  address_t *ap;
 
   if (!a || !a->mailbox)
     return NULL;
@@ -384,21 +403,21 @@ ADDRESS *alias_reverse_lookup (ADDRESS * a)
  */
 int mutt_alias_complete (char *s, size_t buflen)
 {
-  ALIAS *a = Aliases;
-  ALIAS *a_list = NULL, *a_cur = NULL;
+  alias_t *a = Aliases;
+  alias_t *a_list = NULL, *a_cur = NULL;
   char bestname[HUGE_STRING];
   int i;
 
 #define min(a,b)        ((a<b)?a:b)
 
   if (s[0] != 0) {              /* avoid empty string as strstr argument */
-    memset (bestname, 0, sizeof (bestname));
+    p_clear(bestname, countof(bestname));
 
     while (a) {
       if (a->name && strstr (a->name, s) == a->name) {
         if (!bestname[0])       /* init */
-          strfcpy (bestname, a->name,
-                   min (safe_strlen (a->name) + 1, sizeof (bestname)));
+          m_strcpy(bestname, MIN(m_strlen(a->name) + 1, ssizeof(bestname)),
+                   a->name);
         else {
           for (i = 0; a->name[i] && a->name[i] == bestname[i]; i++);
           bestname[i] = 0;
@@ -408,9 +427,9 @@ int mutt_alias_complete (char *s, size_t buflen)
     }
 
     if (bestname[0] != 0) {
-      if (safe_strcmp (bestname, s) != 0) {
+      if (m_strcmp(bestname, s) != 0) {
         /* we are adding something to the completion */
-        strfcpy (s, bestname, safe_strlen (bestname) + 1);
+        m_strcpy(s, m_strlen(bestname) + 1, bestname);
         return 1;
       }
 
@@ -420,12 +439,12 @@ int mutt_alias_complete (char *s, size_t buflen)
       while (a) {
         if (a->name && (strstr (a->name, s) == a->name)) {
           if (!a_list)          /* init */
-            a_cur = a_list = (ALIAS *) safe_malloc (sizeof (ALIAS));
+            a_cur = a_list = alias_new();
           else {
-            a_cur->next = (ALIAS *) safe_malloc (sizeof (ALIAS));
+            a_cur->next = alias_new();
             a_cur = a_cur->next;
           }
-          memcpy (a_cur, a, sizeof (ALIAS));
+          *a_cur = *a;
           a_cur->next = NULL;
         }
         a = a->next;
@@ -436,13 +455,13 @@ int mutt_alias_complete (char *s, size_t buflen)
   bestname[0] = 0;
   mutt_alias_menu (bestname, sizeof (bestname), a_list ? a_list : Aliases);
   if (bestname[0] != 0)
-    strfcpy (s, bestname, buflen);
+    m_strcpy(s, buflen, bestname);
 
   /* free the alias list */
   while (a_list) {
     a_cur = a_list;
     a_list = a_list->next;
-    FREE (&a_cur);
+    p_delete(&a_cur);
   }
 
   /* remove any aliases marked for deletion */
@@ -455,7 +474,7 @@ int mutt_alias_complete (char *s, size_t buflen)
         Aliases = a_cur->next;
 
       a_cur->next = NULL;
-      mutt_free_alias (&a_cur);
+      alias_delete(&a_cur);
 
       if (a_list)
         a_cur = a_list;
@@ -471,75 +490,195 @@ int mutt_alias_complete (char *s, size_t buflen)
   return 0;
 }
 
-static int string_is_address (const char *str, const char *u, const char *d)
+static const format_t *alias_format_str (char *dest, size_t destlen, char op,
+                                     const format_t *src, const char *fmt,
+                                     const char *ifstring __attribute__ ((unused)),
+                                     const char *elsestring __attribute__ ((unused)),
+                                     unsigned long data, format_flag flags __attribute__ ((unused)))
 {
-  char buf[LONG_STRING];
+  char tmp[SHORT_STRING], adr[SHORT_STRING];
+  alias_t *alias = (alias_t *) data;
+
+  switch (op) {
+  case 'f':
+    m_strcpy(dest, destlen, alias->del ? "D" : " ");
+    break;
+  case 'a':
+    mutt_format_s(dest, destlen, fmt, alias->name);
+    break;
+  case 'r':
+    adr[0] = '\0';
+    rfc822_write_address(adr, sizeof(adr), alias->addr, 1);
+    snprintf(tmp, sizeof(tmp), "%%%ss", fmt);
+    snprintf(dest, destlen, tmp, adr);
+    break;
+  case 'n':
+    snprintf(tmp, sizeof(tmp), "%%%sd", fmt);
+    snprintf(dest, destlen, tmp, alias->num + 1);
+    break;
+  case 't':
+    m_strcpy(dest, destlen, alias->tagged ? "*" : " ");
+    break;
+  }
 
-  snprintf (buf, sizeof (buf), "%s@%s", NONULL (u), NONULL (d));
-  if (ascii_strcasecmp (str, buf) == 0)
-    return 1;
+  return (src);
+}
 
-  return 0;
+static void alias_entry (char *s, ssize_t slen, MUTTMENU * m, int num)
+{
+  mutt_FormatString (s, slen, NONULL (AliasFmt), (format_t *)alias_format_str,
+                     (unsigned long)((alias_t **)m->data)[num],
+                     M_FORMAT_ARROWCURSOR);
 }
 
-/* returns TRUE if the given address belongs to the user. */
-int mutt_addr_is_user (ADDRESS * addr)
+static int alias_tag (MUTTMENU * menu, int n, int m)
+{
+  alias_t *cur = ((alias_t **) menu->data)[n];
+  int ot = cur->tagged;
+
+  cur->tagged = (m >= 0 ? m : !cur->tagged);
+
+  return cur->tagged - ot;
+}
+
+static int alias_SortAlias (const void *a, const void *b)
+{
+  alias_t *pa = *(alias_t **) a;
+  alias_t *pb = *(alias_t **) b;
+  int r = m_strcasecmp(pa->name, pb->name);
+
+  return (RSORT (r));
+}
+
+static int alias_SortAddress (const void *a, const void *b)
 {
-  /* NULL address is assumed to be the user. */
-  if (!addr) {
-    dprint (5, (debugfile, "mail_addr_is_user: yes, NULL address\n"));
-    return 1;
+  address_t *pa = (*(alias_t **) a)->addr;
+  address_t *pb = (*(alias_t **) b)->addr;
+  int r;
+
+  if (pa == pb)
+    r = 0;
+  else if (pa == NULL)
+    r = -1;
+  else if (pb == NULL)
+    r = 1;
+  else if (pa->personal) {
+    if (pb->personal)
+      r = m_strcasecmp(pa->personal, pb->personal);
+    else
+      r = 1;
   }
-  if (!addr->mailbox) {
-    dprint (5, (debugfile, "mail_addr_is_user: no, no mailbox\n"));
-    return 0;
+  else if (pb->personal)
+    r = -1;
+  else
+    r = ascii_strcasecmp (pa->mailbox, pb->mailbox);
+  return (RSORT (r));
+}
+
+void mutt_alias_menu (char *buf, size_t buflen, alias_t * aliases)
+{
+  alias_t *aliasp;
+  MUTTMENU *menu;
+  alias_t **AliasTable = NULL;
+  int t = -1;
+  int i, done = 0;
+  int op;
+  char helpstr[SHORT_STRING];
+
+  int omax;
+
+  if (!aliases) {
+    mutt_error _("You have no aliases!");
+
+    return;
   }
 
-  if (ascii_strcasecmp (addr->mailbox, Username) == 0) {
-    dprint (5,
-            (debugfile, "mail_addr_is_user: yes, %s = %s\n", addr->mailbox,
-             Username));
-    return 1;
+  /* tell whoever called me to redraw the screen when I return */
+  set_option (OPTNEEDREDRAW);
+
+  menu = mutt_new_menu ();
+  menu->make_entry = alias_entry;
+  menu->tag = alias_tag;
+  menu->menu = MENU_ALIAS;
+  menu->title = _("Aliases");
+  menu->help = mutt_compile_help(helpstr, sizeof(helpstr),
+                                 MENU_ALIAS, AliasHelp);
+
+new_aliases:
+
+  omax = menu->max;
+
+  /* count the number of aliases */
+  for (aliasp = aliases; aliasp; aliasp = aliasp->next) {
+    aliasp->del = 0;
+    aliasp->tagged = 0;
+    menu->max++;
   }
-  if (string_is_address (addr->mailbox, Username, Hostname)) {
-    dprint (5,
-            (debugfile, "mail_addr_is_user: yes, %s = %s @ %s \n",
-             addr->mailbox, Username, Hostname));
-    return 1;
+
+  p_realloc(&AliasTable, menu->max);
+  menu->data = AliasTable;
+
+  for (i = omax, aliasp = aliases; aliasp; aliasp = aliasp->next, i++) {
+    AliasTable[i] = aliasp;
+    aliases = aliasp;
   }
-  if (string_is_address (addr->mailbox, Username, mutt_fqdn (0))) {
-    dprint (5,
-            (debugfile, "mail_addr_is_user: yes, %s = %s @ %s \n",
-             addr->mailbox, Username, mutt_fqdn (0)));
-    return 1;
+
+  if ((SortAlias & SORT_MASK) != SORT_ORDER) {
+    qsort (AliasTable, i, sizeof (alias_t *),
+           (SortAlias & SORT_MASK) ==
+           SORT_ADDRESS ? alias_SortAddress : alias_SortAlias);
   }
-  if (string_is_address (addr->mailbox, Username, mutt_fqdn (1))) {
-    dprint (5,
-            (debugfile, "mail_addr_is_user: yes, %s = %s @ %s \n",
-             addr->mailbox, Username, mutt_fqdn (1)));
-    return 1;
+
+  for (i = 0; i < menu->max; i++)
+    AliasTable[i]->num = i;
+
+  while (!done) {
+    if (aliases->next) {
+      menu->redraw |= REDRAW_FULL;
+      aliases = aliases->next;
+      goto new_aliases;
+    }
+
+    switch ((op = mutt_menuLoop (menu))) {
+    case OP_DELETE:
+    case OP_UNDELETE:
+      if (menu->tagprefix) {
+        for (i = 0; i < menu->max; i++)
+          if (AliasTable[i]->tagged)
+            AliasTable[i]->del = (op == OP_DELETE) ? 1 : 0;
+        menu->redraw |= REDRAW_INDEX;
+      }
+      else {
+        AliasTable[menu->current]->del = (op == OP_DELETE) ? 1 : 0;
+        menu->redraw |= REDRAW_CURRENT;
+        if (option (OPTRESOLVE) && menu->current < menu->max - 1) {
+          menu->current++;
+          menu->redraw |= REDRAW_INDEX;
+        }
+      }
+      break;
+    case OP_GENERIC_SELECT_ENTRY:
+      t = menu->current;
+    case OP_EXIT:
+      done = 1;
+      break;
+    }
   }
 
-  if (From && !ascii_strcasecmp (From->mailbox, addr->mailbox)) {
-    dprint (5,
-            (debugfile, "mail_addr_is_user: yes, %s = %s\n", addr->mailbox,
-             From->mailbox));
-    return 1;
+  for (i = 0; i < menu->max; i++) {
+    if (AliasTable[i]->tagged) {
+      mutt_addrlist_to_local (AliasTable[i]->addr);
+      rfc822_write_address (buf, buflen, AliasTable[i]->addr, 0);
+      t = -1;
+    }
   }
 
-  if (rx_list_match (Alternates, addr->mailbox)) {
-    dprint (5,
-            (debugfile, "mail_addr_is_user: yes, %s matched by alternates.\n",
-             addr->mailbox));
-    if (rx_list_match (UnAlternates, addr->mailbox))
-      dprint (5,
-              (debugfile,
-               "mail_addr_is_user: but, %s matched by unalternates.\n",
-               addr->mailbox));
-    else
-      return 1;
+  if (t != -1) {
+    mutt_addrlist_to_local (AliasTable[t]->addr);
+    rfc822_write_address (buf, buflen, AliasTable[t]->addr, 0);
   }
 
-  dprint (5, (debugfile, "mail_addr_is_user: no, all failed.\n"));
-  return 0;
+  mutt_menuDestroy (&menu);
+  p_delete(&AliasTable);
+
 }