summaryrefslogtreecommitdiff
path: root/sql/sql_parse.cc
diff options
context:
space:
mode:
authorGeorgi Kodinov <joro@sun.com>2010-01-29 17:04:37 +0200
committerGeorgi Kodinov <joro@sun.com>2010-01-29 17:04:37 +0200
commit6d38c898a6c69c2c7f862d40a323f7feec8934d5 (patch)
tree1878d495d8135589ef763899da62d853133a53c5 /sql/sql_parse.cc
parentb8eaa81dd95259a0a77ecc6fe9a5d736dc1dfcd6 (diff)
downloadmariadb-git-6d38c898a6c69c2c7f862d40a323f7feec8934d5.tar.gz
Bug #49324: more valgrind errors in test_if_skip_sort_order
Fixed 2 problems : 1. test_if_order_by_key() was continuing on the primary key as if it has a primary key suffix (as the secondary keys do). This leads to crashes in ORDER BY <pk>,<pk>. Fixed by not treating the primary key as the secondary one and not depending on it being clustered with a primary key. 2. The cost calculation was trying to read the records per key when operating on ORDER BYs that order on all of the secondary key + some of the primary key. This leads to crashes because of out-of-bounds array access. Fixed by assuming we'll find 1 record per key in such cases.
Diffstat (limited to 'sql/sql_parse.cc')
0 files changed, 0 insertions, 0 deletions