Merging tizen into ckm. Stage 3.

Merged remote-tracking branch 'tizen.org/tizen' into ckm

Change-Id: Iffa2e26113f653cf4c6f685bc7e4dc7d25472ae5

Merging tizen into ckm. Stage 2.

Merged commit '7019d2b04887994ff0f87ab8020cc367ffe1994a' into ckm

Change-Id: I80f23e4825da2bb7167d43863e6ed1331a5f5776

Rename /tests to /ckm to align with tizen branch

[Issue#] N/A
[Feature/Bug] N/A
[Problem] N/A
[Cause] N/A
[Solution] Renamed

[Verification] Compile, run tests

Change-Id: Ic24c4b694de04e44ceb6947f6628872f09def079

Merging tizen into ckm. Stage 1.

Merged commit '4f20901f76f8909da0d07f8b5f8f33e8651ea135' into ckm

Conflicts:
tests/common/dbus_access.cpp
tests/common/dbus_access.h
tests/cynara-tests/common/cynara_test_env.cpp
tests/security-server-tests/security_server_clean_env.cpp

Change-Id: I246988d04c104e990af34f06b10c6b79e3e5793d

CKM: Security-server removed from dependencies

[Issue#] N/A
[Feature/Bug] N/A
[Problem] N/A
[Cause] Security server is not used.
[Solution] Removed.

[Verification] Successfull compilation

Change-Id: I398757dcba891feedd15d6ab51fa374e879975fa

CKM: tizen.org does not use security-server: disabling security-server tests.

Change-Id: I47c1ed4a6f0522b9c32954696521c19373841bbf

CKM: no more "empty password for lockscreen" logics.

Change-Id: Ie30acad07523218f5474d531cc7fcc579584181f

Remove unused members and functions from test runner

Conflicts:
src/framework/include/dpl/test/test_runner.h

Change-Id: Id7daab6582ced29db585322775cdfd54be708b0a

Merge branch 'cynara' into tizen

Conflicts:
src/framework/include/dpl/test/test_runner.h
src/framework/src/test_runner.cpp

Change-Id: I7186c1d1e8f60e453ea5ffe4f7cfd1e1eca3a3bb

Remove unused CollectTestsCasesList function

Change-Id: Ic846bfbf38e74850be7cdfee6ca5ccc88b9ccfb5

Remove unnecessary test result from test runner

Change-Id: I6a5257e0f4325f76219f798bc7e352fbef81376b

Create class for test result

Change-Id: I5da9105e4577805230929ca7c0057f456ef76090

Add PerformanceResult class

Test time measurement framework mechanism refactoring
for performance tests.

Change-Id: I08f3206f0dcd0504cb5bae6fa22f7480d5533a36

Extend list of tests done by security-tests-all.sh

Two more packages will be tested by executing security-tests-all.sh:
* libprivilege-control
* key-manager

Change-Id: I8fdefb6ff2c4a51060f2ac09179289b563596961

Add key-manager tests to test runner

Passing "ckm" argument to security-tests.sh script will result in
execution of "ckm-tests" binary. Previously it had to be called
explicitly.

Change-Id: Icacefde32d99832295ddaad045ae31be3d085ca8

Fix casting from and to void*

There were some places, where reinterpret_cast was used to cast
from or to void* type. This breaks C++ standard.
Fix replaces reinterpret_cast with static_cast.

Change-Id: I9bc5aa3307ce0b0fef4a988d14d06b01f290b853

Add additional libcynara-agent tests

Add test cases for: cynara_agent_initialize(), cynara_agent_finish(),
cynara_agent_get_request(), cynara_agent_put_response()

*) tcag08_get_request_and_respond_with_wrong_id - client shall not
   receive a response.
*) tcag09_get_request_and_correct_responded_id - client shall
   receive the response with correct id.
*) tcag10_cancel_request - client shall receive the confirmation of
   its cancel action.
*) tcag11_cancel_processed_request - the cancelation of the request
   processed by the agent is possible but void.
*) tcag12_create_two_requests - processing of two requests.
*) tcag13_create_many_requests - processing of many requests and
   sending the responses by the agent in LIFO order.
*) tcag14_client_disconnects - the cancel request shall be sent by
   Cynara when client disconnects.
