X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=user_list.c;h=1143f2d58aa1605d77324fea14bf682e409d1ab4;hp=c3b10b1e1d8ca8a915b92a7b4f3080500233210a;hb=45d9984d5a3e475ece85210928958d7cb6f19864;hpb=55ff877a8bf6500cd044c5bd1a71966d341668d9

diff --git a/user_list.c b/user_list.c
index c3b10b1e..1143f2d5 100644
--- a/user_list.c
+++ b/user_list.c
@@ -58,11 +58,9 @@ static void populate_user_list(char *user_list_file)
 		PARA_DEBUG_LOG("found entry for %s\n", n);
 		u = para_malloc(sizeof(struct user));
 		u->name = para_strdup(n);
-		u->rsa = para_malloc(sizeof(RSA));
 		ret = get_rsa_key(k, &u->rsa, LOAD_PUBLIC_KEY);
 		if (ret < 0)
 			break;
-		u->perms = 0;
 		char_ptr = p;
 		num = sscanf(char_ptr, "%200[A-Z_],%200[A-Z_],%200[A-Z_],%200[A-Z_]",
 			tmp[0], tmp[1], tmp[2], tmp[3]);
@@ -93,6 +91,14 @@ out:
 	exit(EXIT_FAILURE);
 }
 
+/**
+ * initialize the list of users allowed to connecto to para_server
+ *
+ * \param user_list_file the file containing access information
+ *
+ * If this function is called a second time, the contents of the
+ * previous call are discarded.
+ */
 void init_user_list(char *user_list_file)
 {
 	struct user *u, *tmp;
@@ -102,7 +108,7 @@ void init_user_list(char *user_list_file)
 		list_for_each_entry_safe(u, tmp, &user_list, node) {
 			list_del(&u->node);
 			free(u->name);
-			free(u->rsa);
+			rsa_free(u->rsa);
 			free(u);
 		}
 	} else
@@ -114,19 +120,18 @@ void init_user_list(char *user_list_file)
 /**
  * lookup user in user_list.
  *
- * \param user: must initially contain the name of the user and is filled
- * in by this function on success.
+ * \param name of the user
  *
- * \return 1 on success and < 0 on errors.
+ * \return a pointer to the corresponding user struct if the user was found,
+ *  \p NULL otherwise.
  */
-int lookup_user(struct user *user)
+struct user *lookup_user(const char *name)
 {
 	struct user *u;
 	list_for_each_entry(u, &user_list, node) {
-		if (strcmp(u->name, user->name))
+		if (strcmp(u->name, name))
 			continue;
-		*user = *u;
-		return 1;
+		return u;
 	}
-	return -E_BAD_USER;
+	return NULL;
 }