service: Set MemoryLimit to 20MB

Change-Id: I2d7387f6d273e2d4628ca528c389e3840a695ffc
Signed-off-by: Unsung Lee <unsung.lee@samsung.com>

Merge branch 'tizen' into tizen_7.0

Change-Id: I9dbac55e67b7b732c1ad436c0a0b859cc1b40aa9

libsessiond: fix unchecked return value in `subsession_get_user_list`

`getpwuid_r`s semantics is a bit tricky: normally it returns 0 and sets
the output parameter (struct passwd **result) accordingly. If no
matching password record was found, it returns 0 and `*result` is set
to NULL. In case of error, an error number is returned, but the
`*result` is also set to NULL. Since for the API client the reason does
not matter, not checking the return value from the `getpwuid_r` syscall
is not an issue (we're returning `SUBSESSION_ERROR_IO_ERROR` in both
cases). However, to satisfy static analysers we have, it's not
necessarily a bad idea to perform an explicit return value check.

Change-Id: Iefaeccd09331cff977b7cc2a449367de7d5c7db1
(cherry picked from commit f111fc670eee90e54ce2e395bb655c5935079d29)

subsession-client-example: Move the call to bus unwatch to the callback

Change-Id: I8e54fa4aff6fb92ff0fe8e8f00f2c663cf73a93c
(cherry picked from commit da996b00efab7c63e94bbcf811c7d890f0d2c3d7)

Merge "libsessiond: Add `sessiond_free_user_list()` API function" into tizen_7.0

libsessiond: Add `sessiond_free_user_list()` API function

Even though it was explicitly stated that one needs to call free()
function after `subsession_get_user_list` to deallocate the user list
array, the addition of the above routine makes the libsessiond
interface more natural, since the API consumer does not need to
reach out to a lower layer (glibc) function.

Change-Id: I1897c05adf4cd1b759ef0bc7ecf25ef966ac5327
(cherry picked from commit 7c9c65b3344cae9177830c83b6ace8f70404f070)

libsession: update doxygen API documentation

Provided doxgen API documentation wasn't complete and accurate:
not all possible error codes were listed as well as the description for
the `unregister_event_callback` API call was inaccurate.

Change-Id: Ib3da4e8f1ad6f50f9b14ac84f25b8d3b6b07c543
(cherry picked from commit 34911f815589051d1006480bd670b8dbfb5a933f)

libsessiond: Add `sessiond_free_user_list()` API function

Even though it was explicitly stated that one needs to call free()
function after `subsession_get_user_list` to deallocate the user list
array, the addition of the above routine makes the libsessiond
interface more natural, since the API consumer does not need to
reach out to a lower layer (glibc) function.

Change-Id: I1897c05adf4cd1b759ef0bc7ecf25ef966ac5327

libsession: update doxygen API documentation

Provided doxgen API documentation wasn't complete and accurate:
not all possible error codes were listed as well as the description for
the `unregister_event_callback` API call was inaccurate.

Change-Id: Ib3da4e8f1ad6f50f9b14ac84f25b8d3b6b07c543

Fix subsession switch user completion tests

Additionally, added more checks and logs.

Change-Id: I999ab1cfba81e73fdb0814a20a5f7ba9821aeffb
(cherry picked from commit 566abde05f647ae3070dca2d4ef8411ea1a7d58b)

Fix subsession switch user completion tests

Additionally, added more checks and logs.

Change-Id: I999ab1cfba81e73fdb0814a20a5f7ba9821aeffb

[fix] sessiond: OnSwitchUser: `prev_subsession_id` being overwritten

Due to incorrect use of `std::string_view`, the value of
`prev_subsession_id` was overwritten by `next_subsession_id`.

Change-Id: Id198573cfae9cdc4f7e3edcc87f006801dfb8395
(cherry picked from commit 5cfe2d3bd559b6779383a56f2a48c33687ab1fb4)

[fix] sessiond: OnSwitchUser: `prev_subsession_id` being overwritten

