Crash in st_table::mark_virtual_columns_for_write with views

Bug #794005 reported by Philip Stoev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MariaDB
Fix Released
Critical
Oleksandr "Sanja" Byelkin

Bug Description

Reproducible on maria-5.2 , maria-5.3, maria-5.3-wl106. Not reproducible on mysql-5.5.

backtrace:

#4 <signal handler called>
#5 0x082e5a55 in st_table::mark_virtual_columns_for_write (this=0xae72a498, insert_fl=false) at table.cc:5163
#6 0x082e5715 in st_table::mark_columns_needed_for_update (this=0xae72a498) at table.cc:5066
#7 0x0832bc4e in mysql_update (thd=0xa34ab10, table_list=0xae712eb8, fields=..., values=..., conds=0x0, order_num=0, order=0x0, limit=18446744073709551615,
    handle_duplicates=DUP_ERROR, ignore=false) at sql_update.cc:382
#8 0x082855e2 in mysql_execute_command (thd=0xa34ab10) at sql_parse.cc:3110
#9 0x0828e704 in mysql_parse (thd=0xa34ab10, rawbuf=0xae712e00 "UPDATE view1 SET field1 = 'really'", length=34, found_semicolon=0xae8ff228)
    at sql_parse.cc:6138
#10 0x08280c88 in dispatch_command (command=COM_QUERY, thd=0xa34ab10, packet=0xa3a34c1 "UPDATE view1 SET field1 = 'really'", packet_length=34)
    at sql_parse.cc:1213
#11 0x08280125 in do_command (thd=0xa34ab10) at sql_parse.cc:903
#12 0x0827d1c5 in handle_one_connection (arg=0xa34ab10) at sql_connect.cc:1177
#13 0x00821919 in start_thread () from /lib/libpthread.so.0
#14 0x0076acce in clone () from /lib/libc.so.6

test case:

CREATE TABLE t1 (field1 INTEGER);
CREATE TABLE t2 (field1 INTEGER);

CREATE OR REPLACE ALGORITHM = MERGE VIEW view2 AS SELECT * FROM t2;
CREATE OR REPLACE ALGORITHM = MERGE VIEW view1 AS SELECT * FROM view2;
CREATE OR REPLACE ALGORITHM = TEMPTABLE VIEW view2 AS SELECT * FROM t1;
UPDATE view1 SET field1 = 'really';

Changed in maria:
milestone: none → 5.2
Changed in maria:
importance: Undecided → Critical
status: New → Confirmed
assignee: nobody → Igor Babaev (igorb-seattle)
Changed in maria:
status: Confirmed → Fix Committed
Revision history for this message
Oleksandr "Sanja" Byelkin (sanja-byelkin) wrote :

It should return error (probably check of updateability is not correct)

Changed in maria:
assignee: Igor Babaev (igorb-seattle) → Oleksandr "Sanja" Byelkin (sanja-byelkin)
status: Fix Committed → In Progress
Changed in maria:
status: In Progress → Fix Committed
Changed in maria:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.