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>

Client example: error handling fixes

 * start handling errors from event (un-)registration and getting user list

 * use SUBSESSION_ERROR constants supplied by the lib instead of magic

 * reword error messages to better reflect what happens

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

Client example: dispel magic around program returns

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

Client example: main loop handling fixes

 * use a GThread instead of std::thread since everything
   else uses glib facilities instead of C++ ones

 * extract the wait at the end into its own function

 * fix the main loop not being unref'd

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

Client example: extract color magic to a function

Could be parametrized for more colors, but somehow
these don't exist (what happened to red errors?).

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

Client example: cleanup globals

 * the loop variable can be local.

 * it is safe to initialize an atomic variable
   non-atomically as long as there's only 1 thread.

 * make global constants into constexpr vars,
   as opposed to regular const vars or macros.

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

clientExample: refactor some code

- removed race condition causing the main thread to terminate before
  returns from all callbacks occured
- refactored some code to conform to the Tizen coding standards
- removed some unused variables
- added some comments to the source code

Change-Id: Ifa080f9f48747db201b7e73016b414dd5fe1b400

Rearrange the API order in sessiond.h from basic to advanced

+ add .gitignore

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

Release 0.3

Change-Id: Iecbc675df568ee7dd88f1589145d5ee686e71e6a

libsessiond: Drop unused code and data

Change-Id: Ia02761d1f9d950e1997f9a590b7d818cd980c001

packaging: Improve package description a bit

Change-Id: I7a22e6554ed808288ab595f83494ad697b17a0b8

Integrate clientExample

Change-Id: I4200e5903e2820652ee26a774e0016508010cf04

Update tests to async callback call result check.

Change-Id: I91eace5907919b6282a3a4b84fe52aabf1f8684b

subsession_{add,remove,switch}_user: make async

Also they now accept null callbacks.

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

Revert 'result' parameter in subsession_reply_callback

Change-Id: I8397ebe73780a05ba4be2cf3f01eaa4974d06e74

Change error returns in subsession_register_event_callback and subsession_unregister_event_callbacks

subsession_register_event will return SUBSESSION_ERROR_ALREADY_EXISTS
if event on the same subsession_uid was previously registered

subsession_unregister_event will return SUBSESSION_ERROR_NOT_AVAILABLE
if event on the subsession wasn't or previously unregistered

to change callback and callbacks data
unregister and register event again is required

To update the calllback and callback's data the call of subsession_unregister_event_callback
and subsession_register_event_callback with new data is required.

Change-Id: I2ad90f7ac04e9e5f8417f5842db920e91c94511c

Reduce some copy-paste

Fixes a memleak in one of the functions.

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

Add parameter validation to `subsession_get_current_user` and `subsession_get_user_list`

Change-Id: Ie70e797c585114381a96f5967b8f98e8a5ac60b1

api: Drop redundant fields from subsession_event_wait_done() and clarify assumed usage

Changes in:
  - public interface header (libsessiond.h) authored by Karol Lewandowski
  - the rest implemented by Michal Bloch

Change-Id: I2d5337f59c868a614674f1b10001dc8c776f6f91

api: Drop unused return value from subsession_event_callback

Changes in:
  - public interface header (libsessiond.h) authored by Karol Lewandowski
  - the rest implemented by Michal Bloch

Change-Id: I89808022e341a0ec10863181ba8788300b55b648

api: Drop unused paramaters and return value from subsession_reply_callback

Changes in:
  - public interface header (libsessiond.h) authored by Karol Lewandowski
  - the rest implemented by Michal Bloch

Change-Id: I7d391d4f57fee0925fa0ccb57c2a4d2e99d88c98

api: Move generic fields to the top of subsession_event_info

Also happens to fix some functions referencing the wrong union member.

Changes in:
  - public interface header (libsessiond.h) authored by Karol Lewandowski
  - the rest implemented by Michal Bloch

Change-Id: I5dde2519a85c2e0ab3fc9621324eb140b0825743

Revert "Restore subsession_*_user_wait_done due to other programs linking to them"

This reverts commit 3d88538370fa18dd4a44554e7b57b4cbfa50d844.

It's not enough to make rest of Tizen build, so better drop it.

Change-Id: Ibb790ee5287e00cae84a1915e02098b8d7b61cf5