diff options
author | unknown <sergefp@mysql.com> | 2007-07-04 17:11:56 +0400 |
---|---|---|
committer | unknown <sergefp@mysql.com> | 2007-07-04 17:11:56 +0400 |
commit | e2ccd8f8496836a253e7c1621c5d2781d17b18cd (patch) | |
tree | dac917ee7c1db0fd953d5ae6852ebc3dac10cda5 | |
parent | 8d035c57f0c5c65ecf9652bd913b3d2f988c8f6e (diff) | |
download | mariadb-git-e2ccd8f8496836a253e7c1621c5d2781d17b18cd.tar.gz |
Backport from 5.2: Fix valgrind failure: Don't access item_func->arguments()
if item_func->argument_count()==0
-rw-r--r-- | sql/sql_show.cc | 6 |
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; } } |