Merge remote-tracking branch 'security-manager' into tizen

Change-Id: Icf50a2a27e4bd3f34deaf4fd654be41ffcca1c5e

Remove unused code

Change-Id: Ib47bd8f50679fbd68051fc9d589694f9f843431a

security-manager: adapt credential tests to nosmack

Change-Id: I8a849f95f7162de76f18aabfef7fb0f7358df693

framework: add timeout parameter to cmdline

Change-Id: I9a82736df504f94bc92489d4cbfe08dd38c7daea

Make deleting unexisting user return success in security_manager_08_user_double_add_double_remove

The removal of code from https://review.tizen.org/gerrit/c/platform/core/security/security-manager/+/324899
makes the removal of unexisting user account return success.

Change-Id: Ib31f4b399919d5e98b4819d355e49e29b8252af3

Add user & app installation fixture

This is to deduplicate AppInstallationFixture

Change-Id: I645a1194955d6579a7ca3d0aeef60b5548d5ad66

Extend owner getter tests

Make them test the API with pid != getpid() as well.

Change-Id: Id0e62e96798f0727af034b5b7661c54bd82d0340

Merge branch 'security-manager' into tizen

Change-Id: I84f2a067fca2dfa14a9ceb6ef6de310b118a3842

Add test for multi-app app-defined privilege provider

Change-Id: I13eb4a2feb6836ae173871a06b986eeb8ba86301

Merge branch 'security-manager' into tizen

Change-Id: I5d506f3e43047f2f7a71062277ba4bd984cc5a19

Fix failing test that went undetected in the previous commit

Change-Id: I217986c54d6a5bfdf65720914c5a0738165bce43

Remove unused function

Change-Id: Ib3ad4d33f82b850bf277f7d5181a5194da5ecc53

Adjust ckm-integration-tests to no-smack

All should pass

Change-Id: I10a10cf92170197ae538ce3eb970d6d9f069dd6a

Return "/Unknown" as a no-smack owner id

If key-manager is not able to identify the client app it stores the data
as an "/Unknown" owner. In case of unprivileged tests we're not able to
install an app or a user so we have to pretend to be an unknown app.

Make getOwnerIdFromSelf() return that string on no-smack.

This fixes all ckm-tests and e2ee-tests on no-smack.

Change-Id: I619eeb9473785cb3289e2eed9e6f28743e051a1d

Adjust initial values tests to no-smack

ckm-privileged-tests --group=T60_INITIAL_VALUES

Also, with this commit all privileged tests should pass

ckm-privileged-tests

Change-Id: I2f2efd2a14faddbf30ef24708bbca2693bfb8455

Adjust "permanent" tests to no-smack

ckm-privileged-tests --group=T151_CKM_STORAGE_PERMANENT_TESTS

Change-Id: I50c25c59257586708c575ff443e9b480b1b1b72e

Adjust async permission tests to no-smack

Also remove unused and duplicated code

ckm-privileged-tests --group=CKM_ASYNC_API

Change-Id: I72fd94887e027c0a76d565454eddbd11ed4e9560

Move alias info tests to separate group & file

ckm-privileged-tests --group=CKMC_ALIAS_INFO

Change-Id: Ic097b2844405fa13050123bb69af0cb00852bdc5

Force building with dwarf-4 to fix debugsource generation

This package doesn't use gbs defined GCC and CC flags.
In that case DWARF-5 is used by default and is not supported
by our rpmbuild. This creates errors when generating debug
packages and results in empty debugsource package.

Change-Id: I49a9cb9750058cae36df7b8ad6cb6958d4c007a3

Adjust access control tests to no-smack

ckm-privileged-tests --group=T300_CKMC_ACCESS_CONTROL_USER_C_API
ckm-privileged-tests --group=T310_CKMC_ACCESS_CONTROL_ROOT_C_API

Change-Id: I782d1c928ef6115427480fa799f5b6bc1fbccdc3

Log errors during data removal

Change-Id: I6ed087ac1794ab3249c0915047187a0f832e85a9

Use effective ids in ScopedAppContext

Change-Id: I3742af42218e924e0a99ddcf64595d227b9d55b4

Adjust system db tests no-smack

Verify with: ckm-privileged-tests --group=T50_SYSTEM_DB

All should pass on both smack and no-smack

Change-Id: I38e11de3e3fd6a2688f0bc4fb6fe34af843869a6

Replace char* with std::string in utilities

Change-Id: I7920d1d2925e6a0cbe47f51cfe89a94735cd39ec

