Details
Major Description
sql/partition_info.cc
bool partition_info::set_partition_bitmaps_from_table(TABLE_LIST *table_list) |
{
|
List<String> *partition_names= table_list ?
|
NULL : table_list->partition_names;
|
return set_partition_bitmaps(partition_names); |
}
|
If table_list is not NULL then we assign NULL to partition_names.
If table_list is NULL then we dereference NULL pointer in table_list->partition_names.
Looks like the order of the ? operator results should be reversed:
bool partition_info::set_partition_bitmaps_from_table(TABLE_LIST *table_list) |
{
|
List<String> *partition_names= table_list ?
|
table_list->partition_names : NULL;
|
return set_partition_bitmaps(partition_names); |
}
|
Attachments
Activity
| Field | Original Value | New Value |
|---|---|---|
| Affects Version/s | 10.3.5 [ 22905 ] |
| Description |
sql/partition_info.cc
{code:c++} bool partition_info::set_partition_bitmaps_from_table(TABLE_LIST *table_list) { List<String> *partition_names= table_list ? NULL : table_list->partition_names; return set_partition_bitmaps(partition_names); } {code} If table_list is not NULL then we assign NULL to partition_names. If table_list is NULL then we dereference NULL pointer in table_list->partition_names. Looks like the order of the ? operator results should be reverted: {code:c++} bool partition_info::set_partition_bitmaps_from_table(TABLE_LIST *table_list) { List<String> *partition_names= table_list ? table_list->partition_names : NULL; return set_partition_bitmaps(partition_names); } {code} |
sql/partition_info.cc
{code:c++} bool partition_info::set_partition_bitmaps_from_table(TABLE_LIST *table_list) { List<String> *partition_names= table_list ? NULL : table_list->partition_names; return set_partition_bitmaps(partition_names); } {code} If table_list is not NULL then we assign NULL to partition_names. If table_list is NULL then we dereference NULL pointer in table_list->partition_names. Looks like the order of the ? operator results should be reversed: {code:c++} bool partition_info::set_partition_bitmaps_from_table(TABLE_LIST *table_list) { List<String> *partition_names= table_list ? table_list->partition_names : NULL; return set_partition_bitmaps(partition_names); } {code} |
| Assignee | Sergei Golubchik [ serg ] |
| Fix Version/s | 10.3 [ 22126 ] | |
| Fix Version/s | 10.4 [ 22408 ] | |
| Fix Version/s | 10.5 [ 23123 ] | |
| Fix Version/s | 10.6 [ 24028 ] | |
| Fix Version/s | 10.7 [ 24805 ] | |
| Fix Version/s | 10.8 [ 26121 ] | |
| Fix Version/s | 10.9 [ 26905 ] | |
| Fix Version/s | 10.10 [ 27530 ] |
| Status | Open [ 1 ] | In Progress [ 3 ] |
| Status | In Progress [ 3 ] | In Testing [ 10301 ] |
| Component/s | OTHER [ 10125 ] | |
| Fix Version/s | 10.3.38 [ 28507 ] | |
| Fix Version/s | 10.4.28 [ 28509 ] | |
| Fix Version/s | 10.5.19 [ 28511 ] | |
| Fix Version/s | 10.6.12 [ 28513 ] | |
| Fix Version/s | 10.7.8 [ 28515 ] | |
| Fix Version/s | 10.8.7 [ 28517 ] | |
| Fix Version/s | 10.9.5 [ 28519 ] | |
| Fix Version/s | 10.10.3 [ 28521 ] | |
| Fix Version/s | 10.3 [ 22126 ] | |
| Fix Version/s | 10.4 [ 22408 ] | |
| Fix Version/s | 10.5 [ 23123 ] | |
| Fix Version/s | 10.6 [ 24028 ] | |
| Fix Version/s | 10.7 [ 24805 ] | |
| Fix Version/s | 10.8 [ 26121 ] | |
| Fix Version/s | 10.9 [ 26905 ] | |
| Fix Version/s | 10.10 [ 27530 ] | |
| Resolution | Fixed [ 1 ] | |
| Status | In Testing [ 10301 ] | Closed [ 6 ] |
Introduced in 10.3.5 - https://github.com/MariaDB/server/commit/b4a2baffa82e5c07b96a1c752228560dcac1359b