Applied CodeRabbit AI suggestions

Author: antoinelochet

src/lib/crypto/OSSLMLDSAPublicKey.cpp

@@ -37,6 +37,27 @@ OSSLMLDSAPublicKey::~OSSLMLDSAPublicKey()
 	}
 }
 
+OSSLMLDSAPublicKey::OSSLMLDSAPublicKey(OSSLMLDSAPublicKey&& other) noexcept
+    : MLDSAPublicKey(std::move(other)), pkey(other.pkey)
+{
+    other.pkey = NULL;
+}
+
+OSSLMLDSAPublicKey& OSSLMLDSAPublicKey::operator=(OSSLMLDSAPublicKey&& other) noexcept
+{
+    if (this != &other)
+    {
+        // move base
+        MLDSAPublicKey::operator=(std::move(other));
+        // release current
+        if (pkey) { EVP_PKEY_free(pkey); }
+        // steal
+        pkey = other.pkey;
+        other.pkey = NULL;
+    }
+    return *this;
+}
+
 // The type
 /*static*/ const char* OSSLMLDSAPublicKey::type = "OpenSSL ML-DSA Public Key";
 

src/lib/crypto/OSSLMLDSAPublicKey.h

@@ -23,6 +23,14 @@ class OSSLMLDSAPublicKey : public MLDSAPublicKey
 	// Destructor
 	virtual ~OSSLMLDSAPublicKey();
 
+	// Non-copyable (raw ownership of EVP_PKEY)
+    OSSLMLDSAPublicKey(const OSSLMLDSAPublicKey&) = delete;
+    OSSLMLDSAPublicKey& operator=(const OSSLMLDSAPublicKey&) = delete;
+
+	// Movable
+	OSSLMLDSAPublicKey(OSSLMLDSAPublicKey&&) noexcept;
+	OSSLMLDSAPublicKey& operator=(OSSLMLDSAPublicKey&&) noexcept;
+
 	// The type
 	static const char* type;
 

src/lib/crypto/OSSLUtil.cpp

@@ -34,8 +34,10 @@
 #include "log.h"
 #include "DerUtil.h"
 #include "OSSLUtil.h"
+#ifdef WITH_ML_DSA
 #include "MLDSAParameters.h"
 #include <map>
+#endif
 #include <openssl/asn1.h>
 #include <openssl/evp.h>
 #include <openssl/err.h>

src/lib/object_store/DBObject.cpp

@@ -520,6 +520,7 @@ static AttributeKind attributeKind(CK_ATTRIBUTE_TYPE type)
 	case CKA_OS_USERPIN: return akBinary;
 
 	case CKA_PARAMETER_SET: return akInteger;
+	case CKA_SEED: return akBinary;
 
 	default: return akUnknown;
 	}