security-manager: Use ScopedAppLauncher for app processes

Change-Id: I7be4f77ac5e774dcbdce445c24d4b2dc38efa5e1

Merge branch 'tizen' into security-manager

Change-Id: I0f839b3a06140566a21cfd6dd1bcb4cbd6907aaf

commons: remove fork() from AppInstallHelper

Change-Id: If175cee6a77cc2ad9ecd9a5aa41ffe099e36ac1a

common: Get system user from GumdRunner

* rename TempTestUser to TestUser
* add getSystemUser to TestUser
* fetch owner user using TestUser instead of
  hardcoding owner uid in test cases.

Change-Id: I6246f6b80e657d64bab64c4073eb774dbb700c49

Add ScopedRemoveData

Change-Id: I83ca4ea4c72826ebcf53d2f03df94c0e0ce1d2fa

dpl: Define smartptr structure based on type name

Smartptr define creates deleter struct with name
based on delete function name. Defining name based on type
name makes it more flexible.

Change-Id: If4bcb86e42ac2b0a81507152f9697f2fcce0d939

Skip scoped resource cleanup in child process

Change-Id: I29e9ec760d9e73d4d0ebb593705c3c62c9e54fe9

Cleanup dependencies

Don't link with libraries already linked by common library.
Make all common include dirs PUBLIC so test binaries know where to look
for headers.

Change-Id: Iab79f69fc6627b002f191d47b4b89bb47d60ea35

Move ScopedAppLauncher to common library

Change-Id: I604b7f85fc2f2ed8b7fcd22f711e93aaf5950fc8

Revert "Add handling uncaught exceptions from the child process function"

This reverts commit a8ae172c7481236669783f870698640fc9409b4d.

Without it, we won't be able to get the failed assertion info in
RUNNER_CHILD_TESTs.

Change-Id: Ib38eee248e950e30e8eae88daf03e0289152d4a4

Merge remote-tracking branch 'origin/ckm' into tizen

Change-Id: If027158514113f53d7173485a8995de195a8a488

Merge remote-tracking branch 'origin/tizen' into ckm

Change-Id: Id8d064063e7c4b065cbe7e3c2801168c2d6cacd5

Merge remote-tracking branch 'origin/tizen' into security-manager

Change-Id: I201ee92e8af3724d2df3ff2d4290f8310b68fcee

Catch exceptions from group init/finish

Change-Id: I1af60066ff1d7c2e0b72a6a765b88f19bd09d535

security-manager: remove manual forks from ScopedAppLauncher

Change-Id: If6b88d39a2b6cea4aff073876f54b47bc21efa93

Add handling uncaught exceptions from the child process function

Change-Id: I052f3980ccd590b870226e180710edaf350b77e3

Fix checking mmap() return value

Change-Id: I2d3355f28caa2e2cdddcae7cecb9c077329d3e42

security-manager: Remove manual calls of fork()

Change-Id: Id39eb0a124175c178d9ad62a362856518bcde0b0

security-manager: Fix segfaults in PolicyRequest

Change-Id: I1273c17991d4c32d5585740b2386d5e04b4e086f

security-manager: Ensure app cleanup in ScopedAppLauncher

Make sure that cleanupApp is called in the ScopedAppLauncher
destructor even if everything else fails

Change-Id: Ife73ededce3b42e3e4a19534a169e78ce029969c

Cleanup tests-common dependency

Change-Id: I03b0aaa1b184524dc3fc702e5391ca055bf78420

Add tests for security_manager_get_app_full_credentials_from_pid

Also fix security_manager_get_app_owner_uid

Change-Id: I2d55c7240fc8eeed73018a8b173df0e2ea555499

Merge branch 'tizen' into 'ckm'

Change-Id: Ib249edc0d7789b5427b5950d9292b41fa096b70f

Fix gdb attaching to the wrong process

In the wild, I experienced a case where gumd process terminated with
failed assertion due to the main process exiting and spawned a child
process to exec gdb and get the backtrace. However, just after spawning
the child, the process gets killed because runner's main process exits.
This caused the spawned child to be reparented to PID 1. Then the child
obtained the pid to debug via getppid(), which returned 1, causing
execution of gdb --pid 1.

This could be harmless, but if this was done
under strace -ff it resulted in deadlock in some GDB subprocess
effectively freezing systemd (PID 1). Consequences included indefinite
freeze in ssh clients trying to connect to the emulator - it was quite
problematic impediment in work.

