forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commita844c29
committed
Prevent memory leaks in parseRelOptions().
parseRelOptions() tended to leak memory in the caller's context. Mostof the time this doesn't really matter since the caller's context isat most query-lifespan, and the function won't be invoked very many times.However, when testing with CLOBBER_CACHE_RECURSIVELY, the same relcacheentry can get rebuilt a *lot* of times in one query, leading to significantintraquery memory bloat if it has any reloptions. Noted whileinvestigating a related report from Tomas Vondra.In passing, get rid of some Asserts that are redundant with the onedone by deconstruct_array().As with other patches to avoid leaks in CLOBBER_CACHE testing, it doesn'treally seem worth back-patching this.1 parentab8c84d commita844c29
1 file changed
+6
-9
lines changedLines changed: 6 additions & 9 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
620 | 620 |
| |
621 | 621 |
| |
622 | 622 |
| |
623 |
| - | |
624 |
| - | |
625 | 623 |
| |
626 | 624 |
| |
627 | 625 |
| |
| |||
777 | 775 |
| |
778 | 776 |
| |
779 | 777 |
| |
780 |
| - | |
781 |
| - | |
782 | 778 |
| |
783 | 779 |
| |
784 | 780 |
| |
| |||
912 | 908 |
| |
913 | 909 |
| |
914 | 910 |
| |
915 |
| - | |
| 911 | + | |
916 | 912 |
| |
917 | 913 |
| |
918 | 914 |
| |
919 |
| - | |
920 |
| - | |
921 |
| - | |
922 |
| - | |
923 | 915 |
| |
924 | 916 |
| |
925 | 917 |
| |
| |||
959 | 951 |
| |
960 | 952 |
| |
961 | 953 |
| |
| 954 | + | |
| 955 | + | |
| 956 | + | |
| 957 | + | |
| 958 | + | |
962 | 959 |
| |
963 | 960 |
| |
964 | 961 |
| |
|
0 commit comments
Comments
(0)