Skip to content

Commit ff69230

Browse files
authored
Merge pull request #34 from postgrespro/PGPRO-5703
Fixed the work of installcheck
2 parents e820f9d + a9168b9 commit ff69230

File tree

2 files changed

+12
-4
lines changed

2 files changed

+12
-4
lines changed

Diff for: Makefile

+5-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,11 @@ ISOLATIONCHECKS = corner_cases
3030

3131
check: isolationcheck
3232

33-
installcheck: isolationcheck
33+
installcheck: submake-isolation
34+
$(MKDIR_P) isolation_output
35+
$(pg_isolation_regress_installcheck) \
36+
--outputdir=isolation_output \
37+
$(ISOLATIONCHECKS)
3438

3539
isolationcheck: | submake-isolation temp-install
3640
$(MKDIR_P) isolation_output

Diff for: pg_query_state.c

+7-3
Original file line numberDiff line numberDiff line change
@@ -541,14 +541,18 @@ pg_query_state(PG_FUNCTION_ARGS)
541541
break;
542542
}
543543
}
544-
pg_atomic_write_u32(&counterpart_userid->n_peers, 1);
545-
params->reqid = ++reqid;
546-
pg_write_barrier();
547544

548545
counterpart_user_id = GetRemoteBackendUserId(proc);
549546
if (!(superuser() || GetUserId() == counterpart_user_id))
547+
{
548+
UnlockShmem(&tag);
550549
ereport(ERROR, (errcode(ERRCODE_INSUFFICIENT_PRIVILEGE),
551550
errmsg("permission denied")));
551+
}
552+
553+
pg_atomic_write_u32(&counterpart_userid->n_peers, 1);
554+
params->reqid = ++reqid;
555+
pg_write_barrier();
552556

553557
bg_worker_procs = GetRemoteBackendWorkers(proc);
554558

0 commit comments

Comments
 (0)