*) tcag15_agent_disconnects - the access denied answer shall be
   sent by Cynara when agent disconnects.

Change-Id: I45c11e79c4977bed6892166a51c336e0414fc02e
Signed-off-by: Radoslaw Bartosiak <r.bartosiak@samsung.com>

Use proper RUNNER_ASSERT macros

Don't print errno message, when it's not valid
or related to assert condition.

Change-Id: I7012d5c97c1504df3d78fec0eac997d99560e374

Add inner tests for deferred macros

Added 11 test cases cover all possible nestings
of deferred macros.

Change-Id: I35898876f7b2dca740836463a5744fe96972f609

Add inner tests for timeout mechanism

Inner tests are needed to check if timeout mechanism works correctly
on platform that tests are runned.
Risk of not working is elevated because it uses std::future::wait_for()
method, which implementation had bugs in older compilers.

Change-Id: I3eae4dcab59b1fcec0c46f870653d0ae18c4a85f

Apply defer macros to callbacks in cynara async client

After all cynara library functions were sorrounded with try catch
macros causing all exceptions to be caught, there is no way of
throwning an exception from inside of callback function.

This is where defer macros mechanism can be applied.
Now all exceptions will be rethrown after comming back from
libcynara-client-async API functions.

Change-Id: I9743557631f2b13cecddf33cb072f280ae689d0a

Add defer macros

Used to defer throwing TestException exceptions (TestFailed,
TestIgnored) by catching them and rethrowing later.

This mechanism can help in breaking test and passing test result
from places where throwing exceptions is not allowed

Change-Id: Ic9baa96596d15fccbbff4ac81b18f91b2eb89c8e

Move non-template functions to timeout.cpp file

Leaving them in timeout.h causes double definition of functions.

Change-Id: Ic425855ffd958b2adbb16672d486fcdcedaa8897

Define a base TestException class

A new base class DPL::Test::TestException is defined as base for
 * DPL::Test::TestFailed (prev. DPL::Test::TestRunner::TestFailed),
 * DPL::Test::TestIgnored (prev. DPL::Test::TestRunner::Ignored).

All three classes were moved into separate files.

Common base class simplifies handling of exceptions that can be
thrown during tests.

Change-Id: I1fadb09b7781bf22a0090043a46ca48c55c9962b

Add basic libcynara-agent tests

Add basic tests for libcynara-admin library:
* tcag01_set_agent_type_policy_without_plugin - check if setting
 a policy with policy type provided by not loaded plugin fails;
* tcag02_set_agent_type_policy_with_plugin_loaded - checks if
 setting policy with policy type provided by loaded plugin succeeds;
* tcag03_check_with_no_agent - run cynara check that causes usage
 of plugin needing agent (but agent is not running) and verify
 that check returns DENY;
* tcag04_agent_initialize - check initialization and deinitialization
 og agent;
* tcag05_agent_request_timeout - run agent and register it in cynara
 service, break connection by resetting cynara after timeout;
* tcag06_check_with_unregistered_agent - run cynara check that causes
 usage of plugin needing agent (agent is running but not registered)
 and verify that chack returns DENY;
* tcag07_get_request - run simplest complete agent usage scenario:
    > agent registers in cynara,
    > a check using plugin and agent is done,
    > test acts as agent receiving and verifying request,
    > an ALLOW response is sent to cynara,
    > client receives proper ALLOW answer.

Change-Id: I781e9bb88b8686334acca4f9cf2c13a13bd643d5

Use functor object as main argument of callAndWait

Previous solution was based on std::function, that causes problems
because std::function template arguments needed to be explicitely
defined.

Functor simplifies usage of callAndWait.

Change-Id: If4f8a640848e41042e6841e49354510c08db5357

Add synchronization after file operations

After creating, copying and removing files program should
wait on disc synchronization, otherwise test can fail, e.g.
when we swap database files and start cynara, which loads
old not-yet-synchronized version of db.

Change-Id: Ie5c26da859e4e2a88311bb68ed7a0138b98e87d7

Load default database at every cynara-test startup

All tests assume that their are run on clear cynara database.
On real image database can contain some data. To save those data
save/restore mechanism existed. It was expanded with 3rd function
loading default database after original image database was saved.

