Commit 5b7566a1 authored by Kristofer Pettersson's avatar Kristofer Pettersson

Bug#34417 Assertion failure in Diagnostics_area::set_error_status()

  
The Diagnostic_area caused an assertion failure in debug mode when
the disk was full.
  
By setting the internal error handler to ignore errors caused by
underlying logging methods, the error is avoided.


sql/log.cc:
  MYSQL_QUERY_LOG::write uses IOCACHE for writing data. If writing fails 
  an error is pushed by the procedure my_error which in turn invokes
  the default error handler my_message_sql. When the error is set the
  Diagnostic_area status changes and later trigger an assertion on the
  next consecutive status change which happens by for example the 
  function my_ok(). By intercepting the error handler from the logger
  interface this error can be silenced.
parent 8d07122a
...@@ -741,10 +741,14 @@ bool Log_to_file_event_handler:: ...@@ -741,10 +741,14 @@ bool Log_to_file_event_handler::
ulonglong query_utime, ulonglong lock_utime, bool is_command, ulonglong query_utime, ulonglong lock_utime, bool is_command,
const char *sql_text, uint sql_text_len) const char *sql_text, uint sql_text_len)
{ {
return mysql_slow_log.write(thd, current_time, query_start_arg, Silence_log_table_errors error_handler;
thd->push_internal_handler(&error_handler);
bool retval= mysql_slow_log.write(thd, current_time, query_start_arg,
user_host, user_host_len, user_host, user_host_len,
query_utime, lock_utime, is_command, query_utime, lock_utime, is_command,
sql_text, sql_text_len); sql_text, sql_text_len);
thd->pop_internal_handler();
return retval;
} }
...@@ -760,9 +764,13 @@ bool Log_to_file_event_handler:: ...@@ -760,9 +764,13 @@ bool Log_to_file_event_handler::
const char *sql_text, uint sql_text_len, const char *sql_text, uint sql_text_len,
CHARSET_INFO *client_cs) CHARSET_INFO *client_cs)
{ {
return mysql_log.write(event_time, user_host, user_host_len, Silence_log_table_errors error_handler;
thd->push_internal_handler(&error_handler);
bool retval= mysql_log.write(event_time, user_host, user_host_len,
thread_id, command_type, command_type_len, thread_id, command_type, command_type_len,
sql_text, sql_text_len); sql_text, sql_text_len);
thd->pop_internal_handler();
return retval;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment