Fix of MDEV-33856, becaouse we have more then 64 fields

 and unsigned long long can not be used as bitfield.

Author: sanja-byelkin

sql/sql_show.cc

@@ -9112,27 +9112,16 @@ static int make_slave_status_old_format(THD *thd, ST_SCHEMA_TABLE *schema_table)
  Name_resolution_context *context= &thd->lex->first_select_lex()->context;
  DBUG_ASSERT(thd->lex->sql_command == SQLCOM_SHOW_SLAVE_STAT);
  bool all_slaves= thd->lex->mi.show_all_slaves;
- ulonglong used_fields= ~0ULL;
-
- if (!all_slaves)
- {
- /*
- Remove 2 first fields (Connection_name and Slave_SQL_State) and all
- fields above and including field 56 (Retried_transactions)
- */
- used_fields&= ~((1ULL << SLAVE_STATUS_COL_CONNECTION_NAME) |
- (1ULL << SLAVE_STATUS_COL_SLAVE_SQL_STATE));
- used_fields&= ((1ULL << SLAVE_STATUS_COL_RETRIED_TRANSACTIONS) - 1);
- }
 
  for (uint i=0; !field_info->end_marker(); field_info++, i++)
  {
- /*
- We have all_slaves here to take into account that we some day may have
- more than 64 fields in the list. If all_slaves is set we should show
- all fields.
- */
- if (all_slaves || (used_fields & ((1ULL << i))))
+ if (all_slaves ||
+ // not SLAVE_STATUS_COL_CONNECTION_NAME,
+ // SLAVE_STATUS_COL_SLAVE_SQL_STATE
+ // and less
+ // SLAVE_STATUS_COL_RETRIED_TRANSACTIONS
+ !(i <= SLAVE_STATUS_COL_SLAVE_SQL_STATE ||
+ i >= SLAVE_STATUS_COL_RETRIED_TRANSACTIONS))
  {
  LEX_CSTRING field_name= field_info->name();
  Item_field *field= new (thd->mem_root)