Re: [PATCH] Added support for dropping privileges to git-daemon.

Tilman Sauerbeck wrote:

It seems "if (user)" would be sufficient here.

For simplicities sake I'd actually suggest allowing group==NULL. So change this test to be

	if (group && !user)
		die("--group supplied without --user");

Then in drop_privileges() do something like:

	struct passwd *p;
	gid_t gid;

	p = getpwnam(user);
	if (!p)
		die("user not found - %s", user);
	if (group == NULL)
		gid = p->pw_gid;
	else {
		struct group *g = getgrnam(group);
		if (!g)
			die("group not found - %s", group);
		gid = g->gr_gid;
	}

Since usually you want to use the same gid that is normally associated with that pid, this just makes things a little easier on the user

-Mitch