@@ -1013,6 +1013,8 @@ options_act_reversible(or_options_t *old_options, char **msg)
/* Setuid/setgid as appropriate */
if (options->User || options->Group) {
+ /* XXXX021 We should only do this the first time through, not on
+ * every setconf. */
if (switch_id(options->User, options->Group) != 0) {
/* No need to roll back, since you can't change the value. */
*msg = tor_strdup("Problem with User or Group value. "