database/mariadb
1
0
Fork 0
mirror of https://github.com/MariaDB/server.git synced 2025-07-29 05:21:33 +03:00
Code Activity

MDEV-14786 Server crashes in Item_cond::transform on 2nd execution of SP querying from a view [fixes #436]

Browse Source
This commit is contained in:
Aleksey Midenkov
2017-12-29 12:28:37 +03:00
parent 19f0b512a9
commit 7069071d7d
3 changed files with 29 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
mysql-test/suite/versioning/r/select_sp.result
View File

@ -232,6 +232,19 @@ called_bad
called_bad
called_bad
# bad() is good.
# MDEV-14786 Server crashes in Item_cond::transform on 2nd execution of SP querying from a view [#436]
create or replace table t1 (called_bad int) with system versioning;
create or replace view v1 as select called_bad from t1 where called_bad < 5;
create or replace procedure bad() select called_bad from v1;
called_bad
called_bad
called_bad
called_bad
called_bad
called_bad
called_bad
called_bad
# bad() is good.
# wildcard expansion on hidden fields.
create or replace table t1(
A int
@ -255,7 +268,7 @@ create or replace table t1 (a int) with system versioning;
create or replace table t2 (a int) with system versioning;
insert into t1 values(1);
insert into t2 values(1);
create view v1 as select * from t2 inner join t1 using (a);
create or replace view v1 as select * from t2 inner join t1 using (a);
select * from v1;
a
1

12
mysql-test/suite/versioning/t/select_sp.test
View File

@ -125,6 +125,16 @@ drop procedure bad;
--enable_query_log
--echo # bad() is good.
--echo # MDEV-14786 Server crashes in Item_cond::transform on 2nd execution of SP querying from a view [#436]
create or replace table t1 (called_bad int) with system versioning;
create or replace view v1 as select called_bad from t1 where called_bad < 5;
create or replace procedure bad() select called_bad from v1;
--disable_query_log
call bad; call bad; call bad; call bad; call bad; call bad; call bad; call bad;
drop procedure bad;
--enable_query_log
--echo # bad() is good.
--echo # wildcard expansion on hidden fields.
create or replace table t1(
A int
@ -149,7 +159,7 @@ create or replace table t1 (a int) with system versioning;
create or replace table t2 (a int) with system versioning;
insert into t1 values(1);
insert into t2 values(1);
create view v1 as select * from t2 inner join t1 using (a);
create or replace view v1 as select * from t2 inner join t1 using (a);
select * from v1;
drop view v1;

4
sql/sql_select.cc
View File

@ -14904,6 +14904,8 @@ static COND* substitute_for_best_equal_field(THD *thd, JOIN_TAB *context_tab,
Item_equal *item_equal;
COND *org_cond= cond; // Return this in case of fatal error
Query_arena_stmt on_stmt_arena(thd);
if (cond->type() == Item::COND_ITEM)
{
List<Item> *cond_list= ((Item_cond*) cond)->argument_list();
@ -16025,6 +16027,8 @@ optimize_cond(JOIN *join, COND *conds,
THD *thd= join->thd;
DBUG_ENTER("optimize_cond");
Query_arena_stmt on_stmt_arena(thd);
if (!conds)
{
*cond_value= Item::COND_TRUE;