1. 29 Jun, 2017 1 commit
    • Vicențiu Ciorbaru's avatar
      Fix debug assert post MDEV-10306 · 5c89f23b
      Vicențiu Ciorbaru authored
      tmp variable now points to str->ptr() buffer, not tmp_value buffer.
      Comparing pointers otherwise can lead to false assertion errors as we
      don't know where buffers are allocated in respect to each other.
      5c89f23b
  2. 28 Jun, 2017 1 commit
  3. 27 Jun, 2017 2 commits
  4. 20 Jun, 2017 1 commit
  5. 19 Jun, 2017 3 commits
    • Sergei Golubchik's avatar
    • Marko Mäkelä's avatar
      MDEV-12975 InnoDB redo log minimum size check uses detected file size instead... · 5e4f4ec8
      Marko Mäkelä authored
      MDEV-12975 InnoDB redo log minimum size check uses detected file size instead of requested innodb_log_file_size
      
      log_calc_max_ages(): Use the requested size in the check, instead of
      the detected redo log size. The redo log will be resized at startup
      if it differs from what has been requested.
      5e4f4ec8
    • Alexander Barkov's avatar
      MDEV-10306 Wrong results with combination of CONCAT, SUBSTR and CONVERT in subquery · 3a37afec
      Alexander Barkov authored
      The bug happens because of a combination of unfortunate circumstances:
      
      1. Arguments args[0] and args[2] of Item_func_concat point recursively
      (through Item_direct_view_ref's) to the same Item_func_conv_charset.
      Both args[0]->args[0]->ref[0] and args[2]->args[0]->ref[0] refer to
      this Item_func_conv_charset.
      
      2. When Item_func_concat::args[0]->val_str() is called,
      Item_func_conv_charset::val_str() writes its result to
      Item_func_conc_charset::tmp_value.
      
      3. Then, for optimization purposes (to avoid copying),
      Item_func_substr::val_str() initializes Item_func_substr::tmp_value
      to point to the buffer fragment owned by Item_func_conv_charset::tmp_value
      Item_func_substr::tmp_value is returned as a result of
      Item_func_concat::args[0]->val_str().
      
      4. Due to optimization to avoid memory reallocs,
      Item_func_concat::val_str() remembers the result of args[0]->val_str()
      in "res" and further uses "res" to collect the return value.
      
      5. When Item_func_concat::args[2]->val_str() is called,
      Item_func_conv_charset::tmp_value gets overwritten (see #1),
      which effectively overwrites args[0]'s Item_func_substr::tmp_value (see #3),
      which effectively overwrites "res" (see #4).
      
      This patch does the following:
      
      a. Changes Item_func_conv_charset::val_str(String *str) to use
         tmp_value and str the other way around. After this change tmp_value
         is used to store a temporary result, while str is used to return the value.
         The fixes the second problem (without SUBSTR):
           SELECT CONCAT(t2,'-',t2) c2
             FROM (SELECT CONVERT(t USING latin1) t2 FROM t1) sub;
         As Item_func_concat::val_str() supplies two different buffers when calling
         args[0]->val_str() and args[2]->val_str(), in the new reduction the result
         created during args[0]->val_str() does not get overwritten by
         args[2]->val_str().
      
      b. Fixing the same problem in val_str() for similar classes
      
         Item_func_to_base64
         Item_func_from_base64
         Item_func_weight_string
         Item_func_hex
         Item_func_unhex
         Item_func_quote
         Item_func_compress
         Item_func_uncompress
         Item_func_des_encrypt
         Item_func_des_decrypt
         Item_func_conv_charset
         Item_func_reverse
         Item_func_soundex
         Item_func_aes_encrypt
         Item_func_aes_decrypt
         Item_func_buffer
      
      c. Fixing Item_func::val_str_from_val_str_ascii() the same way.
         Now Item_str_ascii_func::ascii_buff is used for temporary value,
         while the parameter passed to val_str() is used to return the result.
         This fixes the same problem when conversion (from ASCII to e.g. UCS2)
         takes place. See the ctype_ucs.test for example queries that returned
         wrong results before the fix.
      
      d. Some Item_func descendand classes had temporary String buffers
         (tmp_value and tmp_str), but did not really use them.
         Removing these temporary buffers from:
      
         Item_func_decode_histogram
         Item_func_format
         Item_func_binlog_gtid_pos
         Item_func_spatial_collection:
      
      e. Removing Item_func_buffer::tmp_value, because it's not used any more.
      
      f. Renaming Item_func_[un]compress::buffer to "tmp_value",
         for consistency with other classes.
      
      Note, this patch does not fix the following classes
      (although they have a similar problem):
      
         Item_str_conv
         Item_func_make_set
         Item_char_typecast
      
      They have a complex implementations and simple swapping between "tmp_value"
      and "str" won't work. These classes will be fixed separately.
      3a37afec
  6. 18 Jun, 2017 3 commits
  7. 15 Jun, 2017 3 commits
    • Vicențiu Ciorbaru's avatar
      MDEV-12666: CURRENT_ROLE() and DATABASE() does not work in a view · f0ad9340
      Vicențiu Ciorbaru authored
      The problem lies in how CURRENT_ROLE is defined. The
      Item_func_current_role inherits from Item_func_sysconst, which defines
      a safe_charset_converter to be a const_charset_converter.
      
      During view creation, if there is no role previously set, the current_role()
      function returns NULL.
      
      This is captured on item instantiation and the
      const_charset_converter call subsequently returns an Item_null.
      In turn, the function is replaced with Item_null and the view is
      then created with an Item_null instead of Item_func_current_role.
      
      Without this patch, the first SHOW CREATE VIEW from the testcase would
      have a where clause of WHERE role_name = NULL, while the second SHOW
      CREATE VIEW would show a correctly created view.
      
      The same applies for the DATABASE function, as it can change as well.
      
      There is an additional problem with CURRENT_ROLE() when used in a
      prepared statement. During prepared statement creation we used to set
      the string_value of the function to the current role as well as the
      null_value flag. During execution, if CURRENT_ROLE was not null, the
      null_value flag was never set to not-null during fix_fields.
      
      Item_func_current_user however can never be NULL so it did not show this
      problem in a view before. At the same time, the CURRENT_USER() can not
      be changed between prepared statement execution and creation so the
      implementation where the value is stored during fix_fields is
      sufficient.
      
      Note also that DATABASE() function behaves differently during prepared
      statements. See bug 25843 for details or commit
      7e0ad09e
      f0ad9340
    • Vicențiu Ciorbaru's avatar
      MDEV-10463: Granted as a whole to roles, databases are not show in SHOW DATABASES · 34da3be8
      Vicențiu Ciorbaru authored
      The problem lies in not checking role privileges as well during SHOW
      DATABASES command. This problem is also apparent for SHOW CREATE
      DATABASE command.
      
      Other SHOW COMMANDS make use of check_access, which in turn makes use of
      acl_get for both priv_user and priv_role parts, which allows them to
      function correctly.
      34da3be8
    • =Ian Gilfillan's avatar
      Update MariaDB Foundation sponsors · 2579b252
      =Ian Gilfillan authored
      2579b252
  8. 14 Jun, 2017 4 commits
  9. 13 Jun, 2017 1 commit
  10. 12 Jun, 2017 3 commits
  11. 09 Jun, 2017 2 commits
    • Marko Mäkelä's avatar
      MDEV-13039 innodb_fast_shutdown=0 may fail to purge all undo log · 417434f1
      Marko Mäkelä authored
      When a slow shutdown is performed soon after spawning some work for
      background threads that can create or commit transactions, it is possible
      that new transactions are started or committed after the purge has finished.
      This is violating the specification of innodb_fast_shutdown=0, namely that
      the purge must be completed. (None of the history of the recent transactions
      would be purged.)
      
      Also, it is possible that the purge threads would exit in slow shutdown
      while there exist active transactions, such as recovered incomplete
      transactions that are being rolled back. Thus, the slow shutdown could
      fail to purge some undo log that becomes purgeable after the transaction
      commit or rollback.
      
      srv_undo_sources: A flag that indicates if undo log can be generated
      or the persistent, whether by background threads or by user SQL.
      Even when this flag is clear, active transactions that already exist
      in the system may be committed or rolled back.
      
      innodb_shutdown(): Renamed from innobase_shutdown_for_mysql().
      Do not return an error code; the operation never fails.
      Clear the srv_undo_sources flag, and also ensure that the background
      DROP TABLE queue is empty.
      
      srv_purge_should_exit(): Do not allow the purge to exit if
      srv_undo_sources are active or the background DROP TABLE queue is not
      empty, or in slow shutdown, if any active transactions exist
      (and are being rolled back).
      
      srv_purge_coordinator_thread(): Remove some previous workarounds
      for this bug.
      
      innobase_start_or_create_for_mysql(): Set buf_page_cleaner_is_active
      and srv_dict_stats_thread_active directly. Set srv_undo_sources before
      starting the purge subsystem, to prevent immediate shutdown of the purge.
      Create dict_stats_thread and fts_optimize_thread immediately
      after setting srv_undo_sources, so that shutdown can use this flag to
      determine if these subsystems were started.
      
      dict_stats_shutdown(): Shut down dict_stats_thread. Backported from 10.2.
      
      srv_shutdown_table_bg_threads(): Remove (unused).
      417434f1
    • Marko Mäkelä's avatar
      Correct a merge error of MDEV-11626 · a9117c90
      Marko Mäkelä authored
      a9117c90
  12. 08 Jun, 2017 2 commits
  13. 07 Jun, 2017 2 commits
    • Igor Babaev's avatar
      Fixed the bug mdev-12963. · 151f4e9b
      Igor Babaev authored
      This patch corrects the fix for bug mdev-7599.
      When the min/max optimization of the function
      opt_sum_query() optimizes away all tables of
      a subquery it should not ever be rolled back.
      151f4e9b
    • Igor Babaev's avatar
      Fixed the bug mdev-12838. · c258ca24
      Igor Babaev authored
      If the optimizer chose an execution plan where
      a semi-join nest were materialized and the
      result of materialization was scanned to access
      other tables by ref access it could build a key
      over columns of the tables from the nest that
      were actually inaccessible.
      The patch performs a proper check whether a key
      that uses columns of the tables from a materialized
      semi-join nest can be employed to access outer tables.
      c258ca24
  14. 06 Jun, 2017 1 commit
    • Marko Mäkelä's avatar
      Follow-up to MDEV-12042 (test innodb_page_size variants) · d8d39721
      Marko Mäkelä authored
      innodb_page_size_small: A new set of combinations, for
      innodb_page_size up to 16k. In MariaDB 10.0, this does not
      make a difference, but in 10.1 and later, innodb_page_size
      would cover 32k and 64k, for which ROW_FORMAT=COMPRESSED
      is not available.
      
      Enable these combinations in a few InnoDB tests.
      d8d39721
  15. 05 Jun, 2017 1 commit
  16. 01 Jun, 2017 1 commit
  17. 29 May, 2017 2 commits
  18. 26 May, 2017 1 commit
    • Marko Mäkelä's avatar
      MDEV-12052 Shutdown crash presumably due to master thread activity · 449a88e1
      Marko Mäkelä authored
      InnoDB shutdown assumes that once the server has entered
      SRV_SHUTDOWN_FLUSH_PHASE, no change to persistent data is allowed.
      It was possible for the master thread to wake up while shutdown
      is executing in SRV_SHUTDOWN_FLUSH_PHASE or
      even in SRV_SHUTDOWN_LAST_PHASE.
      
      We do not yet know if further crashes at shutdown are possible.
      Also, we do not know if all the observed crashes could be explained
      by the race conditions that we are now fixing.
      
      srv_shutdown_print_master_pending(): Remove a redundant ut_time() call.
      
      srv_shutdown(): Renamed from srv_master_do_shutdown_tasks().
      
      srv_master_thread(): Do not resume after shutdown has been initiated.
      449a88e1
  19. 23 May, 2017 3 commits
  20. 19 May, 2017 3 commits