Sergei Golubchik added a comment - 2015-05-09 22:35 This is not a bug, it's intentional. You've requested InnoDB to encrypt the data and didn't load an encryption plugin. InnoDB tries to get an encryption key, fails, and crashes the server (which is a typical InnoDB reaction to errors). Despite being intentional, I agree that it's not good. I'll try to do something about it.

Elena Stepanova added a comment - 2015-05-10 00:47 I find it somewhat weird – I mean not technically, but logically – that it only fails with innodb-encryption-threads, but doesn't fail if I only set innodb-encrypt-tables. But anyway, please at least make the assertion more comprehensive for a user, or somehow else print a clear error message in the log.

Sergei Golubchik added a comment - 2015-05-10 11:26 - edited jplindst , could you please look at it? I've fixed the assertion rather simply: diff --git a/storage/xtradb/handler/ha_innodb.cc b/storage/xtradb/handler/ha_inn --- a/storage/xtradb/handler/ha_innodb.cc +++ b/storage/xtradb/handler/ha_innodb.cc @@ -3633,6 +3633,14 @@ innobase_init( } #endif + if ((srv_encrypt_tables || srv_encrypt_log) + && encryption_key_get_latest_version(FIL_DEFAULT_ENCRYPTION_KEY) + == ENCRYPTION_KEY_VERSION_INVALID) { + sql_print_error("InnoDB: cannot enable encryption, " + "encryption plugin is not available"); + goto error; + } + os_innodb_umask = (ulint) my_umask; /* First calculate the default path for innodb_data_home_dir etc., And now I get a crash with --innodb-encryption-threads=4 but without --innodb-encrypt-tables . Assert in buf_page_io_complete() , because the page is corrupted. And the page is considered corrupted in here: /* declare empty pages non-corrupted */ if (checksum_field1 == 0 && checksum_field2 == 0 && *reinterpret_cast<const ib_uint64_t*>(read_buf + FIL_PAGE_LSN) == 0) { /* make sure that the page is really empty */ for (ulint i = 0; i < UNIV_PAGE_SIZE; i++) { if (read_buf[i] != 0) { return(TRUE); } } return(FALSE); } Because the page looks like this: (gdb) p read_buf[0] @ UNIV_PAGE_SIZE $1 = '\000' <repeats 32 times>, "\262\326", '\000' <repeats 16349 times>

Jan Lindström (Inactive) added a comment - 2015-05-11 13:27 Could not easily repeat with above fix. Possible fix candidate anyway: diff --git a/storage/xtradb/handler/ha_innodb.cc b/storage/xtradb/handler/ha_innodb.cc index 47659fb..c89e317 100644 --- a/storage/xtradb/handler/ha_innodb.cc +++ b/storage/xtradb/handler/ha_innodb.cc @@ -3633,6 +3633,21 @@ innobase_init( } #endif + if ((srv_encrypt_tables || srv_encrypt_log) + && encryption_key_get_latest_version(FIL_DEFAULT_ENCRYPTION_KEY) + == ENCRYPTION_KEY_VERSION_INVALID) { + sql_print_error("InnoDB: cannot enable encryption, " + "encryption plugin is not available"); + goto error; + } + + if (!srv_encrypt_tables && srv_n_fil_crypt_threads) { + ib_logf(IB_LOG_LEVEL_INFO, "encryption disabled, " + "setting number of encryption threads to 0"); + srv_n_fil_crypt_threads = 0; + } + +

Sergei Golubchik added a comment - 2015-05-11 22:20 Cannot repeat this anymore. May be it as a broken build or some later change fixed it.