Fix -nouser/-nogroup error handling

The proper way to check for nonexistent users/groups is to set errno to
0 before the get{grg,pwu}id() call, and check it afterwards.

On doing this, it becomes obvious that the call can fail if bftw() is
using all the available FDs, so give them some ephemeral FDs.  It would
be ideal to read the user/group table only once, but this fixes the bug
for now.

1 files changed, 2 insertions, 0 deletions

diff --git a/parse.c b/parse.c
index 8fe65b3..8fa9e79 100644
--- a/parse.c
+++ b/parse.c
@@ -1658,6 +1658,7 @@ static struct expr *parse_nogroup(struct parser_state *state, int arg1, int arg2
 	struct expr *expr = parse_nullary_test(state, eval_nogroup);
 	if (expr) {
 		expr->cost = 9000.0;
+		expr->ephemeral_fds = 1;
 	}
 	return expr;
 }
@@ -1688,6 +1689,7 @@ static struct expr *parse_nouser(struct parser_state *state, int arg1, int arg2)
 	struct expr *expr = parse_nullary_test(state, eval_nouser);
 	if (expr) {
 		expr->cost = 9000.0;
+		expr->ephemeral_fds = 1;
 	}
 	return expr;
 }