- Notifications
You must be signed in to change notification settings - Fork5
Commit6ea8f49
committed
Fix SysCacheGetAttr() to handle the case where the specified syscache has not
been initialized yet. This can happen because there are code paths that callSysCacheGetAttr() on a tuple originally fetched from a different syscache(hopefully on the same catalog) than the one specified in the call. Itdoesn't seem useful or robust to try to prevent that from happening, so justimprove the function to cope instead. Per bug#2678 from Jeff Trout. Thespecific example shown by Jeff is new in 8.1, but to be on the safe sideI'm backpatching 8.0 as well. We could patch 7.x similarly but I thinkthat's probably overkill, given the lack of evidence of old bugs of this ilk.1 parentb9b4f10 commit6ea8f49
File tree
3 files changed
+27
-19
lines changed- src
- backend/utils/cache
- include/utils
3 files changed
+27
-19
lines changedLines changed: 10 additions & 8 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
8 | 8 |
| |
9 | 9 |
| |
10 | 10 |
| |
11 |
| - | |
| 11 | + | |
12 | 12 |
| |
13 | 13 |
| |
14 | 14 |
| |
| |||
976 | 976 |
| |
977 | 977 |
| |
978 | 978 |
| |
979 |
| - | |
| 979 | + | |
980 | 980 |
| |
981 | 981 |
| |
982 | 982 |
| |
| |||
991 | 991 |
| |
992 | 992 |
| |
993 | 993 |
| |
994 |
| - | |
995 |
| - | |
996 |
| - | |
997 |
| - | |
| 994 | + | |
| 995 | + | |
| 996 | + | |
| 997 | + | |
| 998 | + | |
998 | 999 |
| |
999 | 1000 |
| |
1000 |
| - | |
| 1001 | + | |
1001 | 1002 |
| |
1002 | 1003 |
| |
1003 | 1004 |
| |
1004 | 1005 |
| |
1005 |
| - | |
| 1006 | + | |
| 1007 | + | |
1006 | 1008 |
| |
1007 | 1009 |
| |
1008 | 1010 |
| |
|
Lines changed: 15 additions & 9 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
8 | 8 |
| |
9 | 9 |
| |
10 | 10 |
| |
11 |
| - | |
| 11 | + | |
12 | 12 |
| |
13 | 13 |
| |
14 | 14 |
| |
| |||
576 | 576 |
| |
577 | 577 |
| |
578 | 578 |
| |
579 |
| - | |
| 579 | + | |
580 | 580 |
| |
581 | 581 |
| |
582 | 582 |
| |
| |||
773 | 773 |
| |
774 | 774 |
| |
775 | 775 |
| |
| 776 | + | |
| 777 | + | |
| 778 | + | |
776 | 779 |
| |
777 | 780 |
| |
778 | 781 |
| |
| |||
781 | 784 |
| |
782 | 785 |
| |
783 | 786 |
| |
784 |
| - | |
785 |
| - | |
786 |
| - | |
| 787 | + | |
| 788 | + | |
| 789 | + | |
787 | 790 |
| |
788 |
| - | |
| 791 | + | |
| 792 | + | |
789 | 793 |
| |
790 |
| - | |
791 |
| - | |
792 |
| - | |
| 794 | + | |
| 795 | + | |
| 796 | + | |
| 797 | + | |
| 798 | + | |
793 | 799 |
| |
794 | 800 |
| |
795 | 801 |
| |
|
Lines changed: 2 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
13 | 13 |
| |
14 | 14 |
| |
15 | 15 |
| |
16 |
| - | |
| 16 | + | |
17 | 17 |
| |
18 | 18 |
| |
19 | 19 |
| |
| |||
166 | 166 |
| |
167 | 167 |
| |
168 | 168 |
| |
169 |
| - | |
| 169 | + | |
170 | 170 |
| |
171 | 171 |
| |
172 | 172 |
| |
|
0 commit comments
Comments
(0)