diff options
| author | istruewing@chilla.local <> | 2007-07-05 11:31:03 +0200 |
|---|---|---|
| committer | istruewing@chilla.local <> | 2007-07-05 11:31:03 +0200 |
| commit | 5ac6264dd6494d3d0af067eb0813c3e8236e1a66 (patch) | |
| tree | 5f0412e0571ab2361bc8c45e8ce6805f3bf5b4e2 /sql/sql_partition.cc | |
| parent | dc82068c9696f7da81c60b167b8a68d2c91d8bdb (diff) | |
| download | mariadb-git-5ac6264dd6494d3d0af067eb0813c3e8236e1a66.tar.gz | |
Bug#26827 - table->read_set is set incorrectly,
causing update of a different column
Post-pushbuild fix.
bitmap_set_bit() is an inline function in DEBUG builds and
a macro in non-DEBUG builds. The latter evaluates its 'bit'
argument twice. So one must not use increment/decrement operators
on this argument.
Moved increment of pointer out of bitmap_set_bit() call.
Diffstat (limited to 'sql/sql_partition.cc')
| -rw-r--r-- | sql/sql_partition.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sql/sql_partition.cc b/sql/sql_partition.cc index a44af968d9f..191859e0ed4 100644 --- a/sql/sql_partition.cc +++ b/sql/sql_partition.cc @@ -606,8 +606,8 @@ static bool create_full_part_field_array(THD *thd, TABLE *table, partitioning. */ if ((ptr= part_info->full_part_field_array)) - while (*ptr) - bitmap_set_bit(&part_info->full_part_field_set, (*ptr++)->field_index); + for (; *ptr; ptr++) + bitmap_set_bit(&part_info->full_part_field_set, (*ptr)->field_index); end: DBUG_RETURN(result); |