mirror of
https://github.com/MariaDB/server.git
synced 2025-08-07 00:04:31 +03:00
A fix for safe updates
This commit is contained in:
@@ -35,13 +35,13 @@ int mysql_delete(THD *thd, TABLE_LIST *table_list, COND *conds, ORDER *order,
|
|||||||
SQL_SELECT *select=0;
|
SQL_SELECT *select=0;
|
||||||
READ_RECORD info;
|
READ_RECORD info;
|
||||||
bool using_limit=limit != HA_POS_ERROR;
|
bool using_limit=limit != HA_POS_ERROR;
|
||||||
bool transactional_table, log_delayed;
|
bool transactional_table, log_delayed, safe_update;
|
||||||
ha_rows deleted;
|
ha_rows deleted;
|
||||||
DBUG_ENTER("mysql_delete");
|
DBUG_ENTER("mysql_delete");
|
||||||
|
|
||||||
if (!table_list->db)
|
if (!table_list->db)
|
||||||
table_list->db=thd->db;
|
table_list->db=thd->db;
|
||||||
if ((thd->options & OPTION_SAFE_UPDATES) && !conds)
|
if (((safe_update=thd->options & OPTION_SAFE_UPDATES)) && !conds)
|
||||||
{
|
{
|
||||||
send_error(&thd->net,ER_UPDATE_WITHOUT_KEY_IN_SAFE_MODE);
|
send_error(&thd->net,ER_UPDATE_WITHOUT_KEY_IN_SAFE_MODE);
|
||||||
DBUG_RETURN(1);
|
DBUG_RETURN(1);
|
||||||
@@ -57,7 +57,7 @@ int mysql_delete(THD *thd, TABLE_LIST *table_list, COND *conds, ORDER *order,
|
|||||||
|
|
||||||
/* Test if the user wants to delete all rows */
|
/* Test if the user wants to delete all rows */
|
||||||
if (!using_limit && (!conds || conds->const_item()) &&
|
if (!using_limit && (!conds || conds->const_item()) &&
|
||||||
!(specialflag & (SPECIAL_NO_NEW_FUNC | SPECIAL_SAFE_MODE)))
|
!(specialflag & (SPECIAL_NO_NEW_FUNC | SPECIAL_SAFE_MODE)) && !safe_update)
|
||||||
{
|
{
|
||||||
deleted= table->file->records;
|
deleted= table->file->records;
|
||||||
if (!(error=table->file->delete_all_rows()))
|
if (!(error=table->file->delete_all_rows()))
|
||||||
@@ -91,7 +91,7 @@ int mysql_delete(THD *thd, TABLE_LIST *table_list, COND *conds, ORDER *order,
|
|||||||
if (!table->quick_keys)
|
if (!table->quick_keys)
|
||||||
{
|
{
|
||||||
thd->lex.select_lex.options|=QUERY_NO_INDEX_USED;
|
thd->lex.select_lex.options|=QUERY_NO_INDEX_USED;
|
||||||
if ((thd->options & OPTION_SAFE_UPDATES) && limit == HA_POS_ERROR)
|
if (safe_update && !using_limit)
|
||||||
{
|
{
|
||||||
delete select;
|
delete select;
|
||||||
send_error(&thd->net,ER_UPDATE_WITHOUT_KEY_IN_SAFE_MODE);
|
send_error(&thd->net,ER_UPDATE_WITHOUT_KEY_IN_SAFE_MODE);
|
||||||
|
@@ -53,7 +53,7 @@ int mysql_update(THD *thd,
|
|||||||
enum enum_duplicates handle_duplicates,
|
enum enum_duplicates handle_duplicates,
|
||||||
thr_lock_type lock_type)
|
thr_lock_type lock_type)
|
||||||
{
|
{
|
||||||
bool using_limit=limit != HA_POS_ERROR;
|
bool using_limit=limit != HA_POS_ERROR, safe_update= thd->options & OPTION_SAFE_UPDATES;
|
||||||
bool used_key_is_modified, transactional_table, log_delayed;
|
bool used_key_is_modified, transactional_table, log_delayed;
|
||||||
int error=0;
|
int error=0;
|
||||||
uint save_time_stamp, used_index, want_privilege;
|
uint save_time_stamp, used_index, want_privilege;
|
||||||
@@ -117,9 +117,7 @@ int mysql_update(THD *thd,
|
|||||||
table->used_keys=0;
|
table->used_keys=0;
|
||||||
select=make_select(table,0,0,conds,&error);
|
select=make_select(table,0,0,conds,&error);
|
||||||
if (error ||
|
if (error ||
|
||||||
(select && select->check_quick(test(thd->options & OPTION_SAFE_UPDATES),
|
(select && select->check_quick(safe_update, limit)) || !limit)
|
||||||
limit)) ||
|
|
||||||
!limit)
|
|
||||||
{
|
{
|
||||||
delete select;
|
delete select;
|
||||||
table->time_stamp=save_time_stamp; // Restore timestamp pointer
|
table->time_stamp=save_time_stamp; // Restore timestamp pointer
|
||||||
@@ -134,7 +132,7 @@ int mysql_update(THD *thd,
|
|||||||
if (!table->quick_keys)
|
if (!table->quick_keys)
|
||||||
{
|
{
|
||||||
thd->lex.select_lex.options|=QUERY_NO_INDEX_USED;
|
thd->lex.select_lex.options|=QUERY_NO_INDEX_USED;
|
||||||
if ((thd->options & OPTION_SAFE_UPDATES) && limit == HA_POS_ERROR)
|
if (safe_update && !using_limit)
|
||||||
{
|
{
|
||||||
delete select;
|
delete select;
|
||||||
table->time_stamp=save_time_stamp;
|
table->time_stamp=save_time_stamp;
|
||||||
|
Reference in New Issue
Block a user