summaryrefslogtreecommitdiff
path: root/sql/sql_show.cc
diff options
context:
space:
mode:
authorunknown <sergefp@mysql.com>2007-07-04 17:11:56 +0400
committerunknown <sergefp@mysql.com>2007-07-04 17:11:56 +0400
commite2ccd8f8496836a253e7c1621c5d2781d17b18cd (patch)
treedac917ee7c1db0fd953d5ae6852ebc3dac10cda5 /sql/sql_show.cc
parent8d035c57f0c5c65ecf9652bd913b3d2f988c8f6e (diff)
downloadmariadb-git-e2ccd8f8496836a253e7c1621c5d2781d17b18cd.tar.gz
Backport from 5.2: Fix valgrind failure: Don't access item_func->arguments()
if item_func->argument_count()==0
Diffstat (limited to 'sql/sql_show.cc')
-rw-r--r--sql/sql_show.cc6
1 files changed, 2 insertions, 4 deletions
diff --git a/sql/sql_show.cc b/sql/sql_show.cc
index 902b298e423..dc1a7bc3bd3 100644
--- a/sql/sql_show.cc
+++ b/sql/sql_show.cc
@@ -1909,11 +1909,9 @@ bool uses_only_table_name_fields(Item *item, TABLE_LIST *table)
if (item->type() == Item::FUNC_ITEM)
{
Item_func *item_func= (Item_func*)item;
- Item **child;
- Item **item_end= (item_func->arguments()) + item_func->argument_count();
- for (child= item_func->arguments(); child != item_end; child++)
+ for (uint i=0; i<item_func->argument_count(); i++)
{
- if (!uses_only_table_name_fields(*child, table))
+ if (!uses_only_table_name_fields(item_func->arguments()[i], table))
return 0;
}
}