bk commit into 5.1 tree (istruewing:1.2598) BUG#22351

[Ingo Struewing]

Below is the list of changes that have just been committed into a local
5.1 repository of istruewing. When istruewing does a push these changes will
be propagated to the main repository and, within 24 hours after the
push, to the public repository.
For information on how to access the public repository
see http://dev.mysql.com/doc/mysql/en/installing-source-tree.html

ChangeSet@stripped, 2007-10-30 09:40:50+01:00, istruewing@stripped +1 -0
  Bug#22351 - handler::index_next_same() call to key_cmp_if_same() uses
              the wrong buffer
  
  handler::index_next_same() did not take into account that the
  internally called function key_cmp_if_same() uses the fixed
  buffer table->record[0] for key comparison instead of the
  buffer provided by the caller of handler::index_next_same().
  
  Added code to temporarily redirect table->record[0] to the
  record buffer provided by the caller of handler::index_next_same().
  
  The test case is in partition.test already.

  sql/handler.cc@stripped, 2007-10-30 09:40:49+01:00, istruewing@stripped +7 -1
    Bug#22351 - handler::index_next_same() call to key_cmp_if_same() uses
                the wrong buffer
    Added code to temporarily redirect table->record[0] to the
    record buffer provided by the caller of handler::index_next_same().

diff -Nrup a/sql/handler.cc b/sql/handler.cc
--- a/sql/handler.cc	2007-08-31 08:57:57 +02:00
+++ b/sql/handler.cc	2007-10-30 09:40:49 +01:00
@@ -2521,16 +2521,22 @@ int ha_enable_transaction(THD *thd, bool
 
 int handler::index_next_same(uchar *buf, const uchar *key, uint keylen)
 {
+  uchar *save_record_0;
   int error;
+  DBUG_ENTER("index_next_same");
   if (!(error=index_next(buf)))
   {
+    /* key_cmp_if_same() compares table->record[0] against 'key'. */
+    save_record_0= table->record[0];
+    table->record[0]= buf;
     if (key_cmp_if_same(table, key, active_index, keylen))
     {
       table->status=STATUS_NOT_FOUND;
       error=HA_ERR_END_OF_FILE;
     }
+    table->record[0]= save_record_0;
   }
-  return error;
+  DBUG_RETURN(error);
 }