summaryrefslogtreecommitdiff
path: root/storage/myisam
diff options
context:
space:
mode:
authortimour@askmonty.org <timour@askmonty.org>2013-10-29 12:39:03 +0200
committertimour@askmonty.org <timour@askmonty.org>2013-10-29 12:39:03 +0200
commit883af99e7dac91e3f258135a2053e6b8e3c05fc3 (patch)
tree1881d9d2b897cfb3e086ed5a29a3a6bba6e0a410 /storage/myisam
parent94ad392fd86b53f798c7de9489afe901e1f3a9b6 (diff)
downloadmariadb-git-883af99e7dac91e3f258135a2053e6b8e3c05fc3.tar.gz
MDEV-5104 crash in Item_field::used_tables with broken order by
Analysis: st_select_lex_unit::prepare() computes can_skip_order_by as TRUE. As a result join->prepare() gets called with order == NULL, and doesn't do name resolution for the inner ORDER clause. Due to this the prepare phase doesn't detect that the query references non-exiting function and field. Later join->optimize() calls update_used_tables() for a non-resolved Item_field, which understandably has no Field object. This call results in a crash. Solution: Resolve unnecessary ORDER BY clauses to detect if they reference non-exising objects. Then remove such clauses from the JOIN object.
Diffstat (limited to 'storage/myisam')
0 files changed, 0 insertions, 0 deletions