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

Restore subsession_*_user_wait_done due to other programs linking to them

Change-Id: Ic7bed033dc062d1f8eda746206aca6976df52a05

Merge branch 'devel' into tizen

Change-Id: I259e76c89662ad686bcfa1e7a0f95ea18869f618

api: Add documentation

Change-Id: I2a9a913e3a329f9134ecd7fe511b38fea1cc2efc

Changed init_callbacks_list to int type

Internal append_to_callbacks_list internal function
that contains g_mutex management

Change-Id: I4d8380431bef46941cc23e5bfc61bc0a8e968698

Event wait done API function

Function subsession_event_wait_done that replaces:
subsession_add_user_wait_done
subsession_remove_user_wait_done
subsession_switch_wait_done

Error tests for subsession_event_wait_done

Appened permission denied error code to
map_dbus_call_error_to_return_value

Change-Id: I753b6ef5f3ee68f45ddacf58d58065855ae1825d

Merge "Added clientExample" into tizen

Tests with errors

Sending incorrect values to sessiond
server and takes back error codes

for:
 subsession_add_user
 subsession_remove_user
 subsession_switch_user

Removed test_api_adduser.cpp.
Removed test_api_removeuser.cpp
Tests from theese files are moved to test_api_addremove_user.cpp

Fixed errors in libsessiond  detected by tests

subsession_add_user,
subsession_remove_user,
subsession_switch_user will not call callback if error

Validation macros partially moved to lib.h
Validation macros are more generic

Change-Id: Ide27b2cfd8ffc6d31813230f3a43bafb6079584c

Unify internal error code to DBus name mappings

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

Added clientExample

Change-Id: Id3b0b539c5a22060dbedddaf1690b37c51d468ac

Delete duplicated libsessiond.pc.in

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

Implementation of subsession_get_current_user

removed subsession_is_supported

Change-Id: I987fe917859571087a2a2abd237facaa0856e15e

Update test for SUBSESSION_SWITCH_USER_WAIT

Additional call of API subsession_switch_user
that set user to "1" before change it to "2"

Change-Id: I9e0a6637ec7e1a4fb4208a34630788afc0f5dbef

Registration of switch user completion event

Implementation of API function subsession_register_event_callback
for SUBSESSION_SWITCH_USER_COMPLETION flag

Change-Id: I8049e7c9b39d5f726b24353f9155a9374ac83f4c

[fix] sessiond: Rename DBus error enums so they are not confused with
the error codes defined in libsessiond.

Change-Id: I89964a5207a24d7b1b8cea482430dbfc83c731df

Merge "Allow user with id 0 to be switched to" into devel

Allow user with id 0 to be switched to

Change-Id: I0f51ed2c4f8bc58a815b3361e413fbe0b6e8db2d

Prevent removing currently active user

Change-Id: I718e9bb0e2d1be7b7e19d4c4ec295b335f0cf436

Rework DBus exception handling in sessiond and libsessiond

Change-Id: I591af8d9d1dd927bfffddbc45a48540488893699

Add validity checks for `SwitchUser` method

Change-Id: I586044883648768d682c70757b521564ea57d17e

Deduce an array type

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

Fix library installation path

32-bit build: /usr/lib
64-bit build: /usr/lib64

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

remove 'prev_user' parameter from libsessiond

Change-Id: I459e947e3a2177f66f327359a4a9e77aa0da7d8f

Remove redundant `prev_subsession_id` parameter from the
`SwitchUser` method

Change-Id: Ice4709be50884f8dc807edbba5f17555bbcb1a3f

Implement server-side GetCurrentUser method

Change-Id: I765ee5179e4cf4f9c849a7946ee18f5e0fc47b01

Added subsession_unregister_event_callback()

This function unsibscribes event callback by uid and removing it from list.
removed unused unsubscribe handler
API function change:
from subsession_remove_event_callback
to subsession_unregister_event_callback

Change-Id: If58381dfa17c27ce2764aeabf8d698212561ab17

Event's callback registration

Register callback for method - signal:
AddUserWait - AddUserStarted,
RemoveUserWait - RemoveUserStarted,
SwitchUserWait - SwitchUserStarted,
registered by API function:
subsession_register_event_callback
with tests.

API function subsession_switch_wait_done
(called by callback)

More descriptive error codes
Removed deprecated functions

Change-Id: I307fe0fc624292a9c3408169e6653e2eed07544b

Add handler for SUBSESSION_SWITCH_USER_WAIT and SUBSESSION_SWITCH_USER_COMPLETION

Change methods calls in callbacks data  to correct ones.
Remove unused parameter "event_bits" in register_event_callback and subscribe_only_callback

Change-Id: If451d81d5ab193061a0ab46ddd61fa9d3769526d

[fix] Add input parameter validation for GetUserList

Change-Id: Ide73c817315e939ffffc92e96f2b608ec85b482b

Merge "[fix] Add logging `invalid_argument` exception" into devel

Merge changes Ifa0cbed1,I766670d5,I03094609,I57c9e563 into devel

* changes:
  wait_manager: Add some prose
  Add privacy to wait_manager
  Split wait_manager arguments
  Extract `wait_manager` template class into a separate file

wait_manager: Add some prose

Change-Id: Ifa0cbed179e24bd7a840f578c91daae81268eca9

Add privacy to wait_manager

Change-Id: I766670d51f4f062b1113e788c1967e1ad473b2c6

Split wait_manager arguments

Change-Id: I03094609fab5072507081ee2ec4b437d3e602583

fs-helpers: Refactor some code

Change-Id: Idbfdcc8fd014075781f7cd34da187d8754763532

Extract `wait_manager` template class into a separate file

Change-Id: I57c9e56335a9365bcbdbf6a522a3fd32581a7dee