diff options
| author | unknown <jani@hynda.(none)> | 2003-03-22 19:55:09 +0200 |
|---|---|---|
| committer | unknown <jani@hynda.(none)> | 2003-03-22 19:55:09 +0200 |
| commit | ebedd1521c19f9a6a2025d5c1c5be751cdb7b636 (patch) | |
| tree | 7010d786c9d6f639c0538e47d4bd1361d560d33f /sql/sql_parse.cc | |
| parent | 398707501216b30344c02bbe934c715fdff63cb1 (diff) | |
| download | mariadb-git-ebedd1521c19f9a6a2025d5c1c5be751cdb7b636.tar.gz | |
Added error in case auto_increment and default where defined
at the same time.
Diffstat (limited to 'sql/sql_parse.cc')
| -rw-r--r-- | sql/sql_parse.cc | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/sql/sql_parse.cc b/sql/sql_parse.cc index 8439db0be6b..e861aef5fe7 100644 --- a/sql/sql_parse.cc +++ b/sql/sql_parse.cc @@ -2880,21 +2880,29 @@ bool add_field_to_list(char *field_name, enum_field_types type, lex->col_list.empty(); } - if (default_value && default_value->type() == Item::NULL_ITEM) + if (default_value) { - if ((type_modifier & (NOT_NULL_FLAG | AUTO_INCREMENT_FLAG)) == - NOT_NULL_FLAG) + if (default_value->type() == Item::NULL_ITEM) { - net_printf(&thd->net,ER_INVALID_DEFAULT,field_name); + default_value=0; + if ((type_modifier & (NOT_NULL_FLAG | AUTO_INCREMENT_FLAG)) == + NOT_NULL_FLAG) + { + net_printf(&thd->net,ER_INVALID_DEFAULT,field_name); + DBUG_RETURN(1); + } + } + else if (type_modifier & AUTO_INCREMENT_FLAG) + { + net_printf(&thd->net, ER_INVALID_DEFAULT, field_name); DBUG_RETURN(1); } - default_value=0; } if (!(new_field=new create_field())) DBUG_RETURN(1); new_field->field=0; new_field->field_name=field_name; - new_field->def= (type_modifier & AUTO_INCREMENT_FLAG ? 0 : default_value); + new_field->def= default_value; new_field->flags= type_modifier; new_field->unireg_check= (type_modifier & AUTO_INCREMENT_FLAG ? Field::NEXT_NUMBER : Field::NONE); |