- Notifications
You must be signed in to change notification settings - Fork5
Commit90857b4
committed
Preserve caller's memory context in ProcessCompletedNotifies().
This is necessary to avoid long-term memory leakage, because the main loopin PostgresMain expects to be executing in MessageContext, and hence is abit sloppy about freeing stuff that is only needed for the duration ofprocessing the current client message. The known case of an actual leakis when encoding conversion has to be done on the incoming command string,but there might be others. Per report from Per-Olov Esgard.Back-patch to 9.0, where the bug was introduced by the LISTEN/NOTIFYrewrite.1 parent336db7e commit90857b4
1 file changed
+9
-0
lines changedLines changed: 9 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1090 | 1090 |
| |
1091 | 1091 |
| |
1092 | 1092 |
| |
| 1093 | + | |
1093 | 1094 |
| |
1094 | 1095 |
| |
1095 | 1096 |
| |
| |||
1103 | 1104 |
| |
1104 | 1105 |
| |
1105 | 1106 |
| |
| 1107 | + | |
| 1108 | + | |
| 1109 | + | |
| 1110 | + | |
| 1111 | + | |
| 1112 | + | |
1106 | 1113 |
| |
1107 | 1114 |
| |
1108 | 1115 |
| |
| |||
1135 | 1142 |
| |
1136 | 1143 |
| |
1137 | 1144 |
| |
| 1145 | + | |
| 1146 | + | |
1138 | 1147 |
| |
1139 | 1148 |
| |
1140 | 1149 |
| |
|
0 commit comments
Comments
(0)