Remove useless delcaration of index().

[moira.git] / clients / blanche / blanche.c

diff --git a/clients/blanche/blanche.c b/clients/blanche/blanche.c
index 119e12b76117a6d21e28cc6e9e6e3bd9108b9dd0..b311941fbabc51a63ca9d142c90585c5d3e443e6 100644 (file)
--- a/clients/blanche/blanche.c
+++ b/clients/blanche/blanche.c
@@ -15,8 +15,10 @@
    Do we need to improve the support for queue operations? */
 
 #include <mit-copyright.h>
+#include <errno.h>
 #include <stdio.h>
 #include <ctype.h>
+#include <string.h>
 #include <moira.h>
 #include <moira_site.h>
 
@@ -49,8 +51,9 @@ struct save_queue *addlist, *dellist, *memberlist, *synclist;
 
 char *listname, *whoami;
 
-extern char *index();
+#ifndef POSIX
 extern int errno;
+#endif
 
 int show_list_info(), show_list_count(), get_list_members(), scream();
 int show_list_members(), membercmp();
@@ -238,7 +241,7 @@ char **argv;
     while (sq_get_data(addlist, &memberstruct)) {
        /* canonicalize string if necessary */
        if (memberstruct->type == M_STRING &&
-           (p = index(memberstruct->name, '@'))) {
+           (p = strchr(memberstruct->name, '@'))) {
            char *host = canonicalize_hostname(strsave(++p));
            static char **mailhubs = NULL;
            char *argv[4];
@@ -250,7 +253,8 @@ char **argv;
                argv[2] = "*";
                mailhubs = (char **)malloc(sizeof(char *));
                mailhubs[0] = NULL;
-               status = mr_query("get_alias", 3, argv, collect, &mailhubs);
+               status = mr_query("get_alias", 3, argv, collect,
+                                  (char *)&mailhubs);
                if (status != MR_SUCCESS && status != MR_NO_MATCH) {
                    com_err(whoami, status,
                            " while reading list of MAILHUB servers");
@@ -260,7 +264,7 @@ char **argv;
            for (i = 0; p = mailhubs[i]; i++) {
                if (!strcasecmp(p, host)) {
                    host = strsave(memberstruct->name);
-                   *(index(memberstruct->name, '@')) = 0;
+                   *(strchr(memberstruct->name, '@')) = 0;
                    memberstruct->type = M_ANY;
                    fprintf(stderr, "Warning: \"STRING:%s\" converted to \"%s\" because it is a local name.\n",
                            host, memberstruct->name);
@@ -347,6 +351,20 @@ char **argv;
              break;
            else if ((status != MR_LIST && status != MR_NO_MATCH) ||
                     memberstruct->type != M_ANY) {
+               if (status == MR_PERM && memberstruct->type == M_ANY)  {
+                 /* M_ANY means we've fallen through from the user case
+                  * The fact that we didn't get MR_PERM there indicates
+                  * that we had permission to remove the specified member 
+                  * from the list if it is a user, but not a list.  This is 
+                  * if we are the member in question.  Since we exist as a user
+                  * we must have gotten the MR_NO_MATCH error, so we will
+                  * return that, since it will be less confusing.  However,
+                  * This will generate the wrongerror if the user was trying
+                  * to remove the list with his/her username from a list they
+                  * don't administrate, without explicitly specifying "list:".
+                  */
+                 status = MR_NO_MATCH;
+               }
                com_err(whoami, status, "while deleting member %s from %s",
                        memberstruct->name, listname);
                break;
@@ -395,7 +413,7 @@ char **argv;
 usage(argv)
 char **argv;
 {
-    fprintf(stderr, "Usage: %s [options] listname [options]\n",argv[0]);
+    fprintf(stderr, "Usage: %s listname [options]\n",argv[0]);
     fprintf(stderr, "Options are\n");
     fprintf(stderr, "   -v | -verbose\n");
     fprintf(stderr, "   -m | -members\n");
@@ -458,7 +476,7 @@ struct member *memberstruct;
        else if (memberstruct->type == M_KERBEROS)
          printf("KERBEROS:%s\n", memberstruct->name);
        else if (memberstruct->type == M_STRING &&
-                !index(memberstruct->name, '@'))
+                !strchr(memberstruct->name, '@'))
          printf("STRING:%s\n", memberstruct->name);
        else
          printf("%s\n", memberstruct->name);
@@ -538,7 +556,7 @@ recursive_display_list_members()
     data = (struct member *) malloc(count * sizeof(struct member));
     count = 0;
     while (sq_get_data(members, &m))
-      bcopy(m, &data[count++], sizeof(struct member));
+      memcpy(&data[count++], m, sizeof(struct member));
     qsort(data, count, sizeof(struct member), membercmp);
     for (count = 0; count < savecount; count++) {
        show_list_member(&data[count]);
@@ -671,7 +689,7 @@ register char *s;
     if ((m = (struct member *) malloc(sizeof(struct member))) == NULL)
       return(NULL);
 
-    if (p = index(s, ':')) {
+    if (p = strchr(s, ':')) {
        *p = 0;
        m->name = ++p;
        if (!strcasecmp("user", s))
@@ -690,7 +708,7 @@ register char *s;
        m->name = strsave(m->name);
     } else {
        m->name = strsave(s);
-       if (index(s, '@') || index(s, '!') || index(s, '%') || index(s, ' '))
+       if (strchr(s, '@') || strchr(s, '!') || strchr(s, '%') || strchr(s, ' '))
          m->type = M_STRING;
        else
          m->type = M_ANY;