diff options
author | unknown <monty@mysql.com> | 2005-10-13 19:24:01 +0300 |
---|---|---|
committer | unknown <monty@mysql.com> | 2005-10-13 19:24:01 +0300 |
commit | c8067ec8489dfc1ee96ba89100a9ef067578ffca (patch) | |
tree | e80b70fc5dabf499bc561a125c52bda1c74593eb /sql/slave.cc | |
parent | 0e9f9498eadc21f3988a21a86ae310683da501c6 (diff) | |
download | mariadb-git-c8067ec8489dfc1ee96ba89100a9ef067578ffca.tar.gz |
Fixes during review of new code
sql/slave.cc:
Moved mutex_unlock to handle 'err' case properly
vio/vio.c:
Made code clearer (fcntl_mode is always 0 here)
Diffstat (limited to 'sql/slave.cc')
-rw-r--r-- | sql/slave.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sql/slave.cc b/sql/slave.cc index ebae8461f11..2fc4eef0f64 100644 --- a/sql/slave.cc +++ b/sql/slave.cc @@ -4330,7 +4330,8 @@ Before assert, my_b_tell(cur_log)=%s rli->event_relay_log_pos=%s", time_t save_timestamp= rli->last_master_timestamp; rli->last_master_timestamp= 0; - DBUG_ASSERT(rli->relay_log.get_open_count() == rli->cur_log_old_open_count); + DBUG_ASSERT(rli->relay_log.get_open_count() == + rli->cur_log_old_open_count); if (rli->ign_master_log_name_end[0]) { @@ -4341,13 +4342,13 @@ Before assert, my_b_tell(cur_log)=%s rli->event_relay_log_pos=%s", Rotate_log_event::DUP_NAME | Rotate_log_event::ZERO_LEN); rli->ign_master_log_name_end[0]= 0; + pthread_mutex_unlock(log_lock); if (unlikely(!ev)) { errmsg= "Slave SQL thread failed to create a Rotate event " "(out of memory?), SHOW SLAVE STATUS may be inaccurate"; goto err; } - pthread_mutex_unlock(log_lock); ev->server_id= 0; // don't be ignored by slave SQL thread DBUG_RETURN(ev); } |