doveadm: Getting user from USER environment wasn't done in the right …

…place.

doveadm_mail_cmdline_init() is also called when parsing commands from
doveadm-server. The USER environment is supposed to be read only from
doveadm command line.

src/doveadm/doveadm-mail.c

@@ -557,7 +557,6 @@ doveadm_mail_cmdline_init(const struct doveadm_mail_cmd *cmd)
 	ctx->service_flags |= MAIL_STORAGE_SERVICE_FLAG_NO_LOG_INIT;
 	if (doveadm_debug)
 		ctx->service_flags |= MAIL_STORAGE_SERVICE_FLAG_DEBUG;
-	ctx->cur_username = getenv("USER");
 	return ctx;
 }
 
@@ -933,7 +932,7 @@ doveadm_cmd_ver2_to_mail_cmd_wrapper(struct doveadm_cmd_context *cctx)
 	};
 
 	mctx = doveadm_mail_cmdline_init(&mail_cmd);
-
+	mctx->cur_username = cctx->username;
 	mctx->iterate_all_users = FALSE;
 	wildcard_user = NULL;
 	p_array_init(&pargv, mctx->pool, 8);

src/doveadm/doveadm.c

@@ -291,6 +291,7 @@ int main(int argc, char *argv[])
 
 	memset(&cctx,0,sizeof(cctx));
 	cctx.cli = TRUE;
+	cctx.username = getenv("USER");
 
 	i_set_failure_exit_callback(failure_exit_callback);
 	doveadm_dsync_main(&argc, &argv);