test: fix tmpdir value after setenv

Introduced in a2856c6e0f

set_test_env() modifies TMPDIR so we must not use getenv until after
that call.

Fixes #1736
This commit is contained in:
Peter Hutterer 2021-10-26 11:14:06 +10:00
parent c4d5c01429
commit cf3c0431c3

View file

@ -915,9 +915,7 @@ static void run_test(struct pwtest_context *ctx, struct pwtest_suite *c, struct
pid_t pw_daemon = 0;
int read_fds[_FD_LAST], write_fds[_FD_LAST];
int r;
const char *tmpdir = getenv("TMPDIR");
spa_assert_se(tmpdir != NULL);
const char *tmpdir;
if (t->result == PWTEST_SKIP) {
char *buf = pw_array_add(&t->logs[FD_LOG], 64);
@ -934,6 +932,8 @@ static void run_test(struct pwtest_context *ctx, struct pwtest_suite *c, struct
}
set_test_env(ctx, t);
tmpdir = getenv("TMPDIR");
spa_assert_se(tmpdir != NULL);
r = chdir(tmpdir);
if (r < 0) {
t->sig_or_errno = -errno;