include/common_fwd: Include Crypto classes

CryptoManager::cct is now used in CephContext ctor. To provide this
defintion
any ceph_context.cc target must also include Crypto.cc.

crimson-alien-common library which only had ceph_context.cc must now
also include Crypto.cc.
However, the fact that crimson-common also includes Crypto.cc would
cause multiple defintions
to any Crypto classes methods.

To resolve this, let's wrap all Crypto classes with TOPNSPC::common that
would be forwarded using common_fwd logic.

Signed-off-by: Matan Breizman <mbreizma@redhat.com>

diff --git a/src/auth/Crypto.cc b/src/auth/Crypto.cc
index cfb90feae9bec512be53f2866ba33cfee008ea5a..dd3e0f9b710bc71b7c59da55fea3e5f4e8fc63be 100644 (file)
--- a/src/auth/Crypto.cc
+++ b/src/auth/Crypto.cc
@@ -74,6 +74,8 @@ static bool getentropy_works()
   }
 }
 
+namespace TOPNSPC::auth {
+
 CryptoRandom::CryptoRandom() : fd(getentropy_works() ? -1 : open_urandom())
 {}
 
@@ -103,6 +105,8 @@ void CryptoRandom::get_bytes(char *buf, int len)
 
 #include <bcrypt.h>
 
+namespace TOPNSPC::auth {
+
 CryptoRandom::CryptoRandom() : fd(0) {}
 CryptoRandom::~CryptoRandom() = default;
 
@@ -1153,5 +1157,7 @@ bool CryptoManager::crypto_type_supported(int type) const
   return supported_crypto_types.find(type) != supported_crypto_types.end();
 }
 
+} // namespace TOPNSPC::auth
+
 #pragma clang diagnostic pop
 #pragma GCC diagnostic pop

diff --git a/src/auth/Crypto.h b/src/auth/Crypto.h
index 4665a0044f1b1f17b3a6d34b93fd449b89551aed..792b24e79cf4814212b2143bb96ef169dc728e7b 100644 (file)
--- a/src/auth/Crypto.h
+++ b/src/auth/Crypto.h
@@ -25,6 +25,8 @@
 class CryptoKeyContext;
 namespace ceph { class Formatter; }
 
+namespace TOPNSPC::auth {
+
 /*
  * Random byte stream generator suitable for cryptographic use
  */
@@ -237,6 +239,7 @@ inline std::ostream& operator<<(std::ostream& out, const CryptoKey& k)
  * To use these functions, you need to call ceph::crypto::init(), see
  * common/ceph_crypto.h. common_init_finish does this for you.
  */
+
 class CryptoHandler {
 public:
   virtual ~CryptoHandler() {}
@@ -256,6 +259,7 @@ public:
 };
 
 
+
 class CryptoManager {
   CephContext *cct;
   std::shared_ptr<CryptoHandler> crypto_none;
@@ -275,6 +279,6 @@ public:
 
   std::shared_ptr<CryptoHandler> get_handler(int type);
 };
-
+} // namespace TOPNSPC::auth
 
 #endif

diff --git a/src/crimson/os/alienstore/CMakeLists.txt b/src/crimson/os/alienstore/CMakeLists.txt
index 18d9d8b99dccb7f90e91921760a5b8e6ca1d872b..29a325f7a7b2e1baa26a8de875bf386a4f40a0ac 100644 (file)
--- a/src/crimson/os/alienstore/CMakeLists.txt
+++ b/src/crimson/os/alienstore/CMakeLists.txt
@@ -1,6 +1,7 @@
 include_directories(SYSTEM "${CMAKE_SOURCE_DIR}/src/rocksdb/include")
 
 set(crimson_alien_common_srcs
+  ${PROJECT_SOURCE_DIR}/src/auth/Crypto.cc
   ${PROJECT_SOURCE_DIR}/src/common/admin_socket.cc
   ${PROJECT_SOURCE_DIR}/src/common/url_escape.cc
   ${PROJECT_SOURCE_DIR}/src/common/blkdev.cc

diff --git a/src/include/common_fwd.h b/src/include/common_fwd.h
index 3c449513854c10e26e8eca299f2d3cb0d55f0807..6b38e708f3d88be9e1d487b19d62923469e82ed2 100644 (file)
--- a/src/include/common_fwd.h
+++ b/src/include/common_fwd.h
@@ -19,6 +19,14 @@ namespace TOPNSPC::common {
   class RefCountedWaitObject;
   class ConfigProxy;
 }
+
+namespace TOPNSPC::auth {
+  class CryptoManager;
+  class CryptoHandler;
+  class CryptoKey;
+  class CryptoRandom;
+}
+
 using TOPNSPC::common::CephContext;
 using TOPNSPC::common::PerfCounters;
 using TOPNSPC::common::PerfCountersBuilder;
@@ -30,3 +38,7 @@ using TOPNSPC::common::RefCountedObjectSafe;
 using TOPNSPC::common::RefCountedCond;
 using TOPNSPC::common::RefCountedWaitObject;
 using TOPNSPC::common::ConfigProxy;
+using TOPNSPC::auth::CryptoManager;
+using TOPNSPC::auth::CryptoHandler;
+using TOPNSPC::auth::CryptoKey;
+using TOPNSPC::auth::CryptoRandom;