Add internal TC: for Base64, DataType

Change-Id: Ic5bdcd1298e1b76c37ee69f58dff2b7dc39fbcdf
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Clean up move/copy assignment/constructor

Change-Id: If87eacaa85ac5b7d11cede5a256c62e4e71cc935
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Clean up old dpl core sources

errno to string function is too heavy. make it light-weighted
Use dpl log to print assert message and unhandled exception
dpl log can print to several provider(console, journal, dlog) already
by modify configuration file

Change-Id: Ib2e090a0e1c5aafa51bde40c73030b435ae1a1e8
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Remove unused functions in certificate-impl

Change-Id: I343f14a7fa076ea8c7f744b5aa6c2c4babe70633
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Refactor client-capi manager as c++ style

Change-Id: If26aab66bc2b8e4fdfb14c62d9c79300d8af61e0
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Remove useless CKMC error -> CKM error converter

Change-Id: Ia8fcfd5424d2886ffcc535220b301c1bb9ea8078
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Add for_each files handling style when reading dir

Change-Id: I41ecf62acf6277db6651fdbf3ac5b0eb4761f005
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Fix SVACE defects

Use thread-safe functions
Initialize values in constructor
Catch all exceptions

Change-Id: I7ce649b7ba1a11e45949e8f8fca257be4eb7f37d
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Hotfix: image creation failed

/usr/sbin/ldconfig cannot be found.
Use /sbin/ldconfig as it was.

Change-Id: Ieb38a62b2474ae3b89c0305c5bfb20bd9c4dbe9f
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Version 0.1.22

- Fix SVACE defects
- Remove hard-coded paths
- remove dependency from pwdutils -> user/group manage backup plan given up for now...

Change-Id: I91ede36bcbc017a067783fbbf46a6c919cf6c717
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Sync error code description with common package

platform/core/api/common error_message/key-manager.xml

Change-Id: Iae51652c580f4b3ccf4fbd2dec261e97a0a04bcd
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Add description of changed priv on ocsp check API

Change-Id: I8247cd2fd48c973528d801cd3347d963dfa8ade0
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Fix unsafe buffer usage

- sprintf
- strcpy

Change-Id: I85716d6daabc149526146dfe375874a7057550a2
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Remove all of hard coded paths

Change-Id: Ib829bab36e177d36c6093707e5212acc0a82bcf7
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Deprecate key-manager user/group manage backup plan

pwdutils package would be excluded from binary

Change-Id: Ia9bcd5c3e35f319f7ad9810ef5ebbd00a976c1bb
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Remove usage of _datadir macro

TZ_SYS_SHARE is changed to /opt/share by tizen-platform-config commit:
Change: https://review.tizen.org/gerrit/#/c/59623
commit id: bde2d5558c63ce281ea85c294be1816783612f92

Change-Id: I6b00ba7b39e10b30e19ec0346a5aef20c931a9b8
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Version 0.1.21

- Translation of label <-> pkgid made by Security Manager
- Add BSD license for SQL Cipher

Change-Id: I40fadb12f5472a99624ddf342a8b480fc1bf3dba
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Add BSD-2.0 license for sqlcipher

Refer SQL Cipher license info in https://www.zetetic.net/sqlcipher/license/

Change-Id: I54e9569cd529b65e4af8be857c7ac33dd822012b
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Change log level when smacklabel isn't found by SM

Change-Id: Ic06d8346a2fa5e9181b5abbef97ecf49101b5ace
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Fix hard-coded paths and test res installation

Change-Id: Icaf1b070b8fbbc5368643d53755a759f98ed3245
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Add getting pkgId form socket using new SM API

Change-Id: Iabb1c021fd98c3998b4f7031f042d1c35a611fa6

Version 0.1.20

Hotfix: Smack label/type setting failed when installs rpm

Change-Id: I9029495356b1b9d4e5b0c09fdb48adf24b08115c
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Remove '/' from the end of rw_data_dir for manifest

filesystem path attribute on manifest file cannot parse
corretcly with '/' at the end of path. Smack label/type couldn't
set with it.

Add %dir directive on file list on %files. It's for removing all
resources and directories clearly when rpm update/remove.

Change-Id: I6bca24de763cc6754cd3c5a77cbf7a4ac6e86970
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Version 0.1.19

