- Notifications
You must be signed in to change notification settings - Fork5
Commit76ce39e
committed
Prevent ExecInsert() and ExecUpdate() from scribbling on the result tuple
slot of the topmost plan node when a trigger returns a modified tuple.These appear to be the only places where a plan node's caller did nottreat the result slot as read-only, which is an assumption that nodeUniquemakes as of 8.1. Fixes trigger-vs-DISTINCT bug reported by Frank van Vugt.1 parent08ee64e commit76ce39e
File tree
3 files changed
+36
-10
lines changed- src
- backend/executor
- include/nodes
3 files changed
+36
-10
lines changedLines changed: 30 additions & 8 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
26 | 26 |
| |
27 | 27 |
| |
28 | 28 |
| |
29 |
| - | |
| 29 | + | |
30 | 30 |
| |
31 | 31 |
| |
32 | 32 |
| |
| |||
582 | 582 |
| |
583 | 583 |
| |
584 | 584 |
| |
585 |
| - | |
| 585 | + | |
| 586 | + | |
586 | 587 |
| |
587 | 588 |
| |
588 | 589 |
| |
| |||
591 | 592 |
| |
592 | 593 |
| |
593 | 594 |
| |
| 595 | + | |
| 596 | + | |
| 597 | + | |
594 | 598 |
| |
| 599 | + | |
| 600 | + | |
| 601 | + | |
| 602 | + | |
595 | 603 |
| |
596 | 604 |
| |
597 | 605 |
| |
| |||
1399 | 1407 |
| |
1400 | 1408 |
| |
1401 | 1409 |
| |
1402 |
| - | |
1403 |
| - | |
| 1410 | + | |
| 1411 | + | |
1404 | 1412 |
| |
1405 | 1413 |
| |
1406 | 1414 |
| |
1407 |
| - | |
| 1415 | + | |
| 1416 | + | |
| 1417 | + | |
| 1418 | + | |
| 1419 | + | |
| 1420 | + | |
| 1421 | + | |
| 1422 | + | |
1408 | 1423 |
| |
1409 | 1424 |
| |
1410 | 1425 |
| |
| |||
1600 | 1615 |
| |
1601 | 1616 |
| |
1602 | 1617 |
| |
1603 |
| - | |
1604 |
| - | |
| 1618 | + | |
| 1619 | + | |
1605 | 1620 |
| |
1606 | 1621 |
| |
1607 | 1622 |
| |
1608 |
| - | |
| 1623 | + | |
| 1624 | + | |
| 1625 | + | |
| 1626 | + | |
| 1627 | + | |
| 1628 | + | |
| 1629 | + | |
| 1630 | + | |
1609 | 1631 |
| |
1610 | 1632 |
| |
1611 | 1633 |
| |
|
Lines changed: 3 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
8 | 8 |
| |
9 | 9 |
| |
10 | 10 |
| |
11 |
| - | |
| 11 | + | |
12 | 12 |
| |
13 | 13 |
| |
14 | 14 |
| |
| |||
187 | 187 |
| |
188 | 188 |
| |
189 | 189 |
| |
| 190 | + | |
| 191 | + | |
190 | 192 |
| |
191 | 193 |
| |
192 | 194 |
| |
|
Lines changed: 3 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
7 | 7 |
| |
8 | 8 |
| |
9 | 9 |
| |
10 |
| - | |
| 10 | + | |
11 | 11 |
| |
12 | 12 |
| |
13 | 13 |
| |
| |||
304 | 304 |
| |
305 | 305 |
| |
306 | 306 |
| |
| 307 | + | |
| 308 | + | |
307 | 309 |
| |
308 | 310 |
| |
309 | 311 |
| |
|
0 commit comments
Comments
(0)