Version 0.1.14

Change-Id: I3bf2fa3b6a233fca6b46215d7b15a2ce8c3cc8e9

Reverting Tizen 2.x specific workarounds for password change/authtype==none.

Change-Id: Ib888b1df3afc54405cf6a3b48bad86e7fc0c92e4

bugfix: minor memory corruption. Internal tests work.

Change-Id: Ie6cc846ac066a6d86f0d2642a9906c08b4d35068

Key Manager tizen.org session and user management
integration.

Key-Manager integrates with PAM (via pam_key_manager_plugin.so lib
and appropriate configuration changes) and gumd via user removal hook.

PAM configuration needs to be changed to use the .so specified above.
For testing, do the following changes in /etc/pam.d/system-auth:

section password:
* remove pam_deny.so line
* change pam_unix.so from sufficient to required
* add "password    optional      pam_key_manager_plugin.so change_step=before" before the pam_unix.so entry
* add "password    optional      pam_key_manager_plugin.so change_step=after" after the pam_unix.so entry

section session:
* add "session     optional      pam_key_manager_plugin.so" as last item

Change-Id: I2fd29ab527aa3d89c810b9c6d5f74cbbec2e5957

Initial values format adjusted

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Symmetric keys should not hold information about encryption
algorithm. Initial data will be stored as system user but has to be accessible
by ordinary users
[Cause] N/A
[Solution] Symmetric encryption params can be stored separately from key as
data. Encryption params removed from schema. Added permission tag allowing
other users to access system database. XML structure redesigned. Example
updated.

[Verification] Validate example with:
xmllint -schema initial_values.xsd example.xml

Change-Id: I36149b15d6f786e37cec370d632ab74e40efc162

Simplify implementation of ServiceThread

Change-Id: I56ced6bb12e2a6140ab26ab82f9dd68cb2b92b76

Add inter-service communication framework

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Services need to communicate with each other
[Cause] N/A
[Solution] Framework for inter-service communication added.

[Verification] Run ckm-tests-internal -t MESSAGE_MANAGER_TEST

Change-Id: I28714ba52efe25c47402adb6ac1bef52859ed898

Add initial value format schema

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Initial value format needs to be defined
[Cause] N/A
[Solution] Schema and example added

[Verification] Validate example with: xmllint -schema initial_values.xsd example.xml

Change-Id: I5c8979c971e73b07e959e2fdf5d32ee3f9dabf91

Optimize openssl initialization

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Each time Manager or PKCS12 object is created initCryptoLib is called
and mutex is locked inside it.
[Cause] N/A
[Solution] Once openssl is initialized the initalization function pointer is
switched to empty one, thus mutex is not used any more.

[Verification] Run tests. Alternatively check in gdb that client calls
initOpenSSL() only once

Change-Id: I733e4ca6c88a6a51d69ebb0606f560a9b4828e4c

Modify APIs and doxygen to meet ACR(TIZEN 2.4) requirement

Change-Id: I7a883273c6563df23f8e4668d88fbd73d61c2a08
Signed-off-by: yuseok.jeon <yuseok.jeon@samsung.com>

Fix description in ckmc-manager.h

Change-Id: Iceb597c1c8cd10360add0c20a40a2269c53ab2cd

Add symbolic-functions linker flag

Change-Id: I6b014e269f83a48ad516e2b64c1e0de89c546bf9
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Tool for measuring dlopen/dlsym performance