Due to incorrect use of `std::string_view`, the value of
`prev_subsession_id` was overwritten by `next_subsession_id`.

Change-Id: Id198573cfae9cdc4f7e3edcc87f006801dfb8395

libsessiond: fix a memory leak in the `query_dir_via_readdir` routine

`subsession_get_user_list` has non-obvious semantics. When it is called
with the `user_list` parameter set to NULL, the caller is only
interested in the number of subsessions associated with the
`session_uid` user. In this case it is unnecessary and undesirable to
allocate memory for the user list, as it leads to a memory leak, since
we are not passing the pointer to the caller.

Change-Id: Iac30507f6e3ac826ac66254b9404edbf17d95230
(cherry picked from commit 67f5b16ce38ae6188d7c989b35b95f3ab4dc2b22)

sessiond: make subsession remove user operation atomic

Change-Id: I04d6d76062edf4e7de955a2f34cb40ee59ffc123
(cherry picked from commit c0b228d61f38db7ef66588fa5f2fa818fa54e976)

sessiond: make subsession remove user operation atomic

Change-Id: I04d6d76062edf4e7de955a2f34cb40ee59ffc123

Release 0.7.7.2

Hotfix a memory leak in the library.

Change-Id: I268d83c1f22fbb0c1b3d32000189a85788c8886d

Merge "libsessiond: fix a memory leak in the `query_dir_via_readdir` routine" into tizen

libsessiond: fix a memory leak in the `query_dir_via_readdir` routine

`subsession_get_user_list` has non-obvious semantics. When it is called
with the `user_list` parameter set to NULL, the caller is only
interested in the number of subsessions associated with the
`session_uid` user. In this case it is unnecessary and undesirable to
allocate memory for the user list, as it leads to a memory leak, since
we are not passing the pointer to the caller.

Change-Id: Iac30507f6e3ac826ac66254b9404edbf17d95230

libsessiond: fix unchecked return value in `subsession_get_user_list`

`getpwuid_r`s semantics is a bit tricky: normally it returns 0 and sets
the output parameter (struct passwd **result) accordingly. If no
matching password record was found, it returns 0 and `*result` is set
to NULL. In case of error, an error number is returned, but the
`*result` is also set to NULL. Since for the API client the reason does
not matter, not checking the return value from the `getpwuid_r` syscall
is not an issue (we're returning `SUBSESSION_ERROR_IO_ERROR` in both
cases). However, to satisfy static analysers we have, it's not
necessarily a bad idea to perform an explicit return value check.

Change-Id: Iefaeccd09331cff977b7cc2a449367de7d5c7db1

subsession-client-example: Move the call to bus unwatch to the callback

Change-Id: I8e54fa4aff6fb92ff0fe8e8f00f2c663cf73a93c

Merge branch 'tizen' into tizen_7.0

Change-Id: I4b9c669418b4989cade6223aaf65ee248c1bbf05

Fixup: add missing sessiond-platform.h

Change-Id: I1d5117d6745d2ea14d8911703e2a782bb79f661c

Merge branch 'tizen' into tizen_7.0

Change-Id: Icf6ecbf6574b4f884ac7a5ae92d1ba6f0622a4ac

Release 0.7.7

Change-Id: I62e20d21f07227994e0dd8e8474fe4cb803e3fbf

Add the new callback function to catch `sessiond` ready state

Change-Id: I1d4357cfc5a3fadd6b66d1b2a8d5717a1520adc0

Merge branch 'tizen' into tizen_7.0

Change-Id: Ibcfd2e453e64e480f77677f9e1c1f797c1c395f6

Merge "Release 0.7.6" into tizen

Merge "Apply on-boot daemon activation" into tizen

Release 0.7.6

Change-Id: I478dee5fb9cd23e530151cfe205d3a139304f655

Merge "Reimplement `subsession_get_user_list` by performing subsession directory interrogation directly whenever possible" into tizen

Apply on-boot daemon activation

Change-Id: I754eaf42b55bbb4ee8d51f99a06694989fad7d8b

