X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=user_list.c;h=0748ef62f1c5030cc6e5de6975e579c7b4c83357;hp=5652774e0685fc423b462071d72ef728e8b99547;hb=e67fbcaf61d3d0f3ce62332f9615e5ed8e5a1c81;hpb=5b8e525dfb2f18a4e0d67b9da264259159ad2184

diff --git a/user_list.c b/user_list.c
index 5652774e..0748ef62 100644
--- a/user_list.c
+++ b/user_list.c
@@ -4,7 +4,7 @@
  * Licensed under the GPL v2. For licencing details see COPYING.
  */
 
-/** \file user_list.c user handling for para_server */
+/** \file user_list.c User handling for para_server. */
 
 #include <sys/types.h>
 #include <dirent.h>
@@ -37,6 +37,7 @@ static void populate_user_list(char *user_list_file)
 		/* keyword, name, key, perms */
 		char w[255], n[255], k[255], p[255], tmp[4][255];
 		struct user *u;
+		RSA *rsa;
 
 		ret = para_fgets(line, MAXLINE, file_ptr);
 		if (ret <= 0)
@@ -45,16 +46,20 @@ static void populate_user_list(char *user_list_file)
 			continue;
 		if (strcmp(w, "user"))
 			continue;
-		PARA_DEBUG_LOG("found entry for %s\n", n);
+		PARA_DEBUG_LOG("found entry for user %s\n", n);
+		ret = get_rsa_key(k, &rsa, LOAD_PUBLIC_KEY);
+		if (ret < 0) {
+			PARA_NOTICE_LOG("skipping entry for user %s: %s\n", n,
+				para_strerror(-ret));
+			continue;
+		}
 		u = para_malloc(sizeof(*u));
 		u->name = para_strdup(n);
-		ret = get_rsa_key(k, &u->rsa, LOAD_PUBLIC_KEY);
-		if (ret < 0)
-			break;
+		u->rsa = rsa;
+		u->perms = 0;
 		num = sscanf(p, "%200[A-Z_],%200[A-Z_],%200[A-Z_],%200[A-Z_]",
 			tmp[0], tmp[1], tmp[2], tmp[3]);
 		PARA_DEBUG_LOG("found %i perm entries\n", num);
-		u->perms = 0;
 		while (num > 0) {
 			num--;
 			if (!strcmp(tmp[num], "VSS_READ"))
@@ -80,12 +85,12 @@ err:
 }
 
 /**
- * initialize the list of users allowed to connecto to para_server
+ * Initialize the list of users allowed to connect to to para_server.
  *
- * \param user_list_file the file containing access information
+ * \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.
+ * If this function is called for the second time, the contents of the
+ * previous call are discarded, i.e. the user list is reloaded.
  */
 void init_user_list(char *user_list_file)
 {
@@ -106,12 +111,12 @@ void init_user_list(char *user_list_file)
 }
 
 /**
- * lookup user in user_list.
+ * Lookup a user in the user list.
  *
- * \param name of the user
+ * \param name The name of the user.
  *
- * \return a pointer to the corresponding user struct if the user was found,
- *  \p NULL otherwise.
+ * \return A pointer to the corresponding user struct if the user was found, \p
+ * NULL otherwise.
  */
 struct user *lookup_user(const char *name)
 {