X-Git-Url: http://andersk.mit.edu/gitweb/moira.git/blobdiff_plain/f416d13df6cfcccf9c6ede34ac7b2e11c49cc918..049703d6dda7beebfed582fbb8ee877e32b31985:/clients/blanche/blanche.c

diff --git a/clients/blanche/blanche.c b/clients/blanche/blanche.c
index c83a8739..b311941f 100644
--- 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];
@@ -261,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);
@@ -348,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;
@@ -459,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);
@@ -539,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]);
@@ -672,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))
@@ -691,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;