diff options
| author | Noah Misch | 2015-01-08 03:33:58 +0000 |
|---|---|---|
| committer | Noah Misch | 2015-01-08 03:33:58 +0000 |
| commit | e415b469b33ba328765e39fd62edcd28f30d9c3c (patch) | |
| tree | 836c785c7790fd0716092a2844b821f34071b938 /src/test | |
| parent | 39f2594ba54657f3fa8d0fe230447e673bcba2a4 (diff) | |
Reject ANALYZE commands during VACUUM FULL or another ANALYZE.
vacuum()'s static variable handling makes it non-reentrant; an ensuing
null pointer deference crashed the backend. Back-patch to 9.0 (all
supported versions).
Diffstat (limited to 'src/test')
| -rw-r--r-- | src/test/regress/expected/vacuum.out | 14 | ||||
| -rw-r--r-- | src/test/regress/sql/vacuum.sql | 9 |
2 files changed, 21 insertions, 2 deletions
diff --git a/src/test/regress/expected/vacuum.out b/src/test/regress/expected/vacuum.out index 190043c2558..4f2f5e228f4 100644 --- a/src/test/regress/expected/vacuum.out +++ b/src/test/regress/expected/vacuum.out @@ -60,12 +60,24 @@ VACUUM (FULL, FREEZE) vactst; VACUUM (ANALYZE, FULL) vactst; CREATE TABLE vaccluster (i INT PRIMARY KEY); ALTER TABLE vaccluster CLUSTER ON vaccluster_pkey; -INSERT INTO vaccluster SELECT * FROM vactst; CLUSTER vaccluster; +CREATE FUNCTION do_analyze() RETURNS VOID VOLATILE LANGUAGE SQL + AS 'ANALYZE pg_am'; +CREATE FUNCTION wrap_do_analyze(c INT) RETURNS INT IMMUTABLE LANGUAGE SQL + AS 'SELECT $1 FROM do_analyze()'; +CREATE INDEX ON vactst(wrap_do_analyze(i)); +INSERT INTO vactst VALUES (1), (2); +ANALYZE vactst; +ERROR: ANALYZE cannot be executed from VACUUM or ANALYZE +CONTEXT: SQL function "do_analyze" statement 1 +SQL function "wrap_do_analyze" statement 1 VACUUM FULL pg_am; VACUUM FULL pg_class; VACUUM FULL pg_database; VACUUM FULL vaccluster; VACUUM FULL vactst; +ERROR: ANALYZE cannot be executed from VACUUM or ANALYZE +CONTEXT: SQL function "do_analyze" statement 1 +SQL function "wrap_do_analyze" statement 1 DROP TABLE vaccluster; DROP TABLE vactst; diff --git a/src/test/regress/sql/vacuum.sql b/src/test/regress/sql/vacuum.sql index 30551ad1f27..4b624fe379e 100644 --- a/src/test/regress/sql/vacuum.sql +++ b/src/test/regress/sql/vacuum.sql @@ -44,9 +44,16 @@ VACUUM (ANALYZE, FULL) vactst; CREATE TABLE vaccluster (i INT PRIMARY KEY); ALTER TABLE vaccluster CLUSTER ON vaccluster_pkey; -INSERT INTO vaccluster SELECT * FROM vactst; CLUSTER vaccluster; +CREATE FUNCTION do_analyze() RETURNS VOID VOLATILE LANGUAGE SQL + AS 'ANALYZE pg_am'; +CREATE FUNCTION wrap_do_analyze(c INT) RETURNS INT IMMUTABLE LANGUAGE SQL + AS 'SELECT $1 FROM do_analyze()'; +CREATE INDEX ON vactst(wrap_do_analyze(i)); +INSERT INTO vactst VALUES (1), (2); +ANALYZE vactst; + VACUUM FULL pg_am; VACUUM FULL pg_class; VACUUM FULL pg_database; |