X-Git-Url: http://git.tuebingen.mpg.de/?p=adu.git;a=blobdiff_plain;f=select.c;h=7d1212c93cd63fa42d7f54bece7986b3cbf45e7c;hp=28bd492c9521826eb857980ef1fed221a3cb0971;hb=5008f98a8208d96c3c777476d6df7534c6d2ca49;hpb=671cb51713154b62c809603293422f6a5e873cef

diff --git a/select.c b/select.c
index 28bd492..7d1212c 100644
--- a/select.c
+++ b/select.c
@@ -4,9 +4,12 @@
  * Licensed under the GPL v2. For licencing details see COPYING.
  */
 
-/** \file select.c The select mode of adu. */
+/** \file select.c \brief The select mode of adu. */
 
 #include <dirent.h> /* readdir() */
+#include <sys/types.h>
+#include <regex.h>
+
 #include "format.h"
 #include "adu.h"
 #include "gcc-compat.h"
@@ -14,8 +17,10 @@
 #include "fd.h"
 #include "string.h"
 #include "error.h"
-#include "portable_io.h"
+#include "user.h"
+#include "select.cmdline.h"
 
+/** \cond */
 /* global list */
 #define GLOBAL_LIST_ATOMS \
 	ATOM(size, SIZE) \
@@ -66,6 +71,21 @@ struct atom user_list_atoms[] = {
 enum user_list_atoms {USER_LIST_ATOMS};
 #undef ATOM
 
+/* user list header */
+#define USER_LIST_HEADER_ATOMS \
+	ATOM(pw_name, STRING) \
+	ATOM(uid, ID)
+
+#define ATOM(x, y) { .name = #x, .type = AT_ ## y},
+struct atom user_list_header_atoms[] = {
+	USER_LIST_HEADER_ATOMS
+	{.name = NULL}
+};
+#undef ATOM
+#define ATOM(x, y) ulha_ ## x,
+enum user_list_header_atoms {USER_LIST_HEADER_ATOMS};
+#undef ATOM
+
 /* user summary */
 #define USER_SUMMARY_ATOMS \
 	ATOM(pw_name, STRING) \
@@ -84,11 +104,15 @@ struct atom user_summary_atoms[] = {
 enum user_summary_atoms {USER_SUMMARY_ATOMS};
 #undef ATOM
 
+/** \endcond */
+
 struct global_list_info {
 	uint32_t count;
 	int ret;
 	int osl_errno;
 	struct format_info *fi;
+	regex_t *preg;
+	int inverse_matching;
 };
 
 struct global_summary_info {
@@ -98,6 +122,8 @@ struct global_summary_info {
 	uint64_t num_files;
 	/** Global bytes count. */
 	uint64_t num_bytes;
+	regex_t *preg;
+	int inverse_matching;
 	int ret;
 	int osl_errno;
 };
@@ -106,16 +132,29 @@ struct user_list_info {
 	uint32_t count;
 	struct user_info *ui;
 	struct format_info *fi;
+	regex_t *preg;
+	int inverse_matching;
 	int ret;
 	int osl_errno;
 };
 
+struct user_list_format_info {
+	struct format_info *fi;
+	struct format_info *header_fi;
+};
+
 struct user_summary_info {
 	struct user_info *ui;
 	int ret;
 	int osl_errno;
+	regex_t *preg;
+	int inverse_matching;
 };
 
+struct user_summary_line_info {
+	struct format_info *fi;
+	uint32_t count;
+};
 
 static FILE *output_file;
 
@@ -237,6 +276,61 @@ static int get_num_user_bytes(struct osl_row *row, struct user_info *ui,
 	return 1;
 }
 
+static void free_regex(regex_t *preg)
+{
+	if (!preg)
+		return;
+	regfree(preg);
+	free(preg);
+}
+
+static int compile_regex(regex_t **preg, int *invert)
+{
+	int ret;
+	size_t size;
+	char *buf, *p = select_conf.pattern_arg;
+
+	if (!select_conf.pattern_given || !p[0]) {
+		*preg = NULL;
+		return 0;
+	}
+	if (p[0] == '!') {
+		if (!p[1]) {
+			*preg = NULL;
+			return -E_REGEX;
+		}
+		*invert = 1;
+		p++;
+	} else
+		*invert = 0;
+	*preg = adu_malloc(sizeof(regex_t));
+	ret = regcomp(*preg, p, 0);
+	if (!ret)
+		return 1;
+	size = regerror(ret, *preg, NULL, 0);
+	buf = adu_malloc(size);
+	regerror(ret, *preg, buf, size);
+	ERROR_LOG("%s\n", buf);
+	free(buf);
+	free_regex(*preg);
+	*preg = NULL;
+	return -E_REGEX;
+}
+
+static int dir_is_admissible(char *dirname, regex_t *preg, int inverse_matching)
+{
+	int ret;
+
+	if (!preg)
+		return 1;
+	ret = regexec(preg, dirname, 0, NULL, 0);
+	if (ret == REG_NOMATCH && !inverse_matching)
+		return 0;
+	if (ret != REG_NOMATCH && inverse_matching)
+		return 0;
+	return 1;
+}
+
 static int check_loop_return(int ret, int loop_ret, int loop_osl_errno)
 {
 	if (ret >= 0)
@@ -268,6 +362,17 @@ static int global_summary_loop_function(struct osl_row *row, void *data)
 	int ret;
 	uint64_t num;
 
+	if (gsi->preg) {
+		char *dirname;
+		ret = get_dir_name_of_row(row, &dirname);
+		if (ret < 0)
+			goto err;
+		ret = dir_is_admissible(dirname, gsi->preg, gsi->inverse_matching);
+		free(dirname);
+		if (!ret)
+			return 1;
+	}
+
 	ret = get_num_files_of_row(row, &num);
 	if (ret < 0)
 		goto err;
@@ -282,7 +387,7 @@ static int global_summary_loop_function(struct osl_row *row, void *data)
 err:
 	gsi->ret = ret;
 	gsi->osl_errno = (ret == -E_OSL)? osl_errno : 0;
-	return -1;
+	return ret;
 }
 
 static int print_global_summary(struct format_info *fi)
@@ -290,6 +395,8 @@ static int print_global_summary(struct format_info *fi)
 	int ret;
 	char *buf;
 	struct global_summary_info gsi = {.num_dirs = 0};
+	char *header = select_conf.header_given? select_conf.header_arg :
+		"Global summary\n";
 
 	union atom_value values[] = {
 		[gsa_dirs] = {.num_value = 0ULL},
@@ -297,18 +404,21 @@ static int print_global_summary(struct format_info *fi)
 		[gsa_size] = {.num_value =  0ULL}
 	};
 
+	ret = compile_regex(&gsi.preg, &gsi.inverse_matching);
+	if (ret < 0)
+		return ret;
 	ret = adu_loop_reverse(dir_table, DT_BYTES, &gsi,
 		global_summary_loop_function, &gsi.ret, &gsi.osl_errno);
+	free_regex(gsi.preg);
 	if (ret < 0)
 		return ret;
 	values[gsa_dirs].num_value = (long long unsigned)gsi.num_dirs;
 	values[gsa_files].num_value = (long long unsigned)gsi.num_files;
 	values[gsa_size].num_value = (long long unsigned)gsi.num_bytes;
-	if (!select_conf.no_headers_given) {
-		ret = output("Global summary\n");
-		if (ret < 0)
-			return ret;
-	}
+
+	ret = output("%s", header);
+	if (ret < 0)
+		return ret;
 	buf = format_items(fi, values);
 	ret = output("%s", buf);
 	free(buf);
@@ -321,6 +431,16 @@ static int user_summary_loop_function(struct osl_row *row, void *data)
 	uint64_t num;
 	int ret;
 
+	if (usi->preg) {
+		char *dirname;
+		ret = get_dir_name_of_row(row, &dirname);
+		if (ret < 0)
+			goto err;
+		ret = dir_is_admissible(dirname, usi->preg, usi->inverse_matching);
+		free(dirname);
+		if (!ret)
+			return 1;
+	}
 	ret = get_num_user_files(row, usi->ui, &num);
 	if (ret < 0)
 		goto err;
@@ -334,20 +454,25 @@ static int user_summary_loop_function(struct osl_row *row, void *data)
 err:
 	usi->ret = ret;
 	usi->osl_errno = (ret == -E_OSL)? osl_errno : 0;
-	return -1;
+	return ret;
 }
 
 static int compute_user_summary(struct user_info *ui, __a_unused void *data)
 {
 	struct user_summary_info usi = {.ui = ui};
+	int ret = compile_regex(&usi.preg, &usi.inverse_matching);
 
-	return adu_loop_reverse(ui->table, UT_BYTES, &usi, user_summary_loop_function,
+	if (ret < 0)
+		return ret;
+	ret = adu_loop_reverse(ui->table, UT_BYTES, &usi, user_summary_loop_function,
 		&usi.ret, &usi.osl_errno);
+	free_regex(usi.preg);
+	return ret;
 }
 
 static int print_user_summary_line(struct user_info *ui, void *data)
 {
-	struct format_info *fi = data;
+	struct user_summary_line_info *usli = data;
 	union atom_value values[] = {
 		[usa_pw_name] = {.string_value = ui->pw_name?
 			ui->pw_name : "?"},
@@ -357,18 +482,22 @@ static int print_user_summary_line(struct user_info *ui, void *data)
 		[usa_size] = {.num_value =  (long long unsigned)ui->bytes}
 	};
 	char *buf;
-	int ret;
+	int ret = -E_LOOP_COMPLETE;
 
-	buf = format_items(fi, values);
+	if (!usli->count)
+		return ret;
+
+	buf = format_items(usli->fi, values);
 	ret = output("%s", buf);
 	free(buf);
+	usli->count--;
 	return ret;
 }
 
-static int name_comp(const void *a, const void *b)
+static int name_comp(struct user_info *a, struct user_info *b)
 {
-	char *x = ((struct user_info *)a)->pw_name;
-	char *y = ((struct user_info *)b)->pw_name;
+	char *x = a->pw_name;
+	char *y = b->pw_name;
 
 	if (!x)
 		return 1;
@@ -377,56 +506,65 @@ static int name_comp(const void *a, const void *b)
 	return strcmp(x, y);
 }
 
-static int uid_comp(const void *a, const void *b)
+static int uid_comp(struct user_info *a, struct user_info *b)
 {
-	return -NUM_COMPARE(((struct user_info *)a)->uid,
-		((struct user_info *)b)->uid);
+	return -NUM_COMPARE(a->uid, b->uid);
 }
 
-static int dir_count_comp(const void *a, const void *b)
+static int dir_count_comp(struct user_info *a, struct user_info *b)
 {
-	return NUM_COMPARE(((struct user_info *)a)->dirs,
-		((struct user_info *)b)->dirs);
+	return NUM_COMPARE(a->dirs, b->dirs);
 }
 
-static int file_count_comp(const void *a, const void *b)
+static int file_count_comp(struct user_info *a, struct user_info *b)
 {
-	return NUM_COMPARE(((struct user_info *)a)->files,
-		((struct user_info *)b)->files);
+	return NUM_COMPARE(a->files, b->files);
 }
 
-static int size_comp(const void *a, const void *b)
+static int size_comp(struct user_info *a, struct user_info *b)
 {
-	return NUM_COMPARE(((struct user_info *)a)->bytes,
-		((struct user_info *)b)->bytes);
+	return NUM_COMPARE(a->bytes, b->bytes);
 }
 
-/*
- * The comparators for sorting the user summary.
- *
- * This is an array of pointers to functions taking two constant void *
- * pointers and returning an int.
- */
-static int (*summary_comparators[])(const void *, const void *) = {
-	[user_summary_sort_arg_name] = name_comp,
-	[user_summary_sort_arg_uid] = uid_comp,
-	[user_summary_sort_arg_dir_count] = dir_count_comp,
-	[user_summary_sort_arg_file_count] = file_count_comp,
-	[user_summary_sort_arg_size] = size_comp,
-};
-
 static int print_user_summary(struct format_info *fi)
 {
-	if (!select_conf.no_headers_given) {
-		int ret = output("User summary\n");
-		if (ret < 0)
-			return ret;
-	}
-	int ret = for_each_admissible_user(compute_user_summary, fi);
+	int ret;
+	int (*comp)(struct user_info *a, struct user_info *b);
+	struct user_summary_line_info usli = {
+		.fi = fi,
+		.count = select_conf.limit_arg
+	};
+	char *header = select_conf.header_given? select_conf.header_arg :
+		"User summary\n";
+
+	ret = output("%s", header);
+	if (ret < 0)
+		return ret;
+	ret = for_each_admissible_user(compute_user_summary, NULL);
 	if (ret < 0)
 		return ret;
-	sort_hash_table(summary_comparators[select_conf.user_summary_sort_arg]);
-	return for_each_admissible_user(print_user_summary_line, fi);
+	switch (select_conf.user_summary_sort_arg) {
+	case user_summary_sort_arg_name:
+		comp = name_comp;
+		break;
+	case user_summary_sort_arg_uid:
+		comp = uid_comp;
+		break;
+	case user_summary_sort_arg_dir_count:
+		comp = dir_count_comp;
+		break;
+	case user_summary_sort_arg_file_count:
+		comp = file_count_comp;
+		break;
+	case user_summary_sort_arg_size:
+		comp = size_comp;
+		break;
+	}
+	sort_hash_table(comp);
+	ret = for_each_admissible_user(print_user_summary_line, &usli);
+	if (ret == -E_LOOP_COMPLETE)
+		ret = 1;
+	return ret;
 }
 
 static int user_list_loop_function(struct osl_row *row, void *data)
@@ -442,13 +580,22 @@ static int user_list_loop_function(struct osl_row *row, void *data)
 	};
 	uint64_t num;
 	int ret;
-	char *dirname, *buf;
+	char *dirname = NULL, *buf;
 
 	check_signals();
 	ret = -E_LOOP_COMPLETE;
 	if (!uli->count)
 		goto err;
 
+	ret = get_dir_name_of_user_row(row, uli->ui, &dirname);
+	if (ret < 0)
+		goto err;
+	if (!dir_is_admissible(dirname, uli->preg, uli->inverse_matching)) {
+		free(dirname);
+		return 1;
+	}
+	values[ula_dirname].string_value = dirname;
+
 	ret = get_num_user_files(row, uli->ui, &num);
 	if (ret < 0)
 		goto err;
@@ -459,45 +606,69 @@ static int user_list_loop_function(struct osl_row *row, void *data)
 		goto err;
 	values[ula_size].num_value = num;
 
-	ret = get_dir_name_of_user_row(row, uli->ui, &dirname);
-	if (ret < 0)
-		goto err;
-	values[ula_dirname].string_value = dirname;
-
 	buf = format_items(uli->fi, values);
 	free(dirname);
+	dirname = NULL;
 	ret = output("%s", buf);
 	free(buf);
+	if (ret < 0)
+		goto err;
 	uli->count--;
 	return ret;
 err:
+	free(dirname);
 	uli->ret = ret;
 	uli->osl_errno = (ret == -E_OSL)? osl_errno : 0;
-	return -1;
+	return ret;
 }
 
 static int print_user_list(struct user_info *ui, void *data)
 {
-	struct format_info *fi = data;
+	struct user_list_format_info *ulfi = data;
 	int ret;
-	enum user_table_columns sort_column = UT_BYTES;
+	enum user_table_columns sort_column;
 	struct user_list_info uli = {
 		.ui = ui,
-		.fi = fi,
+		.fi = ulfi->fi,
 		.count = select_conf.limit_arg
 	};
+	union atom_value header_values[] = {
+		[ulha_uid] = {.num_value = (long long unsigned)ui->uid},
+		[ulha_pw_name] = {.string_value = ui->pw_name?
+			ui->pw_name : "?"}
+	};
+	char *buf = format_items(ulfi->header_fi, header_values);
 
+	ret = output("%s", buf);
+	free(buf);
+	if (ret < 0)
+		return ret;
 	if (select_conf.list_sort_arg == list_sort_arg_file_count)
 		sort_column = UT_FILES;
+	else
+		sort_column = UT_BYTES;
 
-	if (!select_conf.no_headers_given) {
-		ret = output("%s (uid %u)\n",
-			ui->pw_name? ui->pw_name : "?", (unsigned)ui->uid);
-		if (ret < 0)
-			return ret;
-	}
-	return adu_loop_reverse(ui->table, sort_column, &uli, user_list_loop_function,
-		&uli.ret, &uli.osl_errno);
+	ret = compile_regex(&uli.preg, &uli.inverse_matching);
+	if (ret < 0)
+		return ret;
+	ret = adu_loop_reverse(ui->table, sort_column, &uli,
+		user_list_loop_function, &uli.ret, &uli.osl_errno);
+	free_regex(uli.preg);
+	return ret;
+}
+
+static int print_user_lists(struct format_info *fi)
+{
+	struct user_list_format_info ulfi = {.fi = fi};
+	char *header_fmt = select_conf.header_given?
+		select_conf.header_arg : "uid %(uid)(%(pw_name)):\n";
+	int ret = parse_format_string(header_fmt,
+		user_list_header_atoms, &ulfi.header_fi);
+	if (ret < 0)
+		return ret;
+	ret = for_each_admissible_user(print_user_list, &ulfi);
+	free_format_info(ulfi.header_fi);
+	return ret;
 }
 
 static int global_list_loop_function(struct osl_row *row, void *data)
@@ -509,7 +680,7 @@ static int global_list_loop_function(struct osl_row *row, void *data)
 		[gla_dirname] = {.string_value = NULL}
 	};
 	uint64_t num_files, num_bytes;
