From 1434bc5db17d02ce024a3b8c9335b1e80395d40e Mon Sep 17 00:00:00 2001
From: "guilhem@mysql.com" <>
Date: Wed, 12 Mar 2003 01:05:04 +0100
Subject: [PATCH] Fix for bug #136. We must write the INSERT into u SELECT *
 from t to the binlog BEFORE unlocking t (which was read-locked at least in
 MyISAM), and not the contrary.

---
 sql/sql_select.cc | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/sql/sql_select.cc b/sql/sql_select.cc
index d3934fbd620..ec6b7961267 100644
--- a/sql/sql_select.cc
+++ b/sql/sql_select.cc
@@ -4469,9 +4469,16 @@ do_select(JOIN *join,List<Item> *fields,TABLE *table,Procedure *procedure)
     error=0;
     if (!table)					// If sending data to client
     {
-      join_free(join);				// Unlock all cursors
+      //note that the call below may trigger binlog writing for some commands...
       if (join->result->send_eof())
 	error= 1;				// Don't send error
+      /*
+        ...which must be done before unlocking the read tables (otherwise
+        another thread may, quickly between unlock and binlog-write,
+        update the read table and write to the binlog, which will
+        result in badly ordered binlog events (and replication breaks).
+      */
+      join_free(join);				// Unlock all cursors
     }
     DBUG_PRINT("info",("%ld records output",join->send_records));
   }