summaryrefslogtreecommitdiff
path: root/sql
diff options
context:
space:
mode:
authorunknown <evgen@moonbone.local>2005-10-13 00:58:59 +0400
committerunknown <evgen@moonbone.local>2005-10-13 00:58:59 +0400
commita46e8e230e7aaa4d338364154af514ed342d05f3 (patch)
treeabbb9d4028e7ba336f6f3350a3b08d2e98a9f911 /sql
parentb455f7c575383787b27db9c8ae52fb9799a0a5e7 (diff)
downloadmariadb-git-a46e8e230e7aaa4d338364154af514ed342d05f3.tar.gz
select.test, sql_select.cc, sql_lex.cc, item.cc:
Bug #7672 after merge fix sql/item.cc: Bug #7672 after merge fix sql/sql_lex.cc: Bug #7672 after merge fix sql/sql_select.cc: Bug #7672 after merge fix mysql-test/t/select.test: Bug #7672 after merge fix
Diffstat (limited to 'sql')
-rw-r--r--sql/item.cc11
-rw-r--r--sql/sql_lex.cc3
-rw-r--r--sql/sql_select.cc5
3 files changed, 15 insertions, 4 deletions
diff --git a/sql/item.cc b/sql/item.cc
index 3c3a6d273fe..8df839baa5c 100644
--- a/sql/item.cc
+++ b/sql/item.cc
@@ -1760,10 +1760,15 @@ bool Item_field::fix_fields(THD *thd, TABLE_LIST *tables, Item **ref)
if ((tmp= find_field_in_tables(thd, this, tables, &where, 0)) ==
not_found_field)
{
- if (thd->lex.select_lex.is_item_list_lookup)
+ /* Look up in current select's item_list to find aliased fields */
+ if (thd->lex->current_select->is_item_list_lookup)
{
- Item** res= find_item_in_list(this, thd->lex.select_lex.item_list);
- if (res && *res && (*res)->type() == Item::FIELD_ITEM)
+ uint counter;
+ bool not_used;
+ Item** res= find_item_in_list(this, thd->lex->current_select->item_list,
+ &counter, REPORT_EXCEPT_NOT_FOUND,
+ &not_used);
+ if (res != not_found_item && (*res)->type() == Item::FIELD_ITEM)
{
set_field((*((Item_field**)res))->field);
return 0;
diff --git a/sql/sql_lex.cc b/sql/sql_lex.cc
index 47de2ff36c7..16641ad6dd5 100644
--- a/sql/sql_lex.cc
+++ b/sql/sql_lex.cc
@@ -160,7 +160,6 @@ void lex_start(THD *thd, uchar *buf,uint length)
lex->duplicates= DUP_ERROR;
lex->ignore= 0;
lex->proc_list.first= 0;
- lex->select_lex.is_item_list_lookup= 0;
}
void lex_end(LEX *lex)
@@ -1084,6 +1083,7 @@ void st_select_lex::init_query()
prep_where= 0;
subquery_in_having= explicit_limit= 0;
parsing_place= NO_MATTER;
+ is_item_list_lookup= 0;
}
void st_select_lex::init_select()
@@ -1110,6 +1110,7 @@ void st_select_lex::init_select()
select_limit= HA_POS_ERROR;
offset_limit= 0;
with_sum_func= 0;
+
}
/*
diff --git a/sql/sql_select.cc b/sql/sql_select.cc
index 0d9cab6a36b..f72d897e22d 100644
--- a/sql/sql_select.cc
+++ b/sql/sql_select.cc
@@ -8348,11 +8348,16 @@ find_order_in_list(THD *thd, Item **ref_pointer_array,
'it' reassigned in if condition because fix_field can change it.
*/
+ thd->lex->current_select->is_item_list_lookup= 1;
if (!it->fixed &&
(it->fix_fields(thd, tables, order->item) ||
(it= *order->item)->check_cols(1) ||
thd->is_fatal_error))
+ {
+ thd->lex->current_select->is_item_list_lookup= 0;
return 1; // Wrong field
+ }
+ thd->lex->current_select->is_item_list_lookup= 0;
uint el= all_fields.elements;
all_fields.push_front(it); // Add new field to field list
ref_pointer_array[el]= it;