-	char *dirname, *buf;
+	char *dirname = NULL, *buf;
 	int ret;
 
 	check_signals();
@@ -517,6 +688,15 @@ static int global_list_loop_function(struct osl_row *row, void *data)
 	if (!gli->count)
 		goto err;
 
+	ret = get_dir_name_of_row(row, &dirname);
+	if (ret < 0)
+		goto err;
+	if (!dir_is_admissible(dirname, gli->preg, gli->inverse_matching)) {
+		free(dirname);
+		return 1;
+	}
+	values[gla_dirname].string_value = dirname;
+
 	ret = get_num_files_of_row(row, &num_files);
 	if (ret < 0)
 		goto err;
@@ -527,19 +707,18 @@ static int global_list_loop_function(struct osl_row *row, void *data)
 		goto err;
 	values[gla_size].num_value = (long long unsigned)num_bytes;
 
-	ret = get_dir_name_of_row(row, &dirname);
-	if (ret < 0)
-		goto err;
-	values[gla_dirname].string_value = dirname;
-
 	buf = format_items(gli->fi, values);
 	free(dirname);
+	dirname = NULL;
 	ret = output("%s", buf);
 	free(buf);
+	if (ret < 0)
+		goto err;
 	if (gli->count > 0)
 		gli->count--;
 	return ret;
 err:
