diff options
author | Georgi Kodinov <Georgi.Kodinov@Oracle.com> | 2010-08-20 12:09:17 +0300 |
---|---|---|
committer | Georgi Kodinov <Georgi.Kodinov@Oracle.com> | 2010-08-20 12:09:17 +0300 |
commit | 7d3a9b4cf68475f24b24f64ae43a205d6016b83e (patch) | |
tree | d7d20b0fb3455280191417a9653848ff8d25442e /sql/item_sum.cc | |
parent | dd9329761ae48e3d8518f882d1ca12737a4165d0 (diff) | |
parent | 69c280506808e434e20669d838724d43b675540c (diff) | |
download | mariadb-git-7d3a9b4cf68475f24b24f64ae43a205d6016b83e.tar.gz |
merge
Diffstat (limited to 'sql/item_sum.cc')
-rw-r--r-- | sql/item_sum.cc | 19 |
1 files changed, 18 insertions, 1 deletions
diff --git a/sql/item_sum.cc b/sql/item_sum.cc index f92bde9ce87..25b3bd5d91d 100644 --- a/sql/item_sum.cc +++ b/sql/item_sum.cc @@ -3035,7 +3035,6 @@ Item_func_group_concat::Item_func_group_concat(THD *thd, tree(item->tree), unique_filter(item->unique_filter), table(item->table), - order(item->order), context(item->context), arg_count_order(item->arg_count_order), arg_count_field(item->arg_count_field), @@ -3048,6 +3047,24 @@ Item_func_group_concat::Item_func_group_concat(THD *thd, { quick_group= item->quick_group; result.set_charset(collation.collation); + + /* + Since the ORDER structures pointed to by the elements of the 'order' array + may be modified in find_order_in_list() called from + Item_func_group_concat::setup(), create a copy of those structures so that + such modifications done in this object would not have any effect on the + object being copied. + */ + ORDER *tmp; + if (!(order= (ORDER **) thd->alloc(sizeof(ORDER *) * arg_count_order + + sizeof(ORDER) * arg_count_order))) + return; + tmp= (ORDER *)(order + arg_count_order); + for (uint i= 0; i < arg_count_order; i++, tmp++) + { + memcpy(tmp, item->order[i], sizeof(ORDER)); + order[i]= tmp; + } } |