Reimplement `subsession_get_user_list` by performing subsession
directory interrogation directly whenever possible

Co-author: Michal Bloch <m.bloch@samsung.com>

Change-Id: I6b02fcf590f75fdb8f60db0a328d3ff5409f10cb

[fix] Ensure proper permissions are set for subsession app directories

First-level `apps_rw` subdirectories' permissions should be directly
copied from the corresponding `/etc/skel/apps_rw/*` subdirectories.

Change-Id: I2fe12c5350feece8c1264b2f4c92ef7509c81845

Release 0.7.5

Change-Id: I2314f936c4fa256eec9226f2c5afbc36c3ab0d4d

Tailor the integration tests to account for changes in granting
permissions to subsession app subdirectories

Change-Id: I567b0c211573170de6839fbb044d0eadc3ec2995

Ensure proper permissions and owner/group are set for copied
directories

Change-Id: I6d544e9acb70cc64dd368f954b666a11578c6709

Fix soname

Change-Id: I99c79253b6cde0ed62effe7680d3fec49740b6bb
(cherry picked from commit 0d432d8236a19f8b2e4efef47837b27deff886d7)

Fix soname

Change-Id: I99c79253b6cde0ed62effe7680d3fec49740b6bb

Release 0.7.4

Change-Id: I298f2b47ad2d5ee357eb61bb1f6e4c0542bb3211

Merge "Improve sessiond integration tests" into tizen

Merge "Document transmutation inconsistency" into tizen

Improve sessiond integration tests

This commit:
* fixes permision check error that incorrectly assumed `owner` as
  the only user that can create a subsession
* adds more verbose messages when comparing files, so that it's
  much easier to identify the file reponsible for the error
* adds proper subsessions clean-up when something goes wrong in
  the middle of the test
* removes an unused global variable.

Change-Id: I5fbdd965e367442d9bce85e065d7e1bb0c475341

Fix symlink permission handling

Change-Id: Ibb7e3af28c576dfb262efea2d734551132e5d3a4
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Work-around a sub session creation issue

Seems to be a libsmack issue at a glance.

Change-Id: I10dfc7e3cbc83450d730eaf4d008ac33e9b50a4b
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Document transmutation inconsistency

Change-Id: Ida2f349ce97ca4aee7e1504f73c2fe358294b622
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Add unit tests to SAM exclude list

Change-Id: I0f417c89ffa0af2b5dfa882a68c75d6ad3c1cb48

library: fix a memory leak in the `method_call_sync` routine

Change-Id: I2d4ae28c363996faf859cc33635bc6c777ef2442

Release 0.7.3

Change-Id: I2eb57e6c64f0307542a1fe0271ca83ed1c4f1031

Add comparing file/directory permissions in integration tests

Change-Id: I27f1e680b6dda5234300795aa413904e124a18b6

sessiond: Ensure proper permissions are set for subsession user app dirs
as well as the main `apps_rw` directory

Change-Id: Iab41dcace38d1dcc346bf9bfde12ede1503ce359

Drop -fconcepts from C flags as these are supported for C++ only

Change-Id: Ie1956bd1c890c6fb7402e74bd07dff7770296ba0

library: Add coment about unsupported parameters to subsession_event_wait_done() function

Change-Id: Ia0aa98e8c02c89a405a649dabc40cb6fa5280e73

Modify sessiond integration tests to conform to the changes in setting
ownership of user app subdirectories

Change-Id: Ic7b33dc1783421288007f6d57a30173aff6b86d1

sessiond: Add privacy to the `fs_helpers` module

Change-Id: Iebce977b981135a85bd22d61cddd016d39376aa4

sessiond: Ensure proper ownership is set for subsession user app dirs

