touchup to last fix

sql/log.cc:
  moved mutex back to after is_open() test,
  but still protecting name dereference (the real problem)

sql/log.cc

@@ -513,11 +513,13 @@ bool MYSQL_LOG::is_active(const char* log_file_name)
 
 void MYSQL_LOG::new_file(bool inside_mutex)
 {
+  if (!is_open())
+    return;
+
   if (!inside_mutex)
     VOID(pthread_mutex_lock(&LOCK_log));
-  if (is_open())
-  {
-    char new_name[FN_REFLEN], *old_name=name;
+
+  char new_name[FN_REFLEN], *old_name = name;
  
   if (!no_rotate)
   {
@@ -550,7 +552,7 @@ void MYSQL_LOG::new_file(bool inside_mutex)
   my_free(old_name,MYF(0));
   last_time=query_start=0;
   write_error=0;
-  }
+
   if (!inside_mutex)
     VOID(pthread_mutex_unlock(&LOCK_log));
 }
@@ -564,8 +566,8 @@ bool MYSQL_LOG::write(THD *thd,enum enum_server_command command,
     int error=0;
     VOID(pthread_mutex_lock(&LOCK_log));
 
-    /* Test if someone closed after the is_open test */
-    if (log_type != LOG_CLOSED)
+    /* Test if someone closed between the is_open test and lock */
+    if (is_open())
     {
       time_t skr;
       ulong id;