+	free(dirname);
 	gli->ret = ret;
 	gli->osl_errno = (ret == -E_OSL)? osl_errno : 0;
 	return -1;
@@ -548,121 +727,253 @@ err:
 static int print_global_list(struct format_info *fi)
 {
 	int ret;
-	enum dir_table_columns sort_column = DT_BYTES;
+	enum dir_table_columns sort_column;
 	struct global_list_info gli = {
 		.fi = fi,
 		.count = select_conf.limit_arg
 	};
+	char *header = select_conf.header_given?
+		select_conf.header_arg : "Global list\n";
 
-	if (!select_conf.no_headers_given) {
-		ret = output("Global list\n");
-		if (ret < 0)
-			return ret;
-	}
+	ret = output("%s", header);
+	if (ret < 0)
+		return ret;
 	if (select_conf.list_sort_arg == list_sort_arg_file_count)
 		sort_column = DT_FILES;
-	return adu_loop_reverse(dir_table, sort_column, &gli,
+	else
+		sort_column = DT_BYTES;
+	ret = compile_regex(&gli.preg, &gli.inverse_matching);
+	if (ret < 0)
+		return ret;
+	ret = adu_loop_reverse(dir_table, sort_column, &gli,
 		global_list_loop_function, &gli.ret, &gli.osl_errno);
+	free_regex(gli.preg);
+	return ret;
 }
 
-static int print_statistics(struct select_format_info *sli)
+static int print_statistics(struct format_info *fi)
 {
-	int ret;
-
 	switch (select_conf.select_mode_arg) {
-		case select_mode_arg_global_list:
-			ret = print_global_list(sli->global_list_fi);
-			free_format_info(sli->global_list_fi);
-			return ret;
-		case select_mode_arg_global_summary:
-			ret = print_global_summary(sli->global_summary_fi);
-			free_format_info(sli->global_summary_fi);
-			return ret;
-		case select_mode_arg_user_list:
-			ret = for_each_admissible_user(print_user_list,
-				sli->user_list_fi);
-			free_format_info(sli->user_list_fi);
-			return ret;
-		case select_mode_arg_user_summary:
-			ret = print_user_summary(sli->user_summary_fi);
-			free_format_info(sli->user_summary_fi);
-			return ret;
+	case select_mode_arg_global_list:
+		return print_global_list(fi);
+	case select_mode_arg_global_summary:
+		return print_global_summary(fi);
+	case select_mode_arg_user_list:
+		return print_user_lists(fi);
+	case select_mode_arg_user_summary:
+		return print_user_summary(fi);
 	};
 	ERROR_LOG("bad select mode\n");
-	return ERRNO_TO_ERROR(-EINVAL);
+	return -ERRNO_TO_ERROR(EINVAL);
 }
 
-static int read_uid_file(struct uid_range *admissible_uids)
+static int open_pipe(char *path)
 {
-	size_t size;
-	uint32_t n;
-	char *filename = get_uid_list_name(), *map;
-	int ret = mmap_full_file(filename, O_RDONLY, (void **)&map, &size, NULL);
-	unsigned bits;
+	int p[2], ret, argc;
+	char **argv;
 
-	if (ret < 0) {
-		INFO_LOG("failed to map %s\n", filename);
-		free(filename);
-		return ret;
+	ret = pipe(p);
+	if (ret < 0)
+		return ERRNO_TO_ERROR(errno);
+	ret = fork();
+	if (ret < 0)
+		return ERRNO_TO_ERROR(errno);
+	if (ret) { /* parent */
+		DEBUG_LOG("created process %d\n", ret);
+		close(p[0]);
+		output_file = fdopen(p[1], "w");
+		if (!output_file)
+			return ERRNO_TO_ERROR(errno);
+		return 1;
+	}
+	close(p[1]);
+	if (p[0] != STDIN_FILENO)
+		dup2(p[0], STDIN_FILENO);
+	DEBUG_LOG("executing %s\n", path);
+	argc = split_args(path, &argv, " \t");
+	execvp(argv[0], argv);
+	ERROR_LOG("error executing %s: %s\n", path,
+		adu_strerror(ERRNO_TO_ERROR(errno)));
+	_exit(EXIT_FAILURE);
+}
+
+static int open_output_stream(void)
+{
+	char *p;
+	int ret, flags = O_WRONLY | O_CREAT;
+
+	if (!select_conf.output_given)
+		goto use_stdout;
+	p = select_conf.output_arg;
+	switch (p[0]) {
+	case '\0': /* empty string */
+		goto bad_output_arg;
+	case '-':
+		if (!p[1]) /* "-" means stdout */
+			goto use_stdout;
+		/* string starting with a dash */
+		flags |= O_EXCL;
+		goto open_file;
+	case '>':
+		if (!p[1]) /* ">" is invalid */
+			goto bad_output_arg;
+		if (p[1] != '>') {
+			p++;
+			flags |= O_TRUNC;
+			goto open_file;
+		}
+		/* string starting with ">>" */
+		if (!p[2]) /* ">>" is invalid */
+			goto bad_output_arg;
+		flags |= O_APPEND;
+		p += 2;
+		goto open_file;
+	case '|':
+		if (!p[1]) /* "|" is invalid */
+			goto bad_output_arg;
+		p++;
+		return open_pipe(p);
+	default: /* args starts with no magic character */
+		flags |= O_EXCL;
+		goto open_file;
 	}
-	num_uids = size / 4;
-	INFO_LOG("found %u uids in %s\n", (unsigned)num_uids, filename);
-	free(filename);
+use_stdout:
+	output_file = stdout;
+	return 1;
+bad_output_arg:
+	output_file = NULL;
+	return -E_BAD_OUTPUT_ARG;
+open_file:
 	/*
-	 * Compute number of hash table bits. The hash table size must be a
-	 * power of two and larger than the number of uids.
+	 * glibc's 'x' mode to fopen is not portable, so use open() and
+	 * fdopen().
 	 */
-	bits = 2;
-	while (1 << bits < num_uids)
-		bits++;
-	create_hash_table(bits);
-	for (n = 0; n < num_uids; n++) {
-		uint32_t uid = read_u32(map + n * sizeof(uid));
-		ret = search_uid(uid, admissible_uids, OPEN_USER_TABLE, NULL);
-		if (ret < 0)
-			goto out;
-	}
-out:
-	adu_munmap(map, size);
-	return ret;
+	ret = open(p, flags, 0644);
+	if (ret < 0)
+		return -ERRNO_TO_ERROR(errno);
+	output_file = fdopen(ret, "w");
+	if (!output_file)
+		return -ERRNO_TO_ERROR(errno);
+	return 1;
 }
 
-int run_select_query(struct uid_range *admissible_uids,
-		struct select_format_info *sfi)
+/**
+ * Execute a select query.
+ *
+ * \param admissible_uids User IDs to take into account.
+ * \param fi Format information.
+ *
+ * Called once in select mode or for each \a run command in interactive mode.
+ *
+ * Open the output stream and the dir table if not already open. For each
+ * admissible uid, the user table is opened if necessary. After these
+ * preparations, the output according to \a select_mode and \a fi is written to
+ * the output stream.
+ *
+ * \return Standard.
+ */
+int run_select_query(struct uid_range *admissible_uids, struct format_info *fi)
 {
-	int ret;
-
-	if (select_conf.output_given && strcmp(select_conf.output_arg, "-")) {
-		output_file = fopen(select_conf.output_arg, "w");
-		if (!output_file)
-			return -ERRNO_TO_ERROR(errno);
-	} else
-		output_file = stdout;
+	int ret = open_output_stream();
 
+	if (ret < 0)
+		goto out;
 	ret = open_dir_table(0);
 	if (ret < 0)
 		goto out;
 	check_signals();
-	ret = read_uid_file(admissible_uids);
+	ret = open_admissible_user_tables(admissible_uids);
 	if (ret < 0)
 		goto out;
 	check_signals();
-	ret = print_statistics(sfi);
+	ret = print_statistics(fi);
 out:
-	close_all_tables();
-	if (output_file != stdout)
+	if (output_file && output_file != stdout) {
 		fclose(output_file);
+		output_file = NULL;
+	}
 	return ret;
 }
 
-/* return: < 0: error, >0: OK, == 0: help given */
+/** Default format string for global_list mode. */
+#define GLOBAL_LIST_DFLT_FMT "%(size:r:8) %(files:r:8) %(dirname)\n"
+/** Default format string for global_summary mode. */
+#define GLOBAL_SUMMARY_DFLT_FMT "#directories: %(dirs), #files: %(files), size: %(size)\n\n"
+/** Default format string for user_list mode. */
+#define USER_LIST_DFLT_FMT "%(size:r:5) %(files:r:5) %(dirname)\n"
+/** Default format string for user_summary mode. */
+#define USER_SUMMARY_DFLT_FMT "%(pw_name:l:16) %(uid:r:5) %(dirs:r:5) %(files:r:5) %(size:r:5)\n"
+
+static int setup_format_string(char *fmt, struct format_info **fi)
+{
+	struct atom *atoms;
+
+	if (!fmt)
+		INFO_LOG("using default format string\n");
+	switch (select_conf.select_mode_arg) {
+	case select_mode_arg_global_list:
+		if (!fmt)
+			fmt = GLOBAL_LIST_DFLT_FMT;
+		atoms = global_list_atoms;
+		break;
+	case select_mode_arg_global_summary:
+		if (!fmt)
+			fmt = GLOBAL_SUMMARY_DFLT_FMT;
+		atoms = global_summary_atoms;
+		break;
+	case select_mode_arg_user_list:
+		if (!fmt)
+			fmt = USER_LIST_DFLT_FMT;
+		atoms = user_list_atoms;
+		break;
+	case select_mode_arg_user_summary:
+		if (!fmt)
+			fmt = USER_SUMMARY_DFLT_FMT;
+		atoms = user_summary_atoms;
+		break;
+	default:
+		ERROR_LOG("bad select mode\n");
+		return -ERRNO_TO_ERROR(EINVAL);
+	};
+	INFO_LOG("format string: %s\n", fmt);
+	return parse_format_string(fmt, atoms, fi);
+}
+
+/**
+ * Parse a given format string.
+ *
+ * \param string The format string to parse.
+ * \param params gengetopt parameters.
+ * \param admissible_uids The array of admissible uid ranges.
+ * \param fi The format info to be used with format_items().
+ *
+ * If \a string is not \p NULL, it is broken down into its components using
+ * \ref create_argv() and the resulting argument vector is passed together with
+ * \a params to gengetopt's command line parser. If --help or --detailed-help
+ * was specified in \a string, the corresponding help text is printed and the
+ * function returns zero.
+ *
+ * Otherwise, any --uid or --user options are parsed and transformed into an
+ * array of admissible uids which is returned via \a admissible_uids.
+ *
+ * Finally, the format string given by --format (or the default format string
+ * for the given select mode if no --format option was given in \a string) is
+ * parsed as well resulting in a format_info structure which is returned via
+ * \a fi. The caller uses the \a fi pointer later to format each output line.
+ *
+ * \return Negative on errors, zero if --help or --detailed-help was given,
+ * positive otherwise.
+ *
+ * \sa format_items().
+ */
 int parse_select_options(char *string, struct select_cmdline_parser_params *params,
-		struct uid_range **admissible_uids, struct select_format_info *sfi)
+		struct uid_range **admissible_uids, struct format_info **fi)
 {
-	int ret;
+	int ret, num_uid_ranges;
 	const char **line;
+	char *fmt = NULL;
 
-	if (conf.select_options_given) {
+	if (string) {
 		int argc;
 		char **argv;
 
@@ -676,35 +987,17 @@ int parse_select_options(char *string, struct select_cmdline_parser_params *para
 			return -E_SYNTAX;
 		if (select_conf.help_given || select_conf.detailed_help_given)
 			goto help;
-
+		fmt = select_conf.format_arg;
 	}
 	ret = parse_uid_arg(select_conf.uid_arg, admissible_uids);
 	if (ret < 0)
 		return ret;
-	ret = parse_format_string(select_conf.user_summary_format_arg,
-		user_summary_atoms, &sfi->user_summary_fi);
+	num_uid_ranges = ret;
+	ret = append_users(select_conf.user_arg, select_conf.user_given,
+		admissible_uids, num_uid_ranges);
 	if (ret < 0)
 		return ret;
-	ret = parse_format_string(select_conf.global_summary_format_arg,
-		global_summary_atoms, &sfi->global_summary_fi);
-	if (ret < 0)
-		goto global_summary_err;
-	ret = parse_format_string(select_conf.global_list_format_arg,
-		global_list_atoms, &sfi->global_list_fi);
-	if (ret < 0)
-		goto global_list_err;
-	ret = parse_format_string(select_conf.user_list_format_arg,
-		user_list_atoms, &sfi->user_list_fi);
-	if (ret < 0)
-		goto user_list_err;
-	return 1;
-user_list_err:
-	free_format_info(sfi->global_list_fi);
-global_list_err:
-	free_format_info(sfi->global_summary_fi);
-global_summary_err:
-	free_format_info(sfi->user_summary_fi);
-	return ret;
+	return setup_format_string(fmt, fi);
 help:
 	line = select_conf.detailed_help_given?
 		select_args_info_detailed_help : select_args_info_help;
@@ -718,11 +1011,16 @@ help:
 	return 0;
 }
 
+/**
+ * Main function for select mode.
+ *
+ * \return Standard.
+ */
 int com_select(void)
 {
 	struct uid_range *admissible_uids = NULL;
-	struct select_format_info sfi;
 	int ret;
+	struct format_info *fi;
 	struct select_cmdline_parser_params params = {
 		.override = 1,
 		.initialize = 1,
@@ -731,10 +1029,16 @@ int com_select(void)
 		.print_errors = 1
 	};
 
-	select_cmdline_parser_init(&select_conf);
 	ret = parse_select_options(conf.select_options_arg, &params,
-		&admissible_uids, &sfi);
-	if (ret <= 0) /* do not run query if help was given */
-		return ret;
-	return run_select_query(admissible_uids, &sfi);
+		&admissible_uids, &fi);
+	if (ret > 0) {
+		ret = read_uid_file();
+		if (ret < 0)
+			goto out;
+		ret = run_select_query(admissible_uids, fi);
+		free_format_info(fi);
+	}
+out:
+	select_cmdline_parser_free(&select_conf);
+	return ret;
 }