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

clientExample: proper clean-up in case of subsession errors

Change-Id: I57682b4e7b4649af55435c70186a0a97956e10bf

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

Change-Id: If0e3530c45e236cee1e1fe11a155a07099a0c498

library: Shorten client callback data allocation function name

Change-Id: If8ea696acad3ea8b8c4bd230e44bd908352c8370

library: Simplify comparators

Change-Id: Ie4af052e02f5190dae542bbcc34daa495101430b

library: Drop registered_signal_t

Change-Id: I07b7717b1e197201d10ebe25dd502d1c275903a7

library: Do not initialize callbacks list with placeholder element

This does not seem to be needed.

Change-Id: I44196b316411673b19099aad528d20f98ad72136

removed unecessary NULL case

Change-Id: I8ebbb85f323bb68c8a7efb703505d647540f0ee6

library: Add missing const to src user

Change-Id: I92f2f615cd10ef164d70ca17456989894f601455

Temporarily allow all dbus policy by default

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

Revert "Temporarily allow dbus policy by default"

This reverts commit 330e4429171a18f5f28e49c915c507572deff539.

Change-Id: I945e28a1d37b419d5a32425b79ffd999f08de9c6

Merge "library: Ensure subsession_user_t does not contain garbage" into tizen

User len 19 and len 20 tests

Change-Id: Ibf7710a0248fdd0dfcafa885a1b6d9687fcce6b3

Temporarily allow dbus policy by default

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

Add a note to the sessiond.h header on the "" subsession

Change-Id: Id6ddafd8f017ede6423b20539f0c4d0c341570ba

library: Ensure subsession_user_t does not contain garbage

Change-Id: I06fa8c004af2873a3a2926b5bde4fc62043d8f62

library: Fix subsession_user_copy()

This commit fixes strlen() to be invoked on source string, not
destination.

Change-Id: I9eeef27e6c713134ddfedf79e66a3f28dafb7f1d

Release 0.4.2

Change-Id: Ifaaaede75993b827da12b55e0edaaa36ce7d70e0

library: Use assert() to catch programming errors in make_new_signal_callback_client_data()

Change-Id: I9c0b718cab9a010b8094a079e52844b2d03f0619

library: Unify signal handlers

This commit drops quite a lot of repetition by moving switch
and add+remove handlers common parts to separate functions.

Change-Id: I80fc2d3ab6d7990e2633b2a3660ff3ba527ea558

library: Drop cb_data_

We really don't need it as repetition is negligible.  More imporantly, using explicit
paramaters (cb, cb_data) is far more readable than looking for yet another macro def.

Change-Id: Ibb0c3a3723deec76b4fa615546343393e6b82920

Add static where appropriate

Change-Id: Ic3027ddecbbefd4696ddf0636407c5d4f7a2ae17

Drop dbus_signal_subscribed

... as it is unused.

Change-Id: I9724221dac88448a48ff19d98f561269e620d93e

Removed unused lists

Change-Id: I280f0852aa544d136ea8c0d8b8456ad6807e9bf3

Fix some leakage in the library

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

Release 0.4.1

Change-Id: I9f2e8ca7595cdf3214e99837074110eedfa57a37

Correction in test description

Change-Id: I557a780c119530ed9680377bc82dd8bdb4293ab9

lib: get_current_user: Avoid off-by-one while copying user name

- fix off-by-one error while copying user name
- tizen style fixes

Change-Id: If2fc3204413f748526959f567fad9b032502cd75

lib: get_user_list: Numerous fixes

- use SUBSESSION_* errors only (instead of 0/TIZEN_* domain errors)
- use exact types where possible (subsession_user_t instead of void)
- fix off-by one when copying gvariant string whose lenght >= USER_MAXLEN
- minor style fixes

Change-Id: I47821432306f48caac719e2a09c3af8c1b8b452b

libsessiond: fix a memleak in `subsession_get_user_list`

Change-Id: I543f85dfddd563db71e834acf0874dae09eb3d82

Default user on libsessiond

Fix test to check default user id
Minor fix in clientExample

Change-Id: I1b95e3e6f77141804d35f04f4c2169b40dc244b5

Include default user in `GetUserList` call

Change-Id: I9e2319615f06e2896157adba0c8545c2e41fffdc

Clearly guide how to free user_list from subsession_get_user_list()

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

Release 0.4

Change-Id: Ic3c22630f8947c04289012ef1f057d365216e302

Client example: minor tweaks

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

Adapt clientExample to char *

Change-Id: Ic68fc382f0d1e91f062ff6d40ac7885f1a864fdb

libsessiond: Change user id from int to string type

Change-Id: I89da68279daa124b93eaf3a32db96e2b421d6990

Change subsession_id type to std::string{,_view}

Co-authored-by: Adam Michalski <a.michalski2@partner.samsung.com>
Change-Id: Ie5d645b9172d7423f40b74c5b92c44ac106b8c1e

Remove unnecessary `enum`

Change-Id: Ibd8c2f7efb8999a97d76c69ac04f0ca6e014d42e

Release 0.3.1

Change-Id: If7467fe5fb2b5549a768be32d49fe5dd3b2acc7e

Merge "Add log notification" into tizen

Add log notification

Change-Id: Ia4e3809385d66e0bb7b00aee40c7bbc66a24d524

Fix an inappropriate usage of std::string_view

Change-Id: Ib78a4dee5588cd50ebda2666d9536acc8156de55

sessiond: Apply systemd-based dbus activation

Change-Id: I7c5323eef839ead0fb9e3ae066310c3b50aa357e

Fix a blunder

HMMMMMMMMMMMMMMMMMMMMMMMMMMMMM

Change-Id: I0b993c1e64fb600ef30c854ad688eef1ca6c97d5

Add std::string and std::string_view support to tuple_g_variant_helpers

std::string is supported in both cases. std::string_view is only allowed
in input and not in output.

Change-Id: Ib7d23a434309943e475e70b6a2ace943e55ec8ae

Refactor tuple_g_variant_helpers

... to be a bit more extensible.

Change-Id: I2c37833fd8c5296d2d81413316173a35910e7a8f

Client example: Remove unnecessary header file

<unistd.h> was needed by sleep/usleep calls which were used temporarily
during the client app race condition tests. It's redundant now, that's
why it has been removed.

Change-Id: I9d97189a91d6630daa5853d0f62e6ac9042e4f66

Client example: improve commentary

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

api: Adjust spacing

Change-Id: Iab0a1c38894bf6ce4f0c10ce39337e832cc253e9

Client example: extract duplicate code

 * changes a loop condition to the first-last form,
   which is equivalent and used everywhere else.

 * changes `switch_user_test` to return bool to
   conform with the new convention I am in the
   process of enforcing.

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

Client example: rework mutexes

 * add a lot of commentary, including to the newly added
   example of passing values to the callback.

 * use a conditional variable instead of a single mutex.
   This is because glib mutexes (and their underlying
   default POSIX mutexes) have undefined behaviour when
   trying to lock an already owned mutex from the owning
   thread. As a side effect, this gets rid of the trylock
   mechanism which (even though it worked) was confusing.

 * fix the mutex being left locked on subsession call error.

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