Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit9124860

Browse files
committed
Add tests for error message generation in partition tuple routing
This adds extra tests for the error message generated for partitiontuple routing in the executor, using more than three levels ofpartitioning including partitioned tables with no partitions. Thesetests have been added to fixCVE-2019-10129 on REL_11_STABLE. HEAD hasno active bugs in this area, but it lacked coverage.Author: Michael PaquierReviewed-by: Noah MischSecurity:CVE-2019-10129
1 parenta090505 commit9124860

File tree

2 files changed

+22
-0
lines changed

2 files changed

+22
-0
lines changed

‎src/test/regress/expected/insert.out

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -637,13 +637,22 @@ create table mlparted5 partition of mlparted
637637
for values from (1, 40) to (1, 50) partition by range (c);
638638
create table mlparted5_ab partition of mlparted5
639639
for values from ('a') to ('c') partition by list (c);
640+
-- This partitioned table should remain with no partitions.
641+
create table mlparted5_cd partition of mlparted5
642+
for values from ('c') to ('e') partition by list (c);
640643
create table mlparted5_a partition of mlparted5_ab for values in ('a');
641644
create table mlparted5_b (d int, b int, c text, a int);
642645
alter table mlparted5_ab attach partition mlparted5_b for values in ('b');
643646
truncate mlparted;
644647
insert into mlparted values (1, 2, 'a', 1);
645648
insert into mlparted values (1, 40, 'a', 1); -- goes to mlparted5_a
646649
insert into mlparted values (1, 45, 'b', 1); -- goes to mlparted5_b
650+
insert into mlparted values (1, 45, 'c', 1); -- goes to mlparted5_cd, fails
651+
ERROR: no partition of relation "mlparted5_cd" found for row
652+
DETAIL: Partition key of the failing row contains (c) = (c).
653+
insert into mlparted values (1, 45, 'f', 1); -- goes to mlparted5, fails
654+
ERROR: no partition of relation "mlparted5" found for row
655+
DETAIL: Partition key of the failing row contains (c) = (f).
647656
select tableoid::regclass, * from mlparted order by a, b, c, d;
648657
tableoid | a | b | c | d
649658
-------------+---+----+---+---
@@ -660,6 +669,12 @@ alter table mlparted drop e;
660669
insert into mlparted values (1, 2, 'a', 1);
661670
insert into mlparted values (1, 40, 'a', 1); -- goes to mlparted5_a
662671
insert into mlparted values (1, 45, 'b', 1); -- goes to mlparted5_b
672+
insert into mlparted values (1, 45, 'c', 1); -- goes to mlparted5_cd, fails
673+
ERROR: no partition of relation "mlparted5_cd" found for row
674+
DETAIL: Partition key of the failing row contains (c) = (c).
675+
insert into mlparted values (1, 45, 'f', 1); -- goes to mlparted5, fails
676+
ERROR: no partition of relation "mlparted5" found for row
677+
DETAIL: Partition key of the failing row contains (c) = (f).
663678
select tableoid::regclass, * from mlparted order by a, b, c, d;
664679
tableoid | a | b | c | d
665680
-------------+---+----+---+---

‎src/test/regress/sql/insert.sql

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -409,13 +409,18 @@ create table mlparted5 partition of mlparted
409409
forvaluesfrom (1,40) to (1,50) partition by range (c);
410410
createtablemlparted5_ab partition of mlparted5
411411
forvaluesfrom ('a') to ('c') partition by list (c);
412+
-- This partitioned table should remain with no partitions.
413+
createtablemlparted5_cd partition of mlparted5
414+
forvaluesfrom ('c') to ('e') partition by list (c);
412415
createtablemlparted5_a partition of mlparted5_ab forvaluesin ('a');
413416
createtablemlparted5_b (dint, bint, ctext, aint);
414417
altertable mlparted5_ab attach partition mlparted5_b forvaluesin ('b');
415418
truncate mlparted;
416419
insert into mlpartedvalues (1,2,'a',1);
417420
insert into mlpartedvalues (1,40,'a',1);-- goes to mlparted5_a
418421
insert into mlpartedvalues (1,45,'b',1);-- goes to mlparted5_b
422+
insert into mlpartedvalues (1,45,'c',1);-- goes to mlparted5_cd, fails
423+
insert into mlpartedvalues (1,45,'f',1);-- goes to mlparted5, fails
419424
select tableoid::regclass,*from mlpartedorder by a, b, c, d;
420425
altertable mlparted drop d;
421426
truncate mlparted;
@@ -425,6 +430,8 @@ alter table mlparted drop e;
425430
insert into mlpartedvalues (1,2,'a',1);
426431
insert into mlpartedvalues (1,40,'a',1);-- goes to mlparted5_a
427432
insert into mlpartedvalues (1,45,'b',1);-- goes to mlparted5_b
433+
insert into mlpartedvalues (1,45,'c',1);-- goes to mlparted5_cd, fails
434+
insert into mlpartedvalues (1,45,'f',1);-- goes to mlparted5, fails
428435
select tableoid::regclass,*from mlpartedorder by a, b, c, d;
429436
altertable mlparted drop d;
430437
droptable mlparted5;

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp