mirror of
https://github.com/MariaDB/server.git
synced 2025-07-29 05:21:33 +03:00
Merge bk-internal.mysql.com:/home/bk/mysql-5.1-new-maint
into recycle.(none):/src/bug23491/my51-bug23491 mysql-test/t/mysqldump.test: Auto merged sql/item_create.cc: Auto merged sql/item_func.cc: Auto merged sql/sp.cc: Auto merged sql/sp_head.h: Auto merged sql/sql_yacc.yy: Auto merged mysql-test/r/mysqldump.result: SCCS merged
This commit is contained in:
@ -706,7 +706,7 @@ Warnings:
|
||||
Warning 1356 View 'test.v2' references invalid table(s) or column(s) or function(s) or definer/invoker of view lack rights to use them
|
||||
show create table v3;
|
||||
View Create View
|
||||
v3 CREATE ALGORITHM=UNDEFINED DEFINER=`root`@`localhost` SQL SECURITY DEFINER VIEW `v3` AS select `test`.`sub1`(1) AS `c`
|
||||
v3 CREATE ALGORITHM=UNDEFINED DEFINER=`root`@`localhost` SQL SECURITY DEFINER VIEW `v3` AS select `sub1`(1) AS `c`
|
||||
Warnings:
|
||||
Warning 1356 View 'test.v3' references invalid table(s) or column(s) or function(s) or definer/invoker of view lack rights to use them
|
||||
drop view v2;
|
||||
|
@ -3205,6 +3205,29 @@ INSERT INTO t1 VALUES(1,0xff00fef0);
|
||||
</mysqldump>
|
||||
DROP TABLE t1;
|
||||
#
|
||||
# Bug #23491: MySQLDump prefix function call in a view by database name
|
||||
#
|
||||
create database bug23491_original;
|
||||
create database bug23491_restore;
|
||||
use bug23491_original;
|
||||
create table t1 (c1 int);
|
||||
create view v1 as select * from t1;
|
||||
create procedure p1() select 1;
|
||||
create function f1() returns int return 1;
|
||||
create view v2 as select f1();
|
||||
create function f2() returns int return f1();
|
||||
create view v3 as select bug23491_original.f1();
|
||||
use bug23491_restore;
|
||||
show create view bug23491_restore.v2;
|
||||
View Create View
|
||||
v2 CREATE ALGORITHM=UNDEFINED DEFINER=`root`@`localhost` SQL SECURITY DEFINER VIEW `v2` AS select `f1`() AS `f1()`
|
||||
show create view bug23491_restore.v3;
|
||||
View Create View
|
||||
v3 CREATE ALGORITHM=UNDEFINED DEFINER=`root`@`localhost` SQL SECURITY DEFINER VIEW `v3` AS select `bug23491_original`.`f1`() AS `bug23491_original.f1()`
|
||||
drop database bug23491_original;
|
||||
drop database bug23491_restore;
|
||||
use test;
|
||||
#
|
||||
# Bug#26346: stack + buffer overrun in mysqldump
|
||||
#
|
||||
CREATE TABLE t1(a int);
|
||||
|
@ -187,7 +187,7 @@ Pos Instruction
|
||||
32 set v_dig@4 (v_dig@4 + 1)
|
||||
33 stmt 4 "update sudoku_work set dig = v_dig wh..."
|
||||
34 set v_tcounter@6 (v_tcounter@6 + 1)
|
||||
35 jump_if_not 37(37) (not(`test`.`sudoku_digit_ok`(v_row@7,v_col@8,v_dig@4)))
|
||||
35 jump_if_not 37(37) (not(`sudoku_digit_ok`(v_row@7,v_col@8,v_dig@4)))
|
||||
36 jump 15
|
||||
37 set v_i@3 (v_i@3 + 1)
|
||||
38 jump 15
|
||||
|
@ -159,7 +159,7 @@ EXPLAIN EXTENDED SELECT myfunc_int(fn(MIN(b))) as c FROM t1 GROUP BY a;
|
||||
id select_type table type possible_keys key key_len ref rows filtered Extra
|
||||
1 SIMPLE t1 ALL NULL NULL NULL NULL 2 100.00 Using temporary; Using filesort
|
||||
Warnings:
|
||||
Note 1003 select myfunc_int(`test`.`fn`(min(`test`.`t1`.`b`)) AS `fn(MIN(b))`) AS `c` from `test`.`t1` group by `test`.`t1`.`a`
|
||||
Note 1003 select myfunc_int(`fn`(min(`test`.`t1`.`b`)) AS `fn(MIN(b))`) AS `c` from `test`.`t1` group by `test`.`t1`.`a`
|
||||
EXPLAIN EXTENDED SELECT myfunc_int(test.fn(MIN(b))) as c FROM t1 GROUP BY a;
|
||||
id select_type table type possible_keys key key_len ref rows filtered Extra
|
||||
1 SIMPLE t1 ALL NULL NULL NULL NULL 2 100.00 Using temporary; Using filesort
|
||||
|
@ -1465,6 +1465,38 @@ CREATE TABLE t1 (a int) ENGINE=merge UNION=(t2, t3);
|
||||
--exec $MYSQL_DUMP --skip-comments -c test
|
||||
DROP TABLE t1, t2, t3;
|
||||
|
||||
--echo #
|
||||
--echo # Bug #23491: MySQLDump prefix function call in a view by database name
|
||||
--echo #
|
||||
|
||||
# Setup
|
||||
create database bug23491_original;
|
||||
create database bug23491_restore;
|
||||
use bug23491_original;
|
||||
create table t1 (c1 int);
|
||||
create view v1 as select * from t1;
|
||||
create procedure p1() select 1;
|
||||
create function f1() returns int return 1;
|
||||
create view v2 as select f1();
|
||||
create function f2() returns int return f1();
|
||||
create view v3 as select bug23491_original.f1();
|
||||
|
||||
# Backup.
|
||||
--exec $MYSQL_DUMP --skip-comments -uroot --opt --routines bug23491_original > $MYSQLTEST_VARDIR/tmp/bug23491_backup.sql;
|
||||
|
||||
# Restore.
|
||||
--exec $MYSQL bug23491_restore < $MYSQLTEST_VARDIR/tmp/bug23491_backup.sql;
|
||||
|
||||
# Verify
|
||||
use bug23491_restore;
|
||||
show create view bug23491_restore.v2;
|
||||
show create view bug23491_restore.v3;
|
||||
|
||||
# Cleanup
|
||||
drop database bug23491_original;
|
||||
drop database bug23491_restore;
|
||||
use test;
|
||||
|
||||
--echo #
|
||||
--echo # End of 5.0 tests
|
||||
--echo #
|
||||
|
@ -167,7 +167,7 @@ class Create_sp_func : public Create_qfunc
|
||||
{
|
||||
public:
|
||||
virtual Item* create(THD *thd, LEX_STRING db, LEX_STRING name,
|
||||
List<Item> *item_list);
|
||||
bool use_explicit_name, List<Item> *item_list);
|
||||
|
||||
static Create_sp_func s_singleton;
|
||||
|
||||
@ -2316,7 +2316,7 @@ Create_qfunc::create(THD *thd, LEX_STRING name, List<Item> *item_list)
|
||||
if (thd->copy_db_to(&db.str, &db.length))
|
||||
return NULL;
|
||||
|
||||
return create(thd, db, name, item_list);
|
||||
return create(thd, db, name, false, item_list);
|
||||
}
|
||||
|
||||
|
||||
@ -2433,7 +2433,7 @@ Create_sp_func Create_sp_func::s_singleton;
|
||||
|
||||
Item*
|
||||
Create_sp_func::create(THD *thd, LEX_STRING db, LEX_STRING name,
|
||||
List<Item> *item_list)
|
||||
bool use_explicit_name, List<Item> *item_list)
|
||||
{
|
||||
int arg_count= 0;
|
||||
Item *func= NULL;
|
||||
@ -2458,7 +2458,7 @@ Create_sp_func::create(THD *thd, LEX_STRING db, LEX_STRING name,
|
||||
if (item_list != NULL)
|
||||
arg_count= item_list->elements;
|
||||
|
||||
qname= new (thd->mem_root) sp_name(db, name);
|
||||
qname= new (thd->mem_root) sp_name(db, name, use_explicit_name);
|
||||
qname->init_qname(thd);
|
||||
sp_add_used_routine(lex, thd, qname, TYPE_ENUM_FUNCTION);
|
||||
|
||||
|
@ -87,11 +87,12 @@ public:
|
||||
@param thd The current thread
|
||||
@param db The database name
|
||||
@param name The function name
|
||||
@param use_explicit_name Should the function be represented as 'db.name'?
|
||||
@param item_list The list of arguments to the function, can be NULL
|
||||
@return An item representing the parsed function call
|
||||
*/
|
||||
virtual Item* create(THD *thd, LEX_STRING db, LEX_STRING name,
|
||||
List<Item> *item_list) = 0;
|
||||
bool use_explicit_name, List<Item> *item_list) = 0;
|
||||
|
||||
protected:
|
||||
/** Constructor. */
|
||||
|
@ -5030,7 +5030,7 @@ Item_func_sp::func_name() const
|
||||
{
|
||||
THD *thd= current_thd;
|
||||
/* Calculate length to avoid reallocation of string for sure */
|
||||
uint len= ((m_name->m_db.length +
|
||||
uint len= ((m_name->m_explicit_name ? m_name->m_db.length : 0 +
|
||||
m_name->m_name.length)*2 + //characters*quoting
|
||||
2 + // ` and `
|
||||
1 + // .
|
||||
@ -5040,8 +5040,11 @@ Item_func_sp::func_name() const
|
||||
system_charset_info);
|
||||
|
||||
qname.length(0);
|
||||
if (m_name->m_explicit_name)
|
||||
{
|
||||
append_identifier(thd, &qname, m_name->m_db.str, m_name->m_db.length);
|
||||
qname.append('.');
|
||||
}
|
||||
append_identifier(thd, &qname, m_name->m_name.str, m_name->m_name.length);
|
||||
return qname.ptr();
|
||||
}
|
||||
|
@ -1041,7 +1041,7 @@ sp_exist_routines(THD *thd, TABLE_LIST *routines, bool any, bool no_error)
|
||||
lex_name.length= strlen(routine->table_name);
|
||||
lex_db.str= thd->strmake(routine->db, lex_db.length);
|
||||
lex_name.str= thd->strmake(routine->table_name, lex_name.length);
|
||||
name= new sp_name(lex_db, lex_name);
|
||||
name= new sp_name(lex_db, lex_name, true);
|
||||
name->init_qname(thd);
|
||||
sp_object_found= sp_find_routine(thd, TYPE_ENUM_PROCEDURE, name,
|
||||
&thd->sp_proc_cache, FALSE) != NULL ||
|
||||
|
@ -59,9 +59,10 @@ public:
|
||||
calling set_routine_type().
|
||||
*/
|
||||
LEX_STRING m_sroutines_key;
|
||||
bool m_explicit_name; /**< Prepend the db name? */
|
||||
|
||||
sp_name(LEX_STRING db, LEX_STRING name)
|
||||
: m_db(db), m_name(name)
|
||||
sp_name(LEX_STRING db, LEX_STRING name, bool use_explicit_name)
|
||||
: m_db(db), m_name(name), m_explicit_name(use_explicit_name)
|
||||
{
|
||||
m_qname.str= m_sroutines_key.str= 0;
|
||||
m_qname.length= m_sroutines_key.length= 0;
|
||||
@ -79,6 +80,7 @@ public:
|
||||
m_name.length= m_qname.length= key_len - 1;
|
||||
m_db.str= 0;
|
||||
m_db.length= 0;
|
||||
m_explicit_name= false;
|
||||
}
|
||||
|
||||
// Init. the qualified name from the db and name.
|
||||
|
@ -1931,7 +1931,7 @@ sp_name:
|
||||
my_error(ER_SP_WRONG_NAME, MYF(0), $3.str);
|
||||
MYSQL_YYABORT;
|
||||
}
|
||||
$$= new sp_name($1, $3);
|
||||
$$= new sp_name($1, $3, true);
|
||||
$$->init_qname(YYTHD);
|
||||
}
|
||||
| ident
|
||||
@ -1945,7 +1945,7 @@ sp_name:
|
||||
}
|
||||
if (thd->copy_db_to(&db.str, &db.length))
|
||||
MYSQL_YYABORT;
|
||||
$$= new sp_name(db, $1);
|
||||
$$= new sp_name(db, $1, false);
|
||||
if ($$)
|
||||
$$->init_qname(YYTHD);
|
||||
}
|
||||
@ -6923,7 +6923,7 @@ function_call_generic:
|
||||
|
||||
builder= find_qualified_function_builder(thd);
|
||||
DBUG_ASSERT(builder);
|
||||
item= builder->create(thd, $1, $3, $5);
|
||||
item= builder->create(thd, $1, $3, true, $5);
|
||||
|
||||
if (! ($$= item))
|
||||
{
|
||||
|
Reference in New Issue
Block a user