Now the pid obtained using getpid() before forking. Moreover, getting
killed by SIGKILL in case the parent process died was implemented for
the GDB's main process.

Change-Id: I19251d266d6c4bbd7875b9c4ae56f97f4d94e180

security-manager: Fix private sharing

Adapt to uid-sandboxing

Change-Id: I7f95b8f56161a1b99e7cabb8edd561580767d4b4

security-manager: Change path handling in AppInstallHelper

* properly remove created files
* keep paths in unordered set to eliminate duplicates
* fix tests that drop privileges in the same process
  as AppInstallHelper will be removing its paths

Change-Id: Ie737ef88058c63c3e1ecc868bd4f88b8eeb6797a

Add internet access control tests

Change-Id: Ic5d2132e20afa35cbb1b73775d16b1033d0f9660

Update no-smack list for set_identity

Change-Id: I808bbc65d280e4068db212797ecdbfc60d6215af

Update path setup tests

* Add dependency to libacl.
* Modify access checks to verify ownership and ACLs in no-smack mode.
* Add AGID getter.
* Add ACL helper.
* Rename test app dir to work with AppInstallHelper.
* Make runAccessTest() prepare and cleanup the app.
* Add system_access supplementary group for tests in no-smack mode.
* Update the tests.
* Skip access checks expecting rule removal in no-smack (paths will be
  removed by the installer anyway).
* Make AppInstallHelper use global user id when installing as root (the
  app is installed as a global user in such case).
* Fix AppInstallHelper::createFile().
* Make AppInstallHelper::create*() methods chown files for global
  installations too.
* Update no-smack tests script.
* Minor refactoring.

Change-Id: I7c6b302767ef1122439ea79b3eb2bb4785316120

Add app owner getter test

Change-Id: Ida7103c7c48f34da5ff6d32e5cb0d1c290456a18

Remove DPL::Atomic

Change-Id: I9c7a3afc4cd9ff1cd314b91f5309e0faaea24357

Add security_manager_09g_update_many_apps_in_single_request_duplicated_ids

Change-Id: Idaf017b1e1950ada7cf81b9c9a2dc31448b02f5d

Fix security_manager_09b_install_many_apps_in_single_request_duplicated_ids

Change-Id: I8230c35fcd8d2093963311e9ccc4188d6c74f722

Add run-security-manager-no-smack-tests.sh script facilitating changes verification

Change-Id: I5e340c3ab5d8647d73fd1dae2e890af62a81ccab

Fix security-manager-tests sometimes hanging

Even security-manager-tests --list sometimes hung.

There were 2 problems:
1. Reading from socket was unchecked for read() returing 0 (indicating
   that the socket was closed on the other end) and this resulted in an
   infinite loop calling read() that always returned 0.
2. The socked was closed because it lived in the parent process that
   already died. Even though the child process requests getting SIGKILL
   on parent death with prctl(PR_SET_PDEATHSIG) it was possible that
   parent died before prctl() succeeded causing the program to misbehave
   in the ways descibed above.

Change-Id: Ief50e9addf4ead899c29f5f28faa0dfd95ab3c84

Fix %postun being called on the end of package upgrade/reinstall making tests fail

Change-Id: Id3796fbfbdd43f1e5a1dd46b42ba9546f3ef68cb

Reduce timeout in security_manager_400_prepare_app_series_with_concurrent_install_stress

Previous value could lead to timeout in the testing framework.

Change-Id: I93ededea7574d125c064032ccea8b2fc49b2fdf0

Use RUNNER_CHILD_TEST where necessary

Tests with fork() and child processes throwing exceptions must be added
as RUNNER_CHILD_TESTs so the exceptions are properly handled.

Change-Id: Ic9b841b38e338f47b52d5fcb036ee6a1d43e3523

Migrate AppInstallHelper::getPUID() to use security_manager_set_identity()

Change-Id: I50bfcd0dd1dce336e3b67e18dddd4daf1a45efba

Fix typo

Change-Id: I75cdbd481ae0e2241f73679c0124c580a9c63969

Add missing app cleanup

Change-Id: I413c6998aff7aa06cd3b0752178c13ec42a1cd3b

Adapt tests for security_manager_set_identity

Change-Id: Iff96cc48675e64382c70b30a9d9324c9c7e68c3e

Make security_manager_26_1_security_manager_get_app_owner_uid test work in no-smack mode

Change-Id: Id80490c119b14ffb1937c07fbaa2b6b0eeb85af4

