diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2015-12-14 16:44:40 -0300 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2015-12-14 16:44:40 -0300 |
| commit | b9a46f8ba667556b7a9b34c8c36f5d465f3fc7a2 (patch) | |
| tree | 397a002983ade4f61ee19088bfd68a709b3d62e2 | |
| parent | affae5e981d624735791ab897ff822118005ab44 (diff) | |
| download | postgresql-b9a46f8ba667556b7a9b34c8c36f5d465f3fc7a2.tar.gz postgresql-b9a46f8ba667556b7a9b34c8c36f5d465f3fc7a2.zip | |
Add missing CHECK_FOR_INTERRUPTS in lseg_inside_poly
Apparently, there are bugs in this code that cause it to loop endlessly.
That bug still needs more research, but in the meantime it's clear that
the loop is missing a check for interrupts so that it can be cancelled
timely.
Backpatch to 9.1 -- this has been missing since 49475aab8d0d.
| -rw-r--r-- | src/backend/utils/adt/geo_ops.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/backend/utils/adt/geo_ops.c b/src/backend/utils/adt/geo_ops.c index 6ef420d310e..77871b10ffb 100644 --- a/src/backend/utils/adt/geo_ops.c +++ b/src/backend/utils/adt/geo_ops.c @@ -20,6 +20,7 @@ #include <ctype.h> #include "libpq/pqformat.h" +#include "miscadmin.h" #include "utils/builtins.h" #include "utils/geo_decls.h" @@ -3894,6 +3895,8 @@ lseg_inside_poly(Point *a, Point *b, POLYGON *poly, int start) { Point *interpt; + CHECK_FOR_INTERRUPTS(); + s.p[1] = poly->p[i]; if (on_ps_internal(t.p, &s)) |