diff options
| author | Alexey Botchkov <holyfoot@askmonty.org> | 2022-07-16 16:54:03 +0400 |
|---|---|---|
| committer | Alexey Botchkov <holyfoot@askmonty.org> | 2022-07-17 01:10:43 +0400 |
| commit | 8911823f65a6557ce66ea5f8aecd55b115a85606 (patch) | |
| tree | eeeeee73708c22dbf584ca1444be46321a670fa1 /sql/field.h | |
| parent | 65cc89ed9eb9367a7e068286ddc9a896c478c012 (diff) | |
| download | mariadb-git-8911823f65a6557ce66ea5f8aecd55b115a85606.tar.gz | |
MDEV-26546 SIGSEGV's in spider_db_connect on SHOW TABLE and spider_db… …_mbase::connect (and SIGSEGV's in check_vcol_forward_refs and inline_mysql_mutex_lock)
Not the SPIDER issue - happens to INSERT DELAYED.
the field::make_new_field does't copy the LONG_UNIQUE_HASH_FIELD
flag to the new field. Though the Delayed_insert::get_local_table
copies the field->vcol_info for this field. Ad a result
the parse_vcol_defs doesn't create the expression for that column
so the field->vcol_info->expr is NULL. Which leads to crash.
Backported fix for this from 10.5 - the flagg added in the
Delayed_insert::get_local_table.
Another problem with the USING HASH key is thst the
parse_vcol_defs modifies the table->keys content. Then the same
parse_vcol_defs is called on the table copy that has keys already
modified. Backported fix for that from 10.5 - key copying added
tot the Delayed_insert::get_local_table.
Finally - the created copy has to clear the expr_arena as
this table is not in the thd->open_tables list so won't be
cleared automatically.
Diffstat (limited to 'sql/field.h')
| -rw-r--r-- | sql/field.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sql/field.h b/sql/field.h index d31be2a96cc..7534a506edc 100644 --- a/sql/field.h +++ b/sql/field.h @@ -706,7 +706,7 @@ public: enum imagetype { itRAW, itMBR}; utype unireg_check; - const uint32 field_length; // Length of field + uint32 field_length; // Length of field uint32 flags; uint16 field_index; // field number in fields array uchar null_bit; // Bit used to test null bit |