Change-Id: If580a11b6558b8084c959366dcfa50a1ef92782c

Enhance compareDbs() with additional information

This patch adds printing more debugging information if an error occurs.
In case of database contents mismatch all the files in pattern and
result database are listed. In case of file contents mismatch filename
of corrupted file is printed.

Change-Id: I8e2bc2a6fdc976fac65e521accd6140ddfebef7e

Add checksum files for tests

Cynara will support database contents checksum validation shortly. This
patch adds proper checksum files to database patterns.

Change-Id: Ie1a61689765fe3f3e9fe010e7ba333ccbdae265f

CKM: certificates refreshed (some were already expired).

Change-Id: Iad2449aed5ce3a2e07cdf60e0b57dce56e1e8f10

Add --no-verbose option to XML collector

With --no-verbose param fail/ignore reason is not displayed

Change-Id: I62c78186c4b560f409f66ad74be3a3f4005fda42

Add --no-verbose option to HTML collector

With --no-verbose param fail/ignore reason is not displayed

Change-Id: I790d24f2db8b707df20f88e0df08ec20cf5eab03

Add SynchronizationPipe

A crippled abstraction of widely praised, but often
misused communication mechanism.

Change-Id: I2bd4b63dac76711ce22331c0bc666d4cd6021240

Add UDSHelpers (Helpers for Unix Domain Sockets)

Change-Id: Id4c0429c325bc828d7c928e1c12d2ba9ec2b6a9f

Add PasswdAccess

A wrapper on getpwnam() and getgrnam().

Change-Id: Ib833b6c1922ad45de3f713f29a904283d094600f

Add --no-verbose option to Console collector

With --no-verbose param fail/ignore reason is not displayed

Change-Id: Id2d0b93424f0ece32cad90d23b815146f4c6d111

Modify some test codes according to key-manager's 2.4 API changes

Change-Id: Ibfd46fd0f3c3a4c4b11488788eabbc2bae105d2d

Fix usertype in gum-utils invocation

Change-Id: I501a30549b0d7c5b94653250109db4aae9c1b5c7

CKM: CC mode uses user label to read & write keys.

Buxton does not allow root to modify user owned keys.

Change-Id: I106422a866fb6048ff359721dc30af68153661c8

CKM: access control tests use more descriptive error reporting

Change-Id: I02ce9162f4833b9a9175e88619a418c0045d0285

CKM: Split signture tests into two groups.

Change-Id: I3277c009ad05889ef587923731f8aba3177174bc

Move collectors' file help to collectors' commons

Change-Id: I89d03b06c97de61f06bad3c483c9cf7d4442c3dd

CKM: tests adjusted to tizenorg.

Install vconf key for testing key-manager in packaging install time.

vconf key cannot Install(set at first time)/unset in runtime.
It should be installed in package(rpm) install time.

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

CKM: Add new test to check if getCertificateChain return proper code.

Change-Id: I293e6132dbe179254575940e6c51acd261e0145e

CKM: Adopt tests to new PKCS12 api.

New api may take 2 passwords to decrypt data from pkcs12.

Change-Id: I6ea8e33005ce0b7b1db5cb69a265a68f2cc72398

CKM: Fix Control::restUserPassword test.

resetUserPassword will work if no key exists for the user.

Change-Id: Idc5080dc48e3e965c4af791f86572d2897ee7a43

CKM: Adopt tests to new api behaviour.

If Password is wrong api will return CKM_API_ERROR_AUTHENTICATION_ERROR
instead CKM_API_ERROR_SERVER_ERROR.

Change-Id: I9457414e44a1e95824371716a055bcd48855d370

CKM: Add test for getData with broken password.

Change-Id: Iad82b7c0523c5600a60cc5ed8d0c50a0e9a8f9a5

CKM: Add negative test for CKM::Control::unlock function.

Change-Id: I2ddc8572d033eb7b138fe6ad255669082fc31245

CKM: Add test for PKCS12 api.

Change-Id: I931405c6315053424425bec6ebfbe6e3de01d683

CKM: Add support for CKMC_ERROR_FILE_SYSTEM error.

Change-Id: I5288d565269767b3043ba0c9ae9a4584c85c665c

Add tests for deprecated access control API.

