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

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

Change-Id: I8e54fa4aff6fb92ff0fe8e8f00f2c663cf73a93c

Fixup: add missing sessiond-platform.h

Change-Id: I1d5117d6745d2ea14d8911703e2a782bb79f661c

Release 0.7.7

Change-Id: I62e20d21f07227994e0dd8e8474fe4cb803e3fbf

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

Change-Id: I1d4357cfc5a3fadd6b66d1b2a8d5717a1520adc0

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

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

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