diff options
| author | Nathan Bossart <nathan@postgresql.org> | 2023-05-03 11:32:43 -0700 |
|---|---|---|
| committer | Nathan Bossart <nathan@postgresql.org> | 2023-05-04 16:24:48 -0700 |
| commit | 825ebc9848fdf9f229ba05a9aec3f58d13b17fd4 (patch) | |
| tree | 4b7f8a95f95f7d55500e1d5849f165f386243592 /src/backend/utils/adt/ruleutils.c | |
| parent | ccb479e76ac3032ef942f5d4f6a3ab742c2db03e (diff) | |
Move return statements out of PG_TRY blocks.
If we exit a PG_TRY block early via "continue", "break", "goto", or
"return", we'll skip unwinding its exception stack. This change
moves a couple of such "return" statements in PL/Python out of
PG_TRY blocks. This was introduced in d0aa965c0a and affects all
supported versions.
We might also be able to add compile-time checks to prevent
recurrence, but that is left as a future exercise.
Reported-by: Mikhail Gribkov, Xing Guo
Author: Xing Guo
Reviewed-by: Michael Paquier, Andres Freund, Tom Lane
Discussion: https://postgr.es/m/CAMEv5_v5Y%2B-D%3DCO1%2Bqoe16sAmgC4sbbQjz%2BUtcHmB6zcgS%2B5Ew%40mail.gmail.com
Discussion: https://postgr.es/m/CACpMh%2BCMsGMRKFzFMm3bYTzQmMU5nfEEoEDU2apJcc4hid36AQ%40mail.gmail.com
Backpatch-through: 11 (all supported versions)
Diffstat (limited to 'src/backend/utils/adt/ruleutils.c')
0 files changed, 0 insertions, 0 deletions