diff options
| author | Joe Conway <mail@joeconway.com> | 2020-05-28 13:16:56 -0400 |
|---|---|---|
| committer | Joe Conway <mail@joeconway.com> | 2020-05-28 13:19:10 -0400 |
| commit | 3ccae5445cc82ce93c660a2b21a244a51dbe30a6 (patch) | |
| tree | 5ccc66315e7d9ee7d9a6f980fa5c60caf77efc3a /src/backend/utils/adt/oracle_compat.c | |
| parent | 7b009944c2525fcf8ee35db135c4bc560eb208f2 (diff) | |
| download | postgresql-3ccae5445cc82ce93c660a2b21a244a51dbe30a6.tar.gz postgresql-3ccae5445cc82ce93c660a2b21a244a51dbe30a6.zip | |
Add CHECK_FOR_INTERRUPTS() to the repeat() function
The repeat() function loops for potentially a long time without
ever checking for interrupts. This prevents, for example, a query
cancel from interrupting until the work is all done. Fix by
inserting a CHECK_FOR_INTERRUPTS() into the loop.
Backpatch to all supported versions.
Discussion: https://www.postgresql.org/message-id/flat/8692553c-7fe8-17d9-cbc1-7cddb758f4c6%40joeconway.com
Diffstat (limited to 'src/backend/utils/adt/oracle_compat.c')
| -rw-r--r-- | src/backend/utils/adt/oracle_compat.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/backend/utils/adt/oracle_compat.c b/src/backend/utils/adt/oracle_compat.c index 0fdfee58253..ecd8268b23b 100644 --- a/src/backend/utils/adt/oracle_compat.c +++ b/src/backend/utils/adt/oracle_compat.c @@ -19,7 +19,7 @@ #include "utils/builtins.h" #include "utils/formatting.h" #include "mb/pg_wchar.h" - +#include "miscadmin.h" static text *dotrim(const char *string, int stringlen, const char *set, int setlen, @@ -1062,6 +1062,7 @@ repeat(PG_FUNCTION_ARGS) { memcpy(cp, sp, slen); cp += slen; + CHECK_FOR_INTERRUPTS(); } PG_RETURN_TEXT_P(result); |