XA Crash recovery fails if server crashes in the middle of binlog rotate

When we rotate the binlog, we first close the old log file, which marks the
file as "not crashed". We then create the new log file, and last add it to the
binlog index file.

Suppose we crash after closing the old file but before the new file is added
to the index. And suppose further than at least one transaction had time to
enter the "prepared inside InnoDB" status before we crashed.

Then on next server restart, the binlog code will not realise that we crashed,
as the last file in the binlog index is closed properly. And ha_recover in
handler.cc will fail the startup due to finding a prepared innodb transaction
in a no-crash scenario:

120615 14:41:02 [ERROR] Found 1 prepared transactions! It means that mysqld was not shut down properly last time and critical recovery information (last binlog or tc.log file) was manually deleted after a crash. You have to start mysqld with --tc-heuristic-recover switch to commit or rollback pending transactions.

To reproduce, apply following patch (to get needed debug_sync points) and run
attached test case.

=== modified file 'sql/handler.cc'

--- sql/handler.cc	2012-05-21 18:54:41 +0000

+++ sql/handler.cc	2012-06-15 12:37:51 +0000

@@ -1276,6 +1276,7 @@ int ha_commit_trans(THD *thd, bool all)

     need_prepare_ordered|= (ht->prepare_ordered != NULL);

     need_commit_ordered|= (ht->commit_ordered != NULL);

   }

+  DEBUG_SYNC(thd, "ha_commit_trans_after_prepare");

   DBUG_EXECUTE_IF("crash_commit_after_prepare", DBUG_SUICIDE(););

   if (!is_real_trans)

=== modified file 'sql/log.cc'

--- sql/log.cc	2012-05-21 18:54:41 +0000

+++ sql/log.cc	2012-06-15 12:39:45 +0000

@@ -3189,6 +3189,10 @@ bool MYSQL_BIN_LOG::open(const char *log

     if (write_file_name_to_index_file)

     {

 #ifdef HAVE_REPLICATION

+#ifdef ENABLED_DEBUG_SYNC

+      if (current_thd)

+        DEBUG_SYNC(current_thd, "binlog_open_before_update_index");

+#endif

       DBUG_EXECUTE_IF("crash_create_critical_before_update_index", DBUG_SUICIDE(););

 #endif