Skip to content

Commit 26e1506

Browse files
Robert Golebiowskibjornmu
authored andcommitted
Bug #22849991 FIX CLANG WARNINGS AFTER WL#8041 PUSH (USE ORACLE KEY VAULT FOR KEYRING STORAGE)
(cherry picked from commit 3957b74548c31f6d88c79badc538a2180bdefd1c)
1 parent 0f29b2b commit 26e1506

File tree

3 files changed

+17
-8
lines changed

3 files changed

+17
-8
lines changed

plugin/keyring/keyring.cc

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,11 @@
2323
#define MYSQL_DEFAULT_KEYRINGFILE MYSQL_KEYRINGDIR"/keyring"
2424
#endif
2525

26+
using keyring::Buffered_file_io;
27+
using keyring::Key;
28+
using keyring::Keys_container;
29+
using keyring::Logger;
30+
2631
my_bool create_keyring_dir_if_does_not_exist(const char *keyring_file_path)
2732
{
2833
if (!keyring_file_path || strlen(keyring_file_path) == 0)

plugin/keyring/keyring.h

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,11 @@
2323
#include "keys_container.h"
2424
#include "keyring_memory.h"
2525

26-
using namespace keyring;
26+
using keyring::IKeys_container;
27+
using keyring::IKeyring_io;
28+
using keyring::ILogger;
29+
using keyring::IKey;
30+
2731
namespace keyring
2832
{
2933
/* Always defined. */

plugin/keyring/keyring_impl.cc

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,12 @@ boost::movelib::unique_ptr<char[]> keyring_file_data(NULL);
3333
#ifdef HAVE_PSI_INTERFACE
3434
static PSI_rwlock_info all_keyring_rwlocks[]=
3535
{
36-
{&key_LOCK_keyring, "LOCK_keyring", 0}
36+
{&keyring::key_LOCK_keyring, "LOCK_keyring", 0}
3737
};
3838

3939
static PSI_memory_info all_keyring_memory[]=
4040
{
41-
{&key_memory_KEYRING, "KEYRING", 0}
41+
{&keyring::key_memory_KEYRING, "KEYRING", 0}
4242
};
4343

4444
void keyring_init_psi_keys(void)
@@ -56,7 +56,7 @@ void keyring_init_psi_keys(void)
5656

5757
my_bool init_keyring_locks()
5858
{
59-
if (mysql_rwlock_init(key_LOCK_keyring, &LOCK_keyring))
59+
if (mysql_rwlock_init(keyring::key_LOCK_keyring, &LOCK_keyring))
6060
return TRUE;
6161
return FALSE;
6262
}
@@ -96,10 +96,10 @@ my_bool mysql_key_fetch(boost::movelib::unique_ptr<IKey> key_to_fetch, char **ke
9696
{
9797
*key_len = fetched_key->get_key_data_size();
9898
fetched_key->xor_data();
99-
*reinterpret_cast<uchar **>(key)=fetched_key->release_key_data();
100-
*key_type = my_strdup(key_memory_KEYRING,
101-
fetched_key->get_key_type()->c_str(),
102-
MYF(MY_WME));
99+
*key= static_cast<void*>(fetched_key->release_key_data());
100+
*key_type= my_strdup(keyring::key_memory_KEYRING,
101+
fetched_key->get_key_type()->c_str(),
102+
MYF(MY_WME));
103103
}
104104
else
105105
*key = NULL;

0 commit comments

Comments
 (0)