[Issue#] N/A
[Feature/Bug] N/A
[Problem] We need a tool that will show the influence of the number of symbols
and the size of a library on dlopen/dlsym performance
[Cause] N/A
[Solution] Tool added

[Verification] Run ckm_so_loader [library_path] [symbol_to_load]

Change-Id: I524bb20d4a23a5128e83ee42241161ce15fc2092

Globals in LogSystem adjusted to use in lib constructor

[Issue#] N/A
[Feature/Bug] N/A
[Problem] dlopen() fails with client library
[Cause] The order of global variables construction in common library is unpredictable.
[Solution] Global variable made member. Strings replaced by const char* const.

[Verification] Use ckm_so_loader 2 100 /usr/lib/libkey-manager-client.so ckmc_save_key

Change-Id: I0add0c1fe3c66ac9d42a94b7e59bf21cadecdefc

Fix serialization implementation to support 32 and 64 platform.

Change-Id: I3bf8c4bf1c1fa369ea9b0ba1aa20edfe9228f0d9

Removal of unused build artifact "key-provider".

Change parameters of ckmc_get_pkcs12 function.

New version supports additional passwords that may be used
to secure private key and certificates.

Change-Id: I809e5fbbd090e4ee793745e68256915144bb1cd2

Use _toCkmCertificateVector in pkcs12 client CAPIs

Change-Id: I21caca7f9c39dc5e372977e3a4891e1c71d99c22
Signed-off-by: kyungwook tak <k.tak@samsung.com>

CKM FileSystem versioning with file name format update mechanism

 * DKEK format releaseed on kiran
     (key-<uid>-<autoincreased num>)
 * DKEK format on version 0.1.13
     (key-<uid>)
     (key-backup-<uid>)
 * DKEK format on tizen 2.4 which has container feature
   (not merged from knox-tct branch yet,
    so not included about it in this commit)
     (key-<zone name>-<uid>)
     (key-backup-<zone name>-<uid>

Change-Id: I5ce62528d54268cccb7f9705daf0793aec782513
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Add support for password in Manager::getPCKS12 function.

In function savePKCS12 user may specify passwords to protect
data. Function getPKCS12 wasn't support passwords so it was not
possible to extract PKCS12 secured with this functionality.

Change-Id: I542873b817a2bff1064b2b56254d14fb632d8bdf

unlock with password when resetPassword called in case of first start of device

Change-Id: I536b7b5ff2448990bd0c5fdda87730b34e13c16f
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Fix for gcc4.8 (-ldl)

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Linker fails when gcc4.8 is used
[Cause] Undefined symbols from dynamic linker library because of missing -ldl
option
[Solution] Add -ldl option

[Verification] Successfull linkage

Change-Id: Ida7784fddd9caa92c1a23cb50c5025f257ae7020

Common logging setup for client and service

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Client may use different logging method than service.
[Cause] Service reads environment flags from config file. Client doesn't
[Solution] Make the client read that file too and setup log system properly.

[Verification] Make changes to /etc/sysconfig/central-key-manager file and see
if both service and client uses the same logging setup (provider and log level)
File format is the following:
"
CKM_LOG_PROVIDER=<provider>
CKM_LOG_LEVEL=<level>
"
where:
<provider> is one of JOURNALD, DLOG, CONSOLE
<level> is <0..5>, 0 means not logs at all, 1 means errors only, 5 means all

Change-Id: I1662fe636f9987778345f8a02afa6fb77f7f1fe0

Libraries reorganized to limit the number of exported symbols

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Too much exported symbols
[Cause] Some of the code don't have to be exported or is used by a single binary.
[Solution] Unnecessary exports removed. Part of libkey-manager-common code
moved to client library or key-manager binary

[Verification] Compile key-manager and security-tests. Display the number of
exported symbols before and after with:
nm -g <library>.so | wc -l

Change-Id: Iccb053af2523368d353693247e505a794e783318

Add support for AUTHENTICATION_FAILED code in getData function.

Function getPKCS12, getKey, getData will return proper code when
password does not mach.

Change-Id: I8b506f6c03f7acc5421278360cd839d059b914c2

Symbol visibility changed from default to hidden.

Change-Id: I9b4b7e8af5ff62cd8c063a0ce45a899f166566d7
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Version 0.1.13

Change-Id: I1a7c7abb788ef647bd5e3137011484dc539d4771

Add support for new error code in ckmc API

Change-Id: I111c8b64da39e3a19e5fac144e94a5516b389a32

Deprecated access control API fixed - proper mapping to permissions.

[Issue#] N/A
[Feature/Bug] bug: deprecated access control API not working.
[Problem] deprecated access control API incorrectly mapped given values into permissions.
[Solution] added translation mechanism between old access rights into permissions.
[Verification] compile, run updated test set.

Change-Id: If26c69160a79439774a8ffd800809c0a6f7f85e5

DB related classes moved into CKM::DB namespace.

Change-Id: Ifbf70ffe6865793394d46ea6443f27a0062fe02d

Fix logs in internal tests

[Issue#] N/A
[Feature/Bug] N/A
[Problem] No logs from internal tests
[Cause] LogSystem tag was not set
[Solution] Internal tests refactored and cleaned up. Proper tag set.

[Verification] Run internal tests and see if logs are visible

Change-Id: Ibb8517bad710d06a62ba9ba7fbc7b9b8ed7b7c21

Add file, line & function information to journald log

[Issue#] N/A
[Feature/Bug] N/A
[Problem] File, line & function not visible in default journalctl log
[Cause] Default log format does not display this information and other formats
are unreadable
[Solution] File, line & function information added to log message content

[Verification] Create /etc/sysconfig/central-key-manager with following content
"
CKM_LOG_LEVEL=3
CKM_LOG_PROVIDER=JOURNALD
"

Restart the service and see if journalctl logs contain file, line & function
info:
journalctl -f -u central-key-manager

Change-Id: I01389eda9f7db390f6ca00c8f44e1a5c097e59c8

Remove unused key-manager-util.*

[Issue#] N/A
[Feature/Bug] N/A
[Problem] N/A
[Cause] Unused files
[Solution] Removed

[Verification] Successfull compilation

Change-Id: Ie71d5fd9b8093f262643efe5ad7930ea0d5ff53e

Fix LogSystemSingleton issue

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Logs from key-manager-common not visible
[Cause] 2 instances of LogSystemSingleton were created in key-manager binary
(log.cpp and key-manager-main.cpp)
[Solution] Excessive LogSystemSingleton definition removed.

[Verification] Compile & install ckm & tests. Run dlogutil and see if all logs
are visible (CKM-CLIENT and CKM including logs from key-manager-common library)

Change-Id: Idf508facc33318030db90633afa581d5cf012747

Add CKM database versioning with DB migration mechanism.

Change-Id: I3d773b1b9ff4949a4ae98e25c778e6c010bc8a62

Replace error codes with exceptions in FileSystem module.

Change-Id: Id099e6a3cb5b6db69dedc19c781db4d85f7518a9

Certificate chain in OCSP is assumed to be valid

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Not all certificate chains can be properly verified with OCSP
[Cause] Certificate chain in OCSP verification may contain custom trusted root
CAs as well as untrusted certificates which are not taken into account by
current implementation of OCSP.
[Solution] Chain submitted for verification is treated as valid (i.e. created
with get_certificate_chain() API) and therefore all issuers preceeding a
certificate being currently verified with OCSP are treated as trusted and are
used for OCSP response verification.

[Verification] Run ocsp tests

Change-Id: Ia96e6ba830abfd121f9adc041c55789cbf919cbc

Simplify conversion from RawBuffer to C API objects

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Unnecessary casts and allocation were used during conversion from
RawBuffer to C API objects
[Cause] N/A
[Solution] Conversion simplified

[Verification] Successfull compilation. Run tests

Change-Id: I85d2f3d52dba5e440095be0eab1a924395ce2437

Vconf key used only when sec_product_feature_mdfpp_state_enable 1

Change-Id: I447c3fdbd6617537a7767afa015b0f3065aac6ac
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Add line wrapping and fix formatting in C API

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Too long lines. Unreadable API declarations.
[Cause] N/A
[Solution] Lines wrapped to 100 chars. Common function argument wrapping policy.

[Verification] Successfull compilation.

Change-Id: Ic3a9e732dc311662eb8b18640102ecca952e8b5d

Remove deprecated C++ API for certificate chains

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Old C++ certificate chain API is no longer necessary
[Cause] N/A
[Solution] API removed

[Verification] Successfull tests compilation

Change-Id: I588a99d1d211607dcd70290b9b0610b4732d8ef0

New certificate chain API

[Issue#] N/A
[Feature/Bug] N/A
[Problem] N/A
[Cause] N/A
[Solution] API that allows contol of trusted/untrusted certificates used in
chain creation.

[Verification] Run certificate tests

Change-Id: I390d5889ee9221b3a5ffb5c3d3d501b10e3640d4

Fix error connected with url parsing in ocsp module.

Change-Id: Ief450b4bb645fac652d0d270654a0ec883810f2c

Fix label-related logic during permission setting

[Issue#] N/A
[Feature/Bug] N/A
[Problem] The logic of setPermissionHelper was not accurate. It worked because
in our special case the owner==client.
[Cause] N/A
[Solution] Logic fixed to address generic cases. Few comments added

[Verification] Run tests

Change-Id: I853cb523fb5309600a0f54df8efa65bc910e83cc

Add OCSP check CAPI

Change-Id: I41876e3c8a3ea33c1a9eb200bc9467571b83940b

Create bioLogger in OCSP module.

Change-Id: Iefcbaad56040df18e60bb09147d827de1ad14c38

DB re-factor: added VIEW joining all tables.

Restrictions applied on VIEW result.

Change-Id: I9b4b5962dd47327e39415db0de24896486534308

CAPI cleanup: control "owner" removal, set_permission replaces allow_access.

Change-Id: If56f8336d8782fffe5e4982ac5a86d7b59dc2682

Version 0.1.12

Change-Id: Icd0a80393197637d3b839cb9bbc1bcd42ebb9424

fix for x509 cert chain verification (stric flag added when CC Mode enabled)

Change-Id: I679b1210d94c721343fc851c8a2b51ac9765635e
Signed-off-by: yuseok.jeon <yuseok.jeon@samsung.com>

Add transaction during password change.

Change-Id: Ic236a84959d339ddc19b2f3e8078766cd97545a7

Fix macro definition which is not following grammar

Change-Id: Idf7268bd147fee00e30a3714114f4b65d5a9a0fa
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Version 0.1.11

Change-Id: I77965c93ed03f27e16f35768ce93c985cd049945

Add PKCS12 support.

Change-Id: I389c56611fed80cf27bbbfa279b1c17e55572b14

Prevent desynchronization in "battery remove test".

This commit force to save encrypted data as soon as possible.
There is still small time window then desynhronization may happen.
Desynchronization may occure if you take out battery exectly after the
data was saved by ckm and the confirmation was not send to
security-server yet.

Change-Id: Ib4d4f0299001d9c71b13acdcfa136298d942ab6c

DBDataType refactoring.

Change-Id: I080034c971ae5d8f75747d2091fb34ca0a7c78fa

C++ API change: common method for removing Alias.

No need to deliver separate methods to remove keys, certs or data.
Item is unambiguosly identified by <name, label> pair.

Change-Id: Ice2c418d66fd4f4b776ac752d8d5c6a779ad3b5a

OpenSSL initialization is done by anyone who uses OpenSSL first.

This resolves issue when CKM::PKCS12::create() is called before client is initialized.
Client was initializing the OpenSSL so parsing PKCS12 was failing due to uninitialized OpenSSL.

Change-Id: I809af1f622ce7c0f4764172c2143773629c88b67

Fix for duplicates in all alias list query

[Issue#] N/A
[Feature/Bug] N/A
[Problem] DBperfGetAliasList fails
[Cause] DB_CMD_NAME_SELECT_BY_TYPE_AND_PERMISSION query is incorrect. When
listing objects accessible by given label it will produce one result for every
owned object times the number of accessors allowed.

Example:
Label L1 is owner of object (L1 N1)
Labels L2 and L3 have read permission to (L1 N1)
Listing all aliases accessible by L1 will produce:
L1 N1 (access by L2)
L1 N1 (access by L3)

[Solution] Add GROUP BY clause clause to get rid of duplicated objects

[Verification] ckm-tests-internall succeeds

Change-Id: I63f0d576750038e0abc1d7d41921e05d87cadfe4

New DB layout: CKM_TABLE split into NAME_TABLE and OBJECT_TABLE.

Change-Id: I3a3666b2a928f45294adf97548bac09d54d133de

Simplified code in ckm-logic.

Change-Id: I35d54422a88d075163509f2437d9c72d8e2a5006

Service denies attempt to add data using different label.

The same applies to attempt to modify another label's permissions.

Change-Id: Ib167de4b4ccb59271f2dcfdbf7ce049f3d3dc819

Remove deprecated comments and logs.

This logs are no longer required to during ckm tests.

Change-Id: I74811ad7a14026bbb5fce09eeac421fac549019f

Use AliasSupport in SaveData

Change-Id: I449e49aea2b52a9595d9e1ba8afcf47af3a7d69f
Signed-off-by: kyungwook tak <k.tak@samsung.com>

DBCrypto access control re-factor: access control moved into additional layer.

Change-Id: I3ea1ce1a858b69c4a5a7365dffa1344b64aab0b6

Add cmake flags to support ASLR

Change-Id: Ie074e69a71e14c40f79a83f982c970d6a15b303f
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Housekeeping: KLOC warnings removed.

Change-Id: I36a972fd2e9dbd7f4e5fc11647badaf0173a425d

Bug fix for application DEK memory push

Change-Id: Ia874cac4e57312ed1be99e8ab66142d3f12d1ef0
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Minor re-factoring: boost::vector replaced with std::vector.

<vector> header cleanup.

Change-Id: I8408a4943f22e5fb18a08c40e9e0b5f9c938b34c

Add journald logging and refactor log system

Add journald log provider. Simplify log system and log providers. Add support
for environment flags.

[Verification] Compile and run different build configurations with different
CKM_LOG_LEVEL and CKM_LOG_PROVIDER values. For client apps just export desired
log level/provider as CKM_LOG_LEVEL/PROVIDER env variable and run it. For ckm
service use:

'echo "CKM_LOG_LEVEL=X" > /etc/sysconfig/central-key-manager'
'echo "CKM_LOG_PROVIDER=X" >> /etc/sysconfig/central-key-manager'

and restart ckm service.

Supported CKM_LOG_LEVEL values: <0,5>
Supported CKM_LOG_PROVIDER values: {CONSOLE, DLOG, JOURNALD }

Observe logs.

1.Release build
 -by default (CKM_LOG_LEVEL not set) only errors should be logged
 -by default (CKM_LOG_PROVIDER not set) dlog is used
 -logs can be disabled completely by setting CKM_LOG_LEVEL to 0
2.Debug build
 -by default Debug and higher severity logs should be logged
 -by default (CKM_LOG_PROVIDER not set) dlog is used
 -log level can be adjusted to any level <0,5>

Change-Id: I496d12309360354399cf5011680f081ce42ee58f

Secure logging removed

Change-Id: I926178d7ce975cc64c828585beb75b445fded065

Determine MDPP Mode Disabled to CC mode On

Change-Id: Ibb97e38e60a4165a3d418dd4e89f06708bafdecb
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Modify key-manager-listener smack label & rule

 * smack label of daemon should not be floor(_)
 * smack label of listener is changed to key-manager-listener
 * smack rules which are needed for using package-manager API is added

Change-Id: Ic7c8a0c32e6241b5ee9e11c6ed6a23725dcb44fa
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Remove daemonize codes in listener daemon

 * remove meaningless daemonizing part of key-manager-listener
 * listener daemon is managed by systemd

Change-Id: I0fc41d1a1d9449bbe1451272838d735a8728b1a3
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Remove duplicated error explanations

Change-Id: Ia7b43684e5bc0a70c21b5fe0c4366aeb141d7bf5
Signed-off-by: VBS <yuseok@samsung.com>

Version 0.1.10

Change-Id: I246149610dbc892a8e36481af0b853e84cd214ee

Divide alias into name & label in getCertificateChain: code re-factor.

Change-Id: I373a87f9421dac3b5e43e8c835d99944676822b1

Divide alias into name & label in getCertificateChain

Change-Id: Iacc02c6bd3b082779aa51c4cd296928c3f7f05d3
Signed-off-by: kyungwook tak <k.tak@samsung.com>

Version 0.1.9

Change-Id: I9055033b9742536b25b8c1a48488a85e718cbedf

Listener service dependency fixed

[Verification] Restart listener and see if journalctl complains about service
configuration

Change-Id: Ib7718ded2440677e2fc06ed732e9d7cbb8f38a95

Lock file operations fixed in listener

Change-Id: If94b0167477306d1af4ea5c60a8b17d83fe13b39

Add file lock to key-manager server

[Verification] Try to run 2 instances of key-manager. It should fail.
key-manager-listener should not start key-manager.

Change-Id: Ifa0d6244738b4a48b153e066d5b49fbd967a316a

Helper functions for log/exception message concatenation

Change-Id: I0c7d1146009924f7765af5f3602b46e2e8c8a094

Refactoring ServiceConnection.

* Remove private inheritence from SockRAII
* Rename Connect to prepareConnection and change logic.

Change-Id: I791e3ec2791c155a7da17a3e1d45218de8351b39

Refactoring SockRAII

Change-Id: Ib264bb049ed52d644f2d5873dabcd7be532738a3

Remove doxygen warning

Change-Id: I4a7c90495e6d8653a77edebb4f5fdcea54684d16
Signed-off-by: yuseok <yuseok@samsung.com>

Modify the setting value of POLL_TIMEOUT

Change-Id: I6c9da2626c5b7af518bad6baba4e9c7c51adb192
Signed-off-by: yuseok <yuseok@samsung.com>

C++ API re-factored: establish and keep socket connection to the service.

C API still does create and destroy socket per each API call.

Change-Id: I99f90da5dca8e7ddc615b7d39d8a0a26055984d6

Split alias at server side into pair name and label.

Change-Id: I5cbe62ee49042449127218a95f82db9b59105a73

CC mode logic updated

[Issue#] N/A
[Feature/Bug] N/A
[Problem] CC mode was not checked during key-manager startup. CC mode was passed
via API
[Cause] N/A
[Solution] CC mode checked during key-manager startup. CC mode is being read by
key-manager.

[Verification] Run ckm-tests --group=CKM_CC_MODE

Change-Id: I5b861c53301893773559eca10722604fa24bd6ec

Fix build break on 64bit platforms.

systemd configuration files should be placed in /usr/lib regardless 32bit
or 64bit architectures.

Change-Id: I4baabff03ce8ba14c8c2cfc73ae8c98262345311

remove the description for alias. It will be added later

Change-Id: I52cfb00e528823716afc331dddd5578b1e46351e
Signed-off-by: Dongsun Lee <ds73.lee@samsung.com>

Vesion 0.1.8

Change-Id: Id31b41f64a2746fda3dbb09371477ad08a7eca71

Revert "Alias is not unique user-wide: (alias, label) pair is unique now."

This reverts commit 44333f4dcbcc9e295cde3a2846ffb70c0732b622.

Change-Id: I9aa9fde63082ce395cb54590f74d9e554d992139

Fix spelling mistake in socket files.

Change-Id: I13279239a297a5da7b97b77a689d91860a684321

Alias is not unique user-wide: (alias, label) pair is unique now.

Client can address items using label::alias syntax.

Change-Id: Ib9429e652e8a73d2d6c62a4164d54229e85cf7d5

Add service-side key and certificate verification when saving.

Change-Id: I93b16a73a59d3b8af041715754836b5bc58dcde7

Valgrind warnings removed: all changes were false-positive issues.

Change-Id: I7ff0e4e6ab21ab570fddc9a63432a399c4ed402b

Replace some functions with thread safe version.

Change-Id: I749eaea647c39d7b25125ed72feb5ee22ddc38dd

Fix package description in spec files.

Change-Id: Icc6d20cccda0fe2a0f1c4491be4ba53790787f51