[Issue#] N/A
[Feature/Bug] bug discovered in the key-manager. This commit verifies bug resolution.
[Problem] deprecated access control API incorrectly mapped values into permissions.
[Solution] fix to key-manager plus appropriate tests in the security-tests.
[Verification] all T300_CKMC_ACCESS_CONTROL_C_API should pass.

Change-Id: I9bc52b0e493bb7dbde14b5793d18d5b924806648

CKM:Add compilation flag for MDPP

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Failing CC mode tests
[Cause] Compilation flag that disables MDPP logic has been introduced in
key-manager
[Solution] The same flag added to tests. When MDPP is disabled we expect
reading private keys to succeed.

[Verification] Compile and install key-manager and tests. Run:
ckm-tests --group=CKM_CC_MODE

Change-Id: I497c703108b6e2ee21599311beaa0d153c147d13

RUNNER_ASSERT wrappers moved to ckm-common.h

[Issue#] N/A
[Feature/Bug] N/A
[Problem] N/A
[Cause] Wrappers for RUNNER_ASSERT could be used by other tests.
[Solution] Wrappers moved to ckm-common.h

[Verification] Successfull compilation

Change-Id: Id9766c09d07c142814c3f3acd65a7b581ab5a061

Replace deprecated cert chain API with new one

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Old cert chain API is deprecated
[Cause] N/A
[Solution] Use new API

[Verification] Run tests

Change-Id: Ie1f6197af0afbebfc39e9661c010f176e718f7d3

Test for certificate chain API

[Issue#] N/A
[Feature/Bug] N/A
[Problem] N/A
[Cause] N/A
[Solution] Test for new certificate chain API added

[Verification] Run tests

Change-Id: I2b27db15d3424b8ed511509aadd77af829f9a055

Helper functions for certificate chains

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Excessive code
[Cause] N/A
[Solution] Helper functions used

[Verification] Compile and run tests

Change-Id: I361b6f117b360a15f514cc1fce4e7c3f9bb39d89

Certificate chain tests moved to separate file

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Cpp file too long
[Cause] All CAPI tests in single file
[Solution] Certificate chains part moved to separate file

[Verification] Compile & run tests

Change-Id: Iada8d00c0356aab97defdf3ef60c0f6ebb9088cf

CKM: Add error description in error message.

Change-Id: I9769865577579500bc774332dcc1fee12d828a00

Update cc-mode tests with "Ready" setting

[Issue#] N/A
[Feature/Bug] N/A
[Problem] "Ready" mode is not tested
[Cause] N/A
[Solution] Tests updated

[Verification] Run cc-mode tests

Change-Id: I16a86d9d10b5cef3ebbbd0880f981ef43b0ae6bc

Fixed dbus systemd interface

[Issue#] N/A
[Feature/Bug] N/A
[Problem] Services were not properly restarted. DBusAccess was waiting forever
for systemd job completion.
[Cause] Signal filter was set up too late
[Solution] Signal filter is set up before sending requests to systemd.

[Verification] Run CCMode tests.

Change-Id: I2e71a54ba44ddf0878a065e3b3bb8a7d117a6e3b

Add ocsp test for CAPI.

Change-Id: I3e2b8f862f3129b21795e352366422364ce893fa

Remove pointless code from capi tests.

Change-Id: I7d5f5a03517da471c6aad5c03746f5c8a66ffb31

Add tests for storing PKCS12 files.

Change-Id: I444b07febf2dc78a3ac2a952f1cd29141716ecb9

CAPI changes (set permission, remove alias): tests adjusted.

Change-Id: I1601606f9e4303d83b9840082c7093d9ac751b0a

CAPI tests adjusted to new password logic

[Issue#] N/A
[Feature/Bug] N/A
[Problem] N/A
[Cause] Password logic have changed
[Solution] Tests adjusted

[Verification] Run all CAPI tests

Change-Id: Ifb9a730b749eafa56d5107d13148935ef7353f60

Security-server and CKM integration tests.

Change-Id: I6ed1a68c2b7644c390d5ad05fe1512fc67e4dc80

Add test for using proper DKEK key to decrypt data.

Issue: another label is accessing data, while cached DKEK on data save
was already removed from the cache. This scenario was failing before
key-manager commit 4fe08778f9c31a698c6faace9b13fd6c5a4d9a7c.

Change-Id: I04c26c5ce5a86e83ab84ed78758984f7a0c2b5e4

CKM table separation: tests adjusted to new C++ API.

Change-Id: I74f742a75190fd66618cf0960836c3d7907059d4

Add tests for scenario trying to save/modify other label's data.

Change-Id: I3940c7c88cb1e8bbd4f11cc04eaef1755b72a246

Access Control layer: tests adjusted to C++ API changes.

Change-Id: Ia3b045ab7a935d37186dee37de730ddea4d0ec1b

Tests use new CC variable scheme (Disabled means device disabled, not CC-mode disabled).

Minor logging change in CAPI tests (error code was not logged).

Change-Id: I2bedc75da254ee87977453aac736861162890baa

CommonCriteria tests refactored and updated

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

Change-Id: Idb3e7a8d1e0df2913b3537d7cc4f109f12fffb0f

Start&stop support added to DBusAccess

[Verification] Run security-server-tests-password --output=text
to make sure existing tests are not broken

Change-Id: Ib5b1353c5892fb3685ace4f8f6226ab842c07ec8

Alias is not unique user-wide: modified test set.

Change-Id: I90577f4eaf02c24f5c7d390820c9de6817fc2142

Add service-side data verification test.

Normally, data is validated on the client side.
New tests verify if data is verified on the service side.

Change-Id: Ia6f8ab71b8d03c0dea3140f4e2fb4bc175a26f9d

Merge branch 'tizen' into cynara

Change-Id: I52bb94b85304e141e2462067d8c8dae80fe53081

Add inner security-tests tests

Inner tests might be useful in case of:
 * verification of advanced security-tests mechanisms;
 * testing if platform/image support features used in tests.

Change-Id: Ibd9b5d97af33742434d00fd3d682456c1a8d51ff

Move tests source code from "tests" to "src" directory

Change-Id: Icb325be4a37b9937290d7f3a4d35a1b3625b223a

Merge branch 'cynara' into tizen

Conflicts:
tests/common/dbus_access.cpp
tests/common/dbus_access.h
tests/cynara-tests/common/cynara_test_env.cpp
tests/framework/include/dpl/test/test_runner.h

Change-Id: I91f595f8a92ceafd071c4db0d70f7431447bab48

Rename file operation functions of cynara-tests

Change-Id: Ie24d7360dacf9f236ed36eaaaafbba1913d6e8dd

Refactor gdbbacktrace() function

Add safer, less redundant code and make it part of DPL namespace

Change-Id: I19e4704f4600d9720e939b0d76aa143c5d82ec2f

Add timeout mechanism for time-limited function calls

Timeout mechanism:
1) launches given function in new thread;
2) waits given time period;
3) cancels called function using custom user CancelFunction;
4) checks if function has finished in expected way;
5) returns function answer (value or exception).

Change-Id: Ia65d271095712e6afaaac96932f8d14d61b1702a

Add Agent class wrapping usage of libcynara-agent

With Agent object tests can play role of cynara agent and
communicate with cynara service. That can be useful for:
* testing libcynara-agent library;
* using agent in plugins testing;
* using agent in asynchronous client testing.

Change-Id: Idc783fa963074d5ff6f1dedb6a89b6aae6f65dbe

Add test-agent plugin

This is a test plugin, that sends all requests to agent.
Security tests can play role of this agent, so they can decide
about time and value of a response.

Change-Id: I278229810a84cdb4a3ff01ad8ef714019507f0c1

Add agent data wrapping functions

Add functions for easy joining and spliting data strings
used in communication between agent and plugins:
* wrapAgentData;
* unwrapAgentData.

This functions are defined in new file plugins.cpp - a common file
for both agent and plugins.

Change-Id: I29494928d752832aef7c8e14204a7e4ce63a911e

Add tests for cynara_admin_list_policies_descriptions

Each of added tests uses the fact that every cynara test is wrapped in
CynaraTestEnv that saves cynara service plugins.
Every test clears cynara service plugins folder and restarts cynara.
Then proper combination of plugins (sometimes none) are loaded.
There are 4 tests added:
* tc30_admin_list_descriptions_no_plugins - no plugins are loaded,
 there are only default cynara policies;
* tc31_admin_list_descriptions_1_plugin_single_policy;
* tc32_admin_list_descriptions_1_plugin_multiple_policy;
* tc33_admin_list_descriptions_multiple_plugins.

Change-Id: Id778477c35a6d65a7d4f71df03dcccefcbb6e756

Add wrap for cynara_admin_list_policies_description

Add method Admin::listDescriptions executing
cynara_admin_list_policies_description and checking if received result
match expected.

Change-Id: I1d729f83e2719360be5e4cdf405409561377c410

Add loadServicePlugins() for setting plugins env

Added function removes all cynara service plugins
and then import and load plugins from given directory.

Change-Id: I1fa559111c70c04e7522089a2bfed4ec9837f590

Enhance CynaraTestEnv with operations on service plugins

save() and restore() methods were enhanced with saving and restoring
cynara service plugins libraries.

Change-Id: If21707314d4a628db7c9d4034f1d4d7a9e82c55b

Introduce CynaraMask - a scoped cynara.service masker

Creation of CynaraMask object causes cynara.service to be masked and stopped.
Its removal causes cynara to be unmasked and run again.

Change-Id: Id7c20093fbd4ec19cdba783b20225ece04be5dfb

Separate file operations into separate file

All files operations done in cynara_test_env are moved to new namespace FileOperations
and implemented in cynara_test_file_operations.cpp file.
cynaraDbExists() function was generalized to dirExists(string) function.

Change-Id: Ic610078330074a648dbb24ddec7297a337a41de0

Add test cynara service plugins

There are two added plugins:
* single-policy plugin - declaring single policy type handling
* multiple-policy plugin - declaring multiple policy types handling

Plugins always return DENY policy type as result of check or update.
They are just test stubs for testing cynara_admin_list_policies_description.
Implementation of both plugins is embedded in BaseCynaraTestPlugin class.

Supported policies descriptions are put into plugins.h for later use in tests.

Change-Id: I38f4bcfe9624992eea3ef5a015885a397f5a0380

Remove description param from test result collecting function

Change-Id: I75f757abd6350456cbf572833b68a7a4aa8647e5

Fix uninitialized variable in Admin::listPolicies()

struct cynara_admin_policy **policies was uninitialized.
Normally it should be filled with cynara_admin_list_policies() call.
However, when that function failes, it leaves policies unaffected.
If this pointer is not NULL then it is being freed after it is used
to construct vector of policies.

Change-Id: Iff469205035a5240eee64ec87ea724b83577d105

Add tests for cynara_admin_erase

All tests try erasing policies in different scenarios and then verify
if proper policies were removed from proper buckets, but listing what
is left and comparing to expected result.

There are 7 tests added:
* tc23_admin_erase_empty_bucket - erase policies from empty bucket;
* tc24_admin_erase_no_bucket - try to erase from not existing bucket;
* tc25_admin_erase_single_bucket - check all posibilities of matching
 policies during erase;
* tc26_admin_erase_recursive_not_linked_buckets - erase with recursive
 flag set to TRUE, but in case where no policy lead to subbucket;
* tc27_admin_erase_recursive_linked_buckets - erase with recursive flag
 set to TRUE with existance of policy leading to subbucket;
* tc28_admin_erase_non_recursive_linked_buckets - erase with recursive
 flag set to FALSE with existance of policy leading to subbucket;
* tc29_admin_erase_recursive_from_sub_bucket - start erase test in
 subbucket and check if parent bucket stays unaffected by erase.

Change-Id: Ib712a028d97773e70b98de85c51aff0238bb8586

Add wrap for cynara_admin_list_erase in Admin class

Add method erasePolicies executing cynara_admin_erase
and checking if received result match expected.

Change-Id: I21f9dfa542fe931e56b4e09ec00712ab539d48ef

Add tests for cynara_admin_list_policies

There are 3 tests:
* tc20_admin_list_empty_bucket - test listing policies from empty bucket
* tc21_admin_list_no_bucket - test listing policies from non-existing bucket
* tc22_admin_list_bucket - test listing policies from fully filled bucket

Change-Id: I814ceacf367d63b73b746689ab28e115bf38adfa