BUG#18198

Added new test cases, fixed bugs in relation to those


mysql-test/r/partition_charset.result:
  New test case
mysql-test/r/partition_list.result:
  New test case
mysql-test/t/partition_list.test:
  New test case
sql/partition_info.cc:
  Missing initialisation
sql/sql_partition.cc:
  key_restore made use of field->ptr in VARCHAR fields and
  when used with REPLACE it caused the key_restore to
  become against table->record[0] when buf pointed to
  table->record[1]

2 files changed, 13 insertions, 3 deletions

diff --git a/sql/partition_info.cc b/sql/partition_info.cc
index 3af79f453ad..6dfedde7116 100644
--- a/sql/partition_info.cc
+++ b/sql/partition_info.cc
@@ -951,6 +951,7 @@ bool partition_info::set_up_charset_field_preps()
     if (!(char_ptrs= (char**)sql_alloc(size)))
       goto error;
     subpart_charset_field_array= (Field**)char_ptrs;
+    ptr= subpart_field_array;
     i= 0;
     while ((field= *(ptr++)))
     {
diff --git a/sql/sql_partition.cc b/sql/sql_partition.cc
index 86068cc5c0d..c02beee350c 100644
--- a/sql/sql_partition.cc
+++ b/sql/sql_partition.cc
@@ -3315,13 +3315,16 @@ static uint32 get_sub_part_id_from_key(const TABLE *table,byte *buf,
   uint32 part_id;
   DBUG_ENTER("get_sub_part_id_from_key");
 
-  key_restore(buf, (byte*)key_spec->key, key_info, key_spec->length);
   if (likely(rec0 == buf))
+  {
+    key_restore(buf, (byte*)key_spec->key, key_info, key_spec->length);
     part_id= part_info->get_subpartition_id(part_info);
+  }
   else
   {
     Field **part_field_array= part_info->subpart_field_array;
     set_field_ptr(part_field_array, buf, rec0);
+    key_restore(buf, (byte*)key_spec->key, key_info, key_spec->length);
     part_id= part_info->get_subpartition_id(part_info);
     set_field_ptr(part_field_array, rec0, buf);
   }
@@ -3358,14 +3361,17 @@ bool get_part_id_from_key(const TABLE *table, byte *buf, KEY *key_info,
   longlong func_value;
   DBUG_ENTER("get_part_id_from_key");
 
-  key_restore(buf, (byte*)key_spec->key, key_info, key_spec->length);
   if (likely(rec0 == buf))
+  {
+    key_restore(buf, (byte*)key_spec->key, key_info, key_spec->length);
     result= part_info->get_part_partition_id(part_info, part_id,
                                              &func_value);
+  }
   else
   {
     Field **part_field_array= part_info->part_field_array;
     set_field_ptr(part_field_array, buf, rec0);
+    key_restore(buf, (byte*)key_spec->key, key_info, key_spec->length);
     result= part_info->get_part_partition_id(part_info, part_id,
                                              &func_value);
     set_field_ptr(part_field_array, rec0, buf);
@@ -3405,14 +3411,17 @@ void get_full_part_id_from_key(const TABLE *table, byte *buf,
   longlong func_value;
   DBUG_ENTER("get_full_part_id_from_key");
 
-  key_restore(buf, (byte*)key_spec->key, key_info, key_spec->length);
   if (likely(rec0 == buf))
+  {
+    key_restore(buf, (byte*)key_spec->key, key_info, key_spec->length);
     result= part_info->get_partition_id(part_info, &part_spec->start_part,
                                         &func_value);
+  }
   else
   {
     Field **part_field_array= part_info->full_part_field_array;
     set_field_ptr(part_field_array, buf, rec0);
+    key_restore(buf, (byte*)key_spec->key, key_info, key_spec->length);
     result= part_info->get_partition_id(part_info, &part_spec->start_part,
                                         &func_value);
     set_field_ptr(part_field_array, rec0, buf);