Fix security_manager_08_user_double_add_double_remove test removing uid 0

Change-Id: I4ed00a1a6fe95572d037182f4ebbc80d79b693d2

Adjust some of security-manager tests for no-smack environment

Change-Id: If031bb68e36ee8ad2df0eb00e32637fc78d11f01

Fix tests sometimes running multiple times due to some failing test

Change-Id: I1d03d20fd633fd0c52920de6ef20a5424dcbd203

Remove unused functions

Change-Id: Ia92180da601a967b12c4a3fe0856e4545c844d1e

Disable smack access checks in no-smack

Change-Id: I02505a9584a5cdd34bb2b51938dcfbb9c986e996

Adjust app preparation tests to no-smack mode

Change-Id: I08a98000e404d2d5f8d95fe507fe53f901c235bf

Retain CAP_SETUID in no-smack mode

Change-Id: Ie6016234421ebb1594d12d550a8d175d52d8244b

Add a sample testing app for No-Smack Tizen images

Change-Id: I7d491816ac36cb3cb5855eba4f71c3725f30ac75

Enable DSA 1024 tests

Change-Id: I64bb4ac51002aa1b0286018839a8adebac0ea5dd

Add test for security_manager_is_app_from_pid()

Change-Id: I069cda129f9df3fc52a4bf123692ab1a9fe75a5c

Replace expired certificate

Change-Id: Id4618d45246af4ce9a34da280408126c0d3b7fcf

Add tests for new APIs

* security_manager_get_app_owner_uid
* security_manager_self_is_app

Change-Id: I34bd9a719417cdc1b05554bbaff0886a6b9322ec

Add tests for security_manager_set_identity()

Change-Id: I4a125d42d5e0c9cd579472df0bba54053e5489a2

Merge branch 'tizen' into security-manager

Change-Id: I3a18e67ff22ffd6f9540a45debdcb4b921c8c804

Merge branch 'ckm' into tizen

Change-Id: Iaafaf37a699c40c8e27ebdaff27a587d96ccfa74

Add -Wno-alloc-size-larger-than

Change-Id: I57da775857fe21e5be2a6ce0e323859776fdf510

Remove usleep from app preparation malloc tests

Change-Id: I44ec5aacaaad3f8b6b4eca3fc0f74997f6d66abf

Merge branch 'tizen' into security-manager

Change-Id: I2182b5350daefac56746a1d8ee56f9e871608c94

Merge branch 'ckm' into tizen

Change-Id: Icd1366f901a908c828d4185a7367ebef4c2eef75

Add test for allocations during app preparation

Change-Id: I7540450868479a15d5be0448c8b7155b09746b7a

Fix x86_64 debug build issue

Change-Id: I940ce244e53935156a5bf5d745dccb2be4e0661f

Fix 64bit build

Change-Id: I1c308248820756da20fc572f8caf653a5f4c17b9

Fix building error after build flags update

Change-Id: I725586654155c0c64ad2e8f0c574a270805ae955

Replace expired ca.crt certificate in PKCS chain

Change-Id: I18ef509e06ea70f18a81e96290f1396f2be1b6be

Add tests for same alias case

Change-Id: Iac7cbddcaa89e94d2784de2584ccc97ea2fb77b3

Added tests for Update API.

Change-Id: I2948736744093c8d9d47e3af69502f6b4473d9a6

Add tests of concurrent prepare_app and app_install/uninstall

These tests should properly stress the two-threads implementation
of the daemon that has dedicated thread just for prepare_app call.

The test takes more time so its run as RUNNER_TEST (not CHILD test)
as CHILD tests do  have a timeout in the testing framework internals.

Change-Id: Iad094acfc3d86d9b1d15c79a6b9095b733adda93

Enable ECDSA import tests

Change-Id: Ic5d2c33d10198fec470b0342d4e6943f0ef2b7c5

Add DSA import-sign-verify test

Change-Id: Ibe05f1602bf909ff607c588d646331ddbd440de4

Remove outdated DSA verification comments

DSA verification was fixed in tef-simulator
so the comments are no longer true.

Change-Id: I8a1fbaea88652cb05e846183d5ac0efc74883b80

Test getting/saving KEM keys in TZ

Change-Id: I99f95069a78eec69451656371dd53613f5691e6e

Add EC & PQC hybrid derivation test

Change-Id: I6d5c13d024dedb166f9cac0eac1dfc198d550fe7

Add testing KEM vectors

Change-Id: I443a99dcb4cb7b4b288084db50e6b167715cd860