Change-Id: I10a96d85897401716a053a795de1d9d609416c3f
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Use internal package manager client API

capi-package-manager API is for application layer.

Change-Id: Ia07e0e3514303a62df6d723303d46c5bec791876
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Remove empty listener rpm

Change-Id: I2dec628ffee5af467a5d31eb81a8b2e3909186f5
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Get dir locations from environment

[Problem] Key manager data dir location was hardcoded to /opt/data/ckm/. On
common profile image key-manager fails to start because of lack of /opt/data.
[Solution] Get directory locations from tizen platform config. Copy files
from old location if necessary. Make installation fail if runtime and compile
time settings differ. Provide upgrade script.

[Verification] Install key-manager package. Make sure all the content from
/opt/data/ckm/ has been moved to new location (/usr/data/ckm). Run tests. Make
sure all data is placed in new location. Run upgrade scripts
(/etc/opt/upgrad/*key-manager*.sh) on old version of ckm and check results.

Change-Id: Idafbe0fe43f1140c137f87883273b609a499b4cc

Not to set DKEK value to DEK struct salt attribute

DEK struct salt value is derived from DKEK. It's not
clear to be stored on multiple data which is security-sensitive.

Change-Id: Ie3684e350d12dce132cb9425de3b075e25dbb63e
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

change "protected by a user password" to "protected by Tizen platform" in a document file

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

Integrate glib loop with key-manager.

Change-Id: I218d3794e4405ea668c513b3ba40a6e3f98e044c

Remove key-manager-listener.

Change-Id: I5c56ebadc865ff61b18f7fa6235963aa0d3b4376

Move data dir labeling & chowning to key-manager post

[Problem] Key-manager data dir was labeled and chowned in key-manager-listener
post install section.
[Solution] Moved to key-manager post section

[Verification] Install key-manager. Smack shouldn't report any denials to data
directory.

Change-Id: I345716a676f6abaa8670911e1ec7af3e57e9cb54

Change user from root to key-manager

[Solution] User changed. Smack labels/manifests adjusted. Runtime directory
created. Tests adjusted.

Change-Id: I22b7ed01158b16ce3ac3d04110e4ab2ab3d46711

Fix file permissions.

Change-Id: I8aa70dd580c192d7ec5d2b28ebbcb3d1ee0193a3

Add license header on tests srcs

Change-Id: I9603ef02dec7d541ab0cbd13768ebc8fb0bdc12b
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Add boost license

Change-Id: I08f4157f1653afb7afcc17741ca90892831b39fd
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Apply coding rule

covered Tizen C++ coding rule list

 - [R06] : Do not unnecessarily use braces where a single statement will do.
- Exception: if one branch of a conditional statement is a single statement,
 use braces in both branches.

 - [M08] : Keywords have following space rules
- Put a space after (most) keywords
- example: if, switch, case, for, do, while
- Do not put a space after function like keywords
- example: sizeof, typeof, alignof, __attribute__

 - [M09] : Operators have following space rules
- Put a space around(on each side of) most binary and ternary operators
- Example : = + -< > * / % | & ^ <= >= == != ? :
- Do not put a space after unary operators
- Example: & * + -~ !
- Unary "++" and unary "--" are allowed no space on one side
- Do not put a space around the "." and "->" structure member operators.
- Do not put a space after cast operator

 - [M10] : Seperators have following space rules
- Put a space after closing brace when it has anything on the line.
- Exception : comma after closing brace '},‘
- Put a space after comma
- Do not put space inside parenthesis '(', ')‘
- Do not put space before open square bracket '[‘ and inside square bracket '[', ']‘

 - [M12] : Functions have the opening brace at the beginning of the next line.

 - [M13] : The open brace last on the line. The closing brace is empty on a line of its own
- Exception : else, whileshould follow close brace '}’

 - [M14] : Open braces for enum, union and structgo on the same line

Change-Id: I6e59513ba121a25222e6e626792b3e115ebec1d3
Signed-off-by: sangwan.kwon <sangwan.kwon@samsung.com>

Change exception type in db-crypto.cpp

Change-Id: I537d47a4d6cb4d632a46f0527be17108e1353b4a

Remove duplicated entries in spec

[Problem] Duplicated entries in spec.
[Solution] Removed.

[Verification] Successfull compilation. Run tests.

Change-Id: I2c356b854bee089dc3675b84d7ea32930192dbd1

Fix file copying in scheme test

[Problem] Wrong creation flag was used. Wrong owner used. Files not closed.
[Solution] Change files owner and fix the flag. Close descriptors.

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

Change-Id: I9289f231fc27ff863552c65a1f76627189ed5bdb

Revert "Add PASSWORD_PROTECTION_DISABLE feature"

This reverts commit c960d5d61979802933b7cc44b66a118b5861edf7.

Change-Id: Ia3225c3a460f48dde93a272e931f6a53d461f48c

change log level to warning in case for the failure of FIPS_mode_set

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

Remove digest implementation.

Change-Id: Ib4c38cae9370e3f39ffb59e9d7602238d178c849

Fix implementation of sw-backend.

sw-backend cound not depend from certificate-impl.h and key-impl.h.

Change-Id: I7826f0c94bf18d1ad92ac59820120b6ee45531eb

Detailed documentation for encryption/decryption API

[Problem] The encryption/decryption API documentation in header file wasn't
detailed enough.
[Solution] Add more more details.

[Verification] Rate intelligibility of documentation

Change-Id: I2695651928ccf294e94fed2e4c2d0a4019365582

License date updated

[Problem] Wrong date in LICENSE file
[Solution] Date fixed

[Verification] Compare with current year

Change-Id: Ia9a4d2da70d6f01d5f036c376e220b9637f81870

Conversion PEM -> DER moved to store.

Change-Id: Ia748520a828f1dfcb3097dbde0ac28aaf22a8c52

Load app key when importing initial values

[Problem] Missing app key.
[Solution] Fixed.

[Verification] Run tests

Change-Id: If1a156094bb81a349af41f077443c6503b9cea04

Encrypted Initial Values: decrypting items from XML files.

Change-Id: I08d53475401407c76d8aafbefc0b4d2f4fd82204

Encrypted Initial Values: parsing key values.

Change-Id: Iecebe5cba1ce716e43fff09ddc442a57dcfdf976

Encrypted Initial Values: re-factoring the schema to support new requirements.

New requirements are: support for encrypting data and certificate objects.
This requires providing hybrid encryption: device key encrypts AES key, which
is used to encrypt the values.
Device key cannot directly encrypt values due to block size limitation of RSA OAEP algorithm.

Change-Id: I61f52bad74d7cf2f6018a1e0e38cc852a932619e

Add DataContainer class to contain data, its type and encryption details.

[Problem] Encrypted data details are kept separately to the data.
[Solution] Store provides an interface to handle data, type and encryption details in a single object.

Change-Id: Ic35a0a6205670c2a41a183e56b884be87179ebab

Move call to xmlCleanupParser

[Problem] xmlCleanupParser should be called only before program exit.
[Solution] Moved to global object constructor.

[Verification] Successfull compilation

Change-Id: Id2fedd8b99c9b598eaef14f7c12d37282e74814a

CKMLogic will not depend from InitialValues.

Change-Id: I91ce4dcadd49ae813c1ca59418ea8f730110718b

Merge "Add PASSWORD_PROTECTION_DISABLE feature" into tizen

Add PASSWORD_PROTECTION_DISABLE feature

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

Fix character trimming in xml parser

[Problem] ASCII tag content is being trimmed and it shouldn't be.
[Solution] Don't trim tag content in Characters() callback. If trimming is
necessary it can be done in End() callback. Exception will be thrown only if
non-whitespace characters are found within tags that shouldn't have any
character content (InitialValueHandler, PermissionHandler).

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

Change-Id: I39a928bf15be29ade96986d619c9023fb4cd3234

Fix bug found by Klocwork

[Problem] The command received from encryption service is deserialized into a
variable hiding function argument of the same name. Also the received command
was ignored.
[Solution] Check if received command is equal to requested one.

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

Change-Id: I16e14dbc8497a9b6ea11d93c8c0a48071562d684

try-catch enclosed to be exception safe of CAPI

Change-Id: I8c88402c6ed8f73bb1e5510389fec2aa07cfd48c
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Remove MDFPP related code

Change-Id: I4b2078f2f2ebc8ebbd31fb3b7995eb1807fc3a49
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Protocol refactoring.

Introduce CryptoAlgorithm in internal socket protocol.

Change-Id: I70000a05e0a47d2b12af9b11324adf67da0f5e22

Add encryption scheme tests

[Problem] We need tests that will verify correctness of old and new encryption
scheme support.
[Solution] Tests added.

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

Change-Id: I9f4e24a9e06684d401540646d5560287e35b828d

Use exportable=true when reading certificate from db

[Problem] Key manager allows creating a cert chain from not exportable
certificates.
[Solution] CKMLogic::readCertificateHelper modified to use exportable flag
equal to 'true'.

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

Change-Id: Ib13811282eb9d1267c26741a578d8c2111bdecbb

Return error if password is not empty and row is not password protected

[Problem] If old scheme row is not password protected and the user tries to
read it with non empty password it will get reencrypted with this password.
[Solution] Throw an authentication exception if password is not empty and row
is not password protected.

[Verification] Run ckm-tests-internal -t ENCRYPTION_SCHEME_TEST/T120_Read_wrong_pass

Change-Id: I44b270dbbefd043b6efb9371f0d7a81c1b234b31

Modify encryption scheme

[Problem] Current encryption scheme makes it impossible to remove an entry
protected with custom user password from database.
[Solution] Encryption scheme modified. Store is responsible for encrypting data
with user password. Service encrypts it with app key. Data encrypted with old
scheme that is being read will be automatically reencrypted with new scheme.

[Verification] Run tests from upcoming commit:
ckm-tests-internal -t ENCRYPTION_SCHEME_TEST

Change-Id: I8ed514290d9e75bbc89d74b006939e3cbb0b8bd2

Add scheme encryption test db generator

[Problem] A database filled with all kind of data is needed for encryption
scheme tests.
[Solution] Add tool that fills the database with different kinds of data.

[Verification] Run ckm-db-generator. Use ckm_db_tool 7654 db-pass to verify
that all types of data is present in db.

Change-Id: If2d912afdfe96a535df98c5a6c03a2acb1c84af5

Add support for binary data to GStore

[Problem] Binary data can be imported into store but can't be retrieved from
it.
[Solution] Introduce another intermediate class in GKey hierarhy to support
binary data.

[Verification] Run tests

Change-Id: I45bf5d0a81188f13b0925e982243fdf37b569529

Version 0.1.18

Change-Id: I33e245d9b8b6b6ca81caa326d24e725c1821987b
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Don't check handle value when allocation

Change-Id: I3ab918652dc294107327bc3840bdd5c80bed0cc6
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

[ACR-429]Remove/Deprecated get cert chain with alias

Change-Id: Ib1f775c98c41ef89d10199c03d17a69b7be46008
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Add gitignore file

Change-Id: I491e88d2454a672b77e207aaf95d945c0b464591
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Version 0.1.17

Change-Id: I93d81a35d0bf4fccb4cafbd823014cdeb4939192
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Remove warnings and performance problems reported by cppcheck.

Change-Id: I6c39ff383a19554da5e9f875db51864e0e5941d0

Remove symbol : ckmc_label_shared_owner

Change-Id: I40ba69c4e64ed7d5c1f7f9750b0a161adbd1aa15
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Fix API description of set param functions

 * Set -> Sets in @brief

Change-Id: Icb573f4f535d98acdd5061a7e473a30237b28f32
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Adjust API change request review result

 * remove useless const keyword in ckmc_create_key_aes
 * ckmc_generate_params allocates new ckmc_param_list_s inside of itself
 * ckmc_generate_params is renamed to ckmc_generate_new_params
 * ckmc_param_list_s is changed to ckmc_param_list_h (handle)
 * handle is structure pointer : typedef struct __ckmc_param_list *ckmc_param_list_h
 * ckmc_param_list_add_integer -> ckmc_param_list_set_integer
 * ckmc_param_list_add_buffer  -> ckmc_param_list_set_buffer

 * new line for too long description
 * use dot properly in description (Should not used in params and retvals)

Change-Id: I760465e1ae0d1665d596ff10d402c5f191a6f1d4
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Fix spec file warnings & error after gbs build

Change-Id: I48fa8e4392c2cb0cdf32f56299701ce420ba042e
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Privilege are changed for ACR

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

API refactoring.

CKM::Manager uses private implementation pattern.
Remove most of virtual methods from CKM::Manager.

Change-Id: I171083a6f81716dc78155242989dd97528079554

Revert "Old privileges restored"

This reverts commit 42a14dd9afaec7949cf4dec5d7be261a43b1e0a3.

Change-Id: Ibcea2dd233286e87cc7570f8cfa68e07b5a8e069

Fix support of new error code: CKM_API_ERROR_NOT_SUPPORT.

Change-Id: I59545191904fce8ee9258861a5a579308ae05216

Revert "Add functions required during db debug."

This reverts commit 4b4f7b9e045fadc3c5348e7ef8be628a742907e2.

Change-Id: Iac9f830b91a6ddfae1245a8b973ef51ed441738d

Revert "Disable optional password protection temporary"

This reverts commit ea896bbce6b7f21772de779faf0f0c29de845a24.

Change-Id: Iffa558fb3e9889acffb09d27d9b237b70cb01aea

Ignore failing row decryption during data removal

[Problem] When data is removed we don't know the custom user password used to
protect the row. The row decryption is performed with empty password and may
fail.
[Solution] Because row will be deleted we can ignore the failing decryption.
This is a temporary solution. It won't work for tz-store. The problem will be
fixed when new encryption is applied.

[Verification] Run TCT tests.

Change-Id: I9c24704a83c5511bd53218738460f2b546c3dd05

Move encrypted/decrypted rows instead of copying

[Problem] Rows are copied in CryptoLogic::decryptRow/encryptRow.
[Solution] Copying replaced with std::move

[Verification] Run tests

Change-Id: I362638d8981bbe8e511b417596f4cb67ae6f058e

Disable optional password protection temporary

couldn't remove data which is protected by optional password

Change-Id: I0a0e67ddcf40bd0d0f90585d58469a950317a6f0
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Secure control socket with DAC = 0700

Only root should be able to connect and use this socket.

Change-Id: I903de6f55e34c8a9fb8dbdbe99108ab501769f6e

Add cynara mockup.

Change-Id: If423f8f88546f551ca35f849371343a37b46fe21

Version 0.1.16

Change-Id: I123912571e4776580e2008bae362a75fc3672ae2

Old privileges restored

Change-Id: I62335aa31fa14bf2712a72605c97ad5e9fed8a09
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Integration with cynara.

Change-Id: I75f727890d37b39e7054db4c68baad922eef1fc3

Add a tool for accessing encrypted database

[Problem] No way of debugging an encrypted database
[Solution] Tool added

[Verification] Run:
ckm_db_tool
ckm_db_tool 0
ckm_db_tool 0 <sql_command>
ckm_db_tool <uid> <password>
    > .tables
    > .schema
    > <sql_command>
    > help
    > exit
ckm_db_tool <uid> <password> <sql_command>

Change-Id: I87662831808b0397b01db1e54c38b4dc4ad69129

Reimplement security-manager mockup.

New version of mockup will try to translate smack label into
pkgId by removing prefix "User::App::".

Change-Id: I54316ec1c8e8061cedf09f19016576d202e1e9f1

Add mockup for security-manager.

Security-manager mockup should be used with unit-tests.
CKM compilation with mockup:
gbs lb -A i586 --define "build_type DEBUG" --define "mockup_sm ON"

Note: It's not possible to compile RELEASE version of CKM with mockup.

Change-Id: I793ec55010b3826dd9d4157ce4e33f555dee14c0

Replace smack label with pkgId.

Change-Id: I2775a65349bf2103cf7de4702572b031244d9f28

Check platform feature(network) before check ocsp

Change-Id: I87c60238b0a1c67c853a5d60f635162bf9375e71
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Fix ckmc headers grammar

Change-Id: I525de480255f5b41d39b4d892239a23cd1e28268
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Remove temporary defined TIZEN_ERROR_KEY_MANAGER

TIZEN_ERROR_KEY_MANAGER is in tizen_error.h in platform/core/api/common package

Change-Id: I51b3907f8b73d74cd44a3439a49bfc7a55c8df56
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Remove useless API : ckmc_remove_pkcs12

Change-Id: Ib56c522f649830eba35267e8b122d0268c5cdb66
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

Add functions required during db debug.

Add Crypto::getSchema() and Crypto::getContent()

Change-Id: I46739eaef891edaa5d4f190a8adcadd2958dcc24