From: Krzysztof Jackiewicz <k.jackiewicz@samsung.com>
Date: Thu, 23 Feb 2023 08:47:44 +0000 (+0100)
Subject: Allow saving unexportable data
X-Git-Tag: accepted/tizen/unified/20230406.165733~5^2~25
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=89c20af31cf39ee3e9bf15fc8ac32a8bed0a3082;p=platform%2Fcore%2Fsecurity%2Fkey-manager.git

Allow saving unexportable data

Now it makes sense as such data can be used for key derivation.

Change-Id: Ie8db9cca5b83831045dce45cfadde8215c79a608
---

diff --git a/src/include/ckm/ckm-manager.h b/src/include/ckm/ckm-manager.h
index 001f72f..68f613c 100644
--- a/src/include/ckm/ckm-manager.h
+++ b/src/include/ckm/ckm-manager.h
@@ -55,10 +55,6 @@ public:
 		const Policy &keyPolicy,
 		const Policy &certPolicy);
 
-	/*
-	 * Data must be extractable. If you set extractable bit to false function will
-	 * return ERROR_INPUT_PARAM.
-	 */
 	int saveData(const Alias &alias, const RawBuffer &data, const Policy &policy);
 
 	int removeAlias(const Alias &alias);
diff --git a/src/manager/client/client-manager-impl.cpp b/src/manager/client/client-manager-impl.cpp
index 19a2815..a4905dc 100644
--- a/src/manager/client/client-manager-impl.cpp
+++ b/src/manager/client/client-manager-impl.cpp
@@ -173,9 +173,7 @@ int Manager::Impl::saveCertificate(
 int Manager::Impl::saveData(const Alias &alias, const RawBuffer &rawData,
 							const Policy &policy)
 {
-	return !policy.extractable
-		? CKM_API_ERROR_INPUT_PARAM
-		: saveBinaryData(alias, DataType::BINARY_DATA, rawData, policy);
+	return saveBinaryData(alias, DataType::BINARY_DATA, rawData, policy);
 }