summaryrefslogtreecommitdiff
path: root/sql
diff options
context:
space:
mode:
authorAlexey Botchkov <holyfoot@askmonty.org>2011-09-05 09:49:46 +0500
committerAlexey Botchkov <holyfoot@askmonty.org>2011-09-05 09:49:46 +0500
commita1315808b4a067fd82877ea1cfbd8867f9a8588a (patch)
tree1914e59fde6e1d390d294484e6490bd1aa63871d /sql
parent97eae1cd63e3bb05c98a4b050ec3abaf7925c382 (diff)
downloadmariadb-git-a1315808b4a067fd82877ea1cfbd8867f9a8588a.tar.gz
bug 839327 Crash in Gcalc_operation_reducer::end_couple with ST_UNION and MULTIPOLYGONs in 5.3-gis.
When edges of a polygon coicide, it can form an pike, that is turned into a line after an operation. In this case a former polygon point can be an end of a single line, and that case wasn't properly handled. per-file comments: mysql-test/r/gis-precise.result bug 839327 Crash in Gcalc_operation_reducer::end_couple with ST_UNION and MULTIPOLYGONs in 5.3-gis. test result updated. mysql-test/t/gis-precise.test bug 839327 Crash in Gcalc_operation_reducer::end_couple with ST_UNION and MULTIPOLYGONs in 5.3-gis. test case added. sql/gcalc_tools.cc bug 839327 Crash in Gcalc_operation_reducer::end_couple with ST_UNION and MULTIPOLYGONs in 5.3-gis. in the scev_two_ends case check if we have single line ending on a polygon node.
Diffstat (limited to 'sql')
-rw-r--r--sql/gcalc_tools.cc16
1 files changed, 13 insertions, 3 deletions
diff --git a/sql/gcalc_tools.cc b/sql/gcalc_tools.cc
index e502d54a160..3ee58646ca4 100644
--- a/sql/gcalc_tools.cc
+++ b/sql/gcalc_tools.cc
@@ -626,9 +626,19 @@ int Gcalc_operation_reducer::count_slice(Gcalc_scan_iterator *si)
}
case scev_two_ends:
{
- if (cur_t->enabled() &&
- end_couple(cur_t, cur_t->get_next(), events->pi))
- return 1;
+ if (cur_t->enabled() && cur_t->get_next()->enabled())
+ {
+ /* When two threads are ended here */
+ if (end_couple(cur_t, cur_t->get_next(), events->pi))
+ return 1;
+ }
+ else if (cur_t->enabled() || cur_t->get_next()->enabled())
+ {
+ /* Rare case when edges of a polygon coincide */
+ if (end_line(cur_t->enabled() ? cur_t : cur_t->get_next(),
+ events->pi, si))
+ return 1;
+ }
*cur_t_hook= cur_t->get_next()->get_next();
free_item(cur_t->next);
free_item(cur_t);