Previously, all [SUBSESSION USER/apps_rw/*] subdirectories had their
UID:GID hardcoded as [subsession_uid]:`users`. Since some of the Tizen
app subdirectories have the group set to `system_share`, the correct
way is to copy the ownership from the corresponding
`/home/SUBSESSION USER/apps_rw/[some.app]/*` subdirectories.

Change-Id: Ie24d55179b8c2216dc57dd35b27a14a449c84642

sessiond: Drop fs_helpers namespace

This is small project, probably code could be moved to sessiond namespace evetually.

Currently no other file uses sparate namespace (wait manager, hash, globals)
so dropping it from fs_helpers should make code a bit more consistent.

Change-Id: I5d5786f55e844abaf13e1d73358d0d980611f0fc

sessiond: rename the header file from `fs_helpers.h` to
`fs_helpers.hpp` for consistency with naming convention
used throughout the project.

Change-Id: If2e6a490b01788a81894f517b0300a26c41a2918

Release 0.7.2

Change-Id: Iddc79db71513e293b26a0473404cbf0a6468135b

Move all sources below src/

Change-Id: I10a75ffe378790a7f7605c3b6aea2818ef7f2e29

Move all tests under single directory hierarchy

This commit:
 - moves api tests to tests/api_tests,
 - moves integration tests under tests/integration and adds it into tests package
 - moves client example application under test/client_example
 - drops no-op sessiond tests (test template)
 - drops unused infrastructure for runnnig tests at build time

Change-Id: I6047f66c358b089710c30709454f7afa891bd10b

Add sessiond test suite

Change-Id: I503e27b76dec485b132f6f9f85bf9e7f673d1121

Merge "Make sessiond more fail-resistant" into tizen

[fix] sessiond: Correct handling of the TRANSMUTE attribute

Due to its different nature (being inheritable) it wasn't handled
properly in some cases (e.g., for subdirectories that had it set
to false whose parent was set to true).

Also significantly simplified the logic by adding a helper function
and refactoring the `copy_smack_attributes` routine.

Change-Id: I3ee146056682e2380932fdd9052efe25e5d7f735
Co-author: Michal Bloch <m.bloch@samsung.com>

Make sessiond more fail-resistant

Change-Id: I5e7c95564960523d5d6ffea4d2d24ed94a82d0dd

Change subsession API argument name (from cb_data to user_data)

Change-Id: I58969118f4a0412ef781d1c277ca222e70b02723
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

Merge "library: Simplify subsession_event_wait_done() error path" into tizen

[fix] sessiond: do not follow symlinks while setting SMACK attributes
of the copied files

Change-Id: Ia775dc9af277a333dbd5e75ee8e4ba2eebe88098

Release 0.7.0

Change-Id: Ie35da62d64e3f985060fa0a03641864dd942417c

Merge "Log using DLog and not on stdout in sessiond" into tizen

[fix] sessiond: do not follow symlinks while setting the owner/group
of the copied files

Change-Id: I64aec17972aa916d471b16684a00a073c30a2016

Log using DLog and not on stdout in sessiond

Change-Id: Id9db6b275da9964c221bcf54d36fe717deecd29f

Revert "Temporarily allow all dbus policy by default"

This reverts commit 82d39e1f34062fa443103bc1f2980bd54e295cf2.

Change-Id: I85c6780dcbe1ec4a4fd3a49e48f4d69870bb7ddd

library: Simplify subsession_event_wait_done() error path

Change-Id: Ic701196aa0501a5adb3a1c8742609f82ded69909

library: Automatically free GError* in method_call_sync()

Change-Id: Iddb19ec2556106e2ac1f5052dd4512dbff43194d

library: Use same terms in error mapping functions (in both directions)

Change-Id: I7f6e2a12a42848feabb49a34fd64633a747a7265

library: Drop dead code in subsession_get_current_user()

We don't need to explicitly check type of returned gvariant
as this will be handled by g_dbus_method_call_sync() (as invoked
via method_call_sync()).

Change-Id: I5327b670ec002f3be00e8fe40e8ba20314ccf3fe

library: Drop unused macros

Change-Id: I13314d9bd04f88ef89e7ed159456897a7d38378b

Release 0.6.0

Change-Id: I52110e67708b97fa35c443d736d0edcecc27b9d5

library: Rework comparators

- use assert() if our internally added data is of correct type (if it's not, it's an
  programming error)
- use explicit typechecked_g_variant_get()

Change-Id: Iad18df2f323b0d9f71edec52dbf46cebe9752a2d

library: Be more verbose in error messages when invalid signals are received

Change-Id: Ibdf14033919604569182d587f3bdb99c73725ff0

library: Use generic method_call() in get_current_user() API

Change-Id: I6973fc7f414edb65251c8c6f90cb1d06f1a98525

library: Add macro to validate and get gvariant without explicit 'return'

This allows more flexible handling of type errors, without creating
new macro for each new case.

Change-Id: I6ea3311ed24da9021e6a2b65eab31cea894ae0c0

library: Replace dynamic error checking with asserts in unregister()

It's programming error to pass NULL callbacks_data - catch it early
with assert.

Change-Id: Id6ed5861c2a5ce38e0e148e9cf177170a1661d4f

library: fix: ensure dbus connection is avaliable in unregister_event_callback

Normally this function is called only after register function is called,
which checks if dbus is available.  As it's public api we must not make
assumption this behaviour is followed in real code.

Change-Id: I41b95bb746d954b6af66e31a0c9735d7f21e88b2

Unduplicate code

Change-Id: Ic52222eb6157a5507689a3a91f6103d13de5ca02
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Create `apps_rw/*` subdirectories and files with [subsession_uid]:users UID:GID

Change-Id: I53045101838b12b983e3cb46055614dccf3ccc6f

Release 0.5.0

Bump version due to behaviour change - from now on subsession
user and it's apps_rw directory is created with <session_uid>:system_share
UID:GID.

Change-Id: Ic26a8050769da3a3e48ac574171b528bfce195ca

Merge "Create subsession and `apps_rw` directories with [subsession_uid]:system_share UID:GID" into tizen

Create subsession and `apps_rw` directories with
[subsession_uid]:system_share UID:GID

Change-Id: Ib1f8d18d715ba6ff52c97cfe78c56d00ae2e9d37

library: Rename async method call to follow method_call_<sync/async> convention

Additional style fixes along the way.

Change-Id: I6eb43013019a0a701cc191ff25e1dd0463526124

library: Ensure map_dbus_call_error_to_return_value handles NULL

Allowing the funcion to accept NULL GError* will allow to simplify
code in other places.

Change-Id: I13e172907651da119d13119a8d07c4c7906489ee

library: Move all of dbus signal subscription logic to register_event_callback

Change-Id: I2fa04d1095569e709adbbce6e297a68f6668fb94

library: Merge dbus sync call helpers

This commit merges method_call_no_signal() and method_call_no_signal_return()
functions into one, as the only difference between these was that returned
variant was passed to caller or not._

Change-Id: Idedac845a6b9690cade29f54dc9fbc372e7a8642

Release 0.4.4

Change-Id: Ic647ec27f5f86cad2e066aeef21dc81d2468c1aa

library: Rework client_data_cleansweep_mt

This commit:
 - renames function to callbacks_data_remove_client_data as it operates on callbacks_data list
 - changes order of function paramters to follow convention used by callbacks_data_* functions
 - adds asserts to ensure parameters are always valid

Change-Id: I0671a6a6bea783224a0bbe966bea08b4c0ec5d03

Merge "sessiond: Re-shuffle include files to get rid of a nasty "unused variable" warning" into tizen

library: simplify logic in callback_data functions

Change-Id: I6f707a201780012a3c138fca85f7a0d0b9f9a26c

library: make callback functions follow subject-action convention

This commit renames the following functions:
 - take_from_callbacks_data_and_remove_from_list -> callbacks_data_pop
 - find_in_callbacks_data                        -> callbacks_data_find
 - subsession_register_event_in_list             -> callbacks_data_push

Change-Id: Iae4ec775a7626e45667452ff52dd2b19dc2dd6c8

Merge "clientExample: proper clean-up in case of subsession errors" into tizen