fix logging

Logging internal definitions moved from libdbuspolicy.h to
dlogger files.

Change-Id: If4040ac2943a860a915681fa5d44855773d5f10f

add error logs to bus initializing and policy parsing

Added dlog error logs to dbuspolicy1_init and policy parsing for easier
debugging. Changed behavior when parsing: exceptions thrown while
parsing main system.conf/session.conf files cause interruption
of parsing. Exceptions from parsing other .conf files are logged, but
parsing is continued.

Change-Id: I615d1a49fb552518d0040df1d875ba20ca6839f7

Add memory dump functionality

Useful for memory consumption analysis, prints out content of
structures used to keep policy and their memory.

Change-Id: I127e1ab36f498d1dedff85e7a2ce4751b276d75e

Added latency & memory usage test

Change-Id: I2daf0306bce9468b397fa78f2f5a8437c5bd8389

svace fix

Change-Id: Ib6b047eb6fe90476d6c28980589ed7eb777eb4e5
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Use expat SAX parser instead of boost::ptree (DOM)

Modified xml parsing methods to use expat parser, which is much
faster than boost::property_tree (initialization time decreased
by about 30%). Added missing includes after the removal of ptree.

Change-Id: I8973fe7ad7bbfe7928ef261fe07305f54e640b76

added checking bus-access rules

Implemented enforcing access/opening dbus rules. Added a check to dbuspolicy1_init method
(dbuspolicy1_init will fail if the application is not allowed to access chosen dbus bus).
Moved opening kbus after the check to avoid situation when a process that isn't allowed to
access the bus will hold an open kbus descriptor. No changes are required in glib and dbus
libraries to work properly with the new behaviour of dbuspolicy1_init, but some reordering
should be made to assure that the attempt to access bus will fail exactly in the same place
as when using dbus-daemon.

Refactored logging items (more refactoring of tslog needed).
More compact decision logging for better logs readability. Fixed incorrect logging of checking own policy.
Refactored updating groups to facilitate mocking and testing.
Addedd access/open bus tests. Removed repetitions from Makefile.
Fixed static_assert error msg.

Change-Id: I5042ba50b21ec6cc2c6423b3ade1e5c78d45625b

refactored parsing xml, added handling <include> tags

Up to now <include> tags were ignored, only <includedir> worked.
Assuming that we won't use selinux, so we can ignore includes
tagged as if_selinux_enabled.
Fixed undefined behaviour when modifying std::string.c_str() using dirname.

Change-Id: Ia56f030ec509721584dd2ccd476976ea8f66a440

removed duplicate code via templates

Removed duplicates. Corrected type of iterator used in switch/case.

Change-Id: I35a6de8bea5ee04b9a107c5b41206623a7e72a4b

refactored parsing policy

Changed unnecessarily recursive code into code that reflects the actual
configuration syntax, for better readability and maintainability.
Removed unused __tag_state and __attr variables.
Fixed potential future bug: ptree api doesn't guarantee that <xmlattr> will be
the first child, so we shouldn't assume it will be first.
Fixed bug, when policy at_console was parsed as the last seen policy type
(or as unitialized type, if it was the first policy read).
Added test case to assure that at_console policies will always be ignored.

Change-Id: I7e28262ee4837547e10ca1b33f7d5a90166f4667

fixed ItemOwn::__name potential memory leak

Bugfix. Fixed leaks reported by valgrind.

Change-Id: Icd81120781313b662fad22d17f6ff9dbadc5cf03

new implementation of own/own_prefix rules handling

Bugfix/rewrite. Previous implementation was based on the incorrect assumption
that "ab" is a prefix of "abc", while according to the dbus specification
"ab" is a prefix of "ab.c" but not "abc" (elements/tokens must be dot-separated).
Added multiple regression tests, covering both simple and exotic cases
(like allowing a prefix "a.b.c", but denying "a.b.c").
All expected test results verified against dbus-daemon (except cynara checks).
All new tests use default-deny-own policy, changed tests directory structure.
Fixed fragile DecisionItem class which assumed ownership of __privilege,
but didn't hide it's copy-constructor, which could result in freeing freed memory.

Change-Id: I98924ccf475ddd5e49e2eac99c9f1b586f982b90

fixed loading incorrect files as configuration

Bugfix. libdbuspolicy should load only files with names ending with ".conf",
as dbus-daemon does. Added an incorrect configuration file as regression test.

Change-Id: I868c04d6475101abfe2ffcd551b8eb9ce9bf4305

naive_policy_db: remove try-catch

* dlopen deadlock issue
 Both dlopen and internal try{}catch for stl::map.at() are trying to acquire 'global lock'

Change-Id: I185b3aea9964cb1eb0a87a0f00bce9ec15633315
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

modify linker option

To prevent unload libdbuspolicy.so at runtime, add linker option 'nodelete'

Change-Id: If92547f63a14473f58d36d9fc07028cbbf13622b
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

svace fix

add null pointer handling for getpwuid_r

Change-Id: Ic9d33067b4da30cd9d336a4a75766d59f9010c57
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

svace fix

Change-Id: I9290d145b40606711992a4a01fea83d13ed7582b
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

policy checking scheme: add mutex lock for data structure(std::map and std::vector)

Change-Id: I4a213d90dca2447cbd5678083a41c18f10e958fa
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>
(cherry picked from commit 82139a81405a9e4ce38ce37439612079643864be)

kdbus: modified to get label for signal

Change-Id: Ia6fa61fe49084e25270b15387980299fe0b40b44
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

policy checking scheme: support supplementary group

Change-Id: I163bc517d0fdcdbbc8b971ccdf927aa5df47af4a
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

dbuspolicy1_check_in: add handling null sender

null sender at dbuspolicy1_check_in meaning sender has no well-known name,
We can change this to garbage char.

Change-Id: I7ee36d3b77bd6cf86cbea0286f0652f7d1a01774
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

dbuspolicy1_check_out/in: enable policy check for signal

Change-Id: I241f69bc5657ab904d2bf5eaba006188e53cb9aa
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

dbuspolicy1_check_out: enable receive_sender rule

Change-Id: I48ad1e77353424c73371ded14e671fef499782e4
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Modification of coding rule violation

Change-Id: I3e0c88f7c85200267d8cca500826da0112ebe323
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Modification of coding rule violation

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: Id6cafd73c09a8d4c444a04d145da6686ef3ec8ac
(cherry picked from commit b7b3a3a5986174a460adc96d6d5f4208c3bde4e2)

dbuspolicy1_check_out/in: ignore signal

Change-Id: Ifb95079fe93a04ab1f871c49e13fa0cec54c6935
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

policy checking scheme: modified to get connection information from kdbus

To check dbus policy it is needed to get every information, especially multiple names and label.

For example,
if Service A have multiple name such as name_A1, name_A2, and name_A3,
and if Service A specify the rule as below,

<deny send_destination="name_A2"/>
<allow send_destination="name_A1"/>

then according to dbus policy,
last specified rule '<allow send_destination="name_A1"/>' is applied,

*** In dbus policy world dbus treat multiple names of Service A as same name,
this also affect every other names of Service A, meaning,
<allow send_destination="name_A1"/> or <allow send_destination="name_A2"/> or <allow send_destination="name_A3"/>

After all, Service B can send message to name_A1, name_A2, and name_A3.
send message to unique name of Service A also allowed by this rule.

*** unique name also treat as same as well-known name of Service A.

Change-Id: I2253296267d8c844c53813455c829bf625f4360c
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

cynara : modify cyad option. requested by security team

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: I5d58b692a30c9c3570d93c04effbada32e61dc1e

dbuspolicy1_check_out: change sender to NULL

if sender is unique id, can_recv will return wrong value

Change-Id: Idf8b3ce964cc4d5d064d8bf125071772b4987a2b
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Merge "Revert "retrieve label on every cynara check"" into tizen

Revert "retrieve label on every cynara check"

This reverts commit fe403b9aa7ad1ff7e56327af2d88477b26094746.

Change-Id: I8dc57d2e6d821650e5d0d5198c6e1a0cd79fed84

svace fix

Change-Id: I72700b1e9a1929388c181eb536214eaa98a696f2
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Merge "modified to init global instance once" into tizen

modified to init global instance once

Change-Id: Ib5092c6bee952b7a51f7da90d9d65414a0f10697
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

fix build error for gcc 6.2

Change-Id: Idec2ea8f8d55140eb1864d0f981f0071b5898bba
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>
(cherry picked from commit 74494cf8da978299c73034518e0900ac0488bd23)

fix build error for gcc 6.2

Change-Id: Idec2ea8f8d55140eb1864d0f981f0071b5898bba
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

svace:handling wrong policy syntax

Change-Id: I8871c77bf5f58ad89885c128f96442ce009197a8
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

tests: Add test package for libdbuspolicy - again

This reverts commit 376f38dafa0b7ee5eae3d8f0ba0333c84a79742d,
which reverted commit bb7d32bb63f1d0fc463255000fbc773a119314a9.
This is different from commit bb7d32bb63f1d0fc463255000fbc773a119314a9
in handling builds - it provides single spec file.

Additionally, corrections for tests, mostly
excluding initialization of the library from "test cases".

Author: Kazimierz Krosman
with some corrections by Adrian Szyndela <adrian.s@samsung.com>

Change-Id: I64a58818c5c7f692db38f8225bec1e7a11954de6
(cherry picked from commit 575b04077b18ca810aa1974314785e32340d4bb6)

tests: Add test package for libdbuspolicy - again

This reverts commit 376f38dafa0b7ee5eae3d8f0ba0333c84a79742d,
which reverted commit bb7d32bb63f1d0fc463255000fbc773a119314a9.
This is different from commit bb7d32bb63f1d0fc463255000fbc773a119314a9
in handling builds - it provides single spec file.

Additionally, corrections for tests, mostly
excluding initialization of the library from "test cases".

Author: Kazimierz Krosman
with some corrections by Adrian Szyndela <adrian.s@samsung.com>

Change-Id: I64a58818c5c7f692db38f8225bec1e7a11954de6

Revert "tests: Add test package for libdbuspolicy"

This reverts commit bb7d32bb63f1d0fc463255000fbc773a119314a9.

Change-Id: I6bc92edf4cd438c07f6ed99acf4e8bfe531d1d1a
(cherry picked from commit 376f38dafa0b7ee5eae3d8f0ba0333c84a79742d)

svace:handling wrong policy syntax

Change-Id: I6230b4487625240a0ee9f422c0203f0ec33a5d19
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Merge "Revert "tests: Add test package for libdbuspolicy"" into tizen_3.0

Revert "tests: Add test package for libdbuspolicy"

This reverts commit bb7d32bb63f1d0fc463255000fbc773a119314a9.

Change-Id: I6bc92edf4cd438c07f6ed99acf4e8bfe531d1d1a

retrieve label on every cynara check

Change-Id: I5e6fb0f44589ebb295af1db43b8aa5f1838d3c72
(cherry picked from commit fe403b9aa7ad1ff7e56327af2d88477b26094746)

remove unused g_udesc.{user,group}

Change-Id: I767b9c7b095692466c0bb4d25cb8ed707e088a23
(cherry picked from commit 3b8cddc1e85c5cde1c9ed30a5d374e70c959c318)

libdbuspolicy.spec: disable tests [fix critical runtime breakage]

Change-Id: I9c3be5ccbbd1f5326542d11ac818b350038076f6
(cherry picked from commit d260c0a6cb0ca0e6552e0370bfd93eff85b0107c)

retrieve label on every cynara check

Change-Id: I5e6fb0f44589ebb295af1db43b8aa5f1838d3c72

remove unused g_udesc.{user,group}

Change-Id: I767b9c7b095692466c0bb4d25cb8ed707e088a23

libdbuspolicy.spec: disable tests [fix critical runtime breakage]

Change-Id: I9c3be5ccbbd1f5326542d11ac818b350038076f6

tests: Add test package for libdbuspolicy

This patch introduces several modification that
allows test package building.

* Test version of libdbuspolicy shared library reads
policy files from /usr/lib/dbus-tests/config/ instead
of /etc/dbus-1/ ...

* new package libdbuspolicy1-test contains test
version of libdbuspolicy library with test policies
and test binaries.

* test-runner binary is compatible with dbus tests
framework.

Change-Id: I87bffc8bd24c05180441d74df546d4148a238ef1
(cherry picked from commit 42f084f08653aa5020342451c3c1a82aebf65a7a)

tests: Add test package for libdbuspolicy

This patch introduces several modification that
allows test package building.

* Test version of libdbuspolicy shared library reads
policy files from /usr/lib/dbus-tests/config/ instead
of /etc/dbus-1/ ...

* new package libdbuspolicy1-test contains test
version of libdbuspolicy library with test policies
and test binaries.

* test-runner binary is compatible with dbus tests
framework.

Change-Id: I87bffc8bd24c05180441d74df546d4148a238ef1

remove unnecessary cynara_finish() call

Change-Id: Ifcf14f42b8fc2b65c8a5598525b7b0a905f42aee
(cherry picked from commit dadb0935c68e2d09cbedbda1b3afd552f75abc43)

remove unnecessary cynara_finish() call

Change-Id: Ifcf14f42b8fc2b65c8a5598525b7b0a905f42aee

circumvent destruction of ::policy_checker, remove destructors that are empty or never called

Motivated by KONA bugs (ex. DF161028-00875). VD product environment
apparently likes calling libdbuspolicy1 public functions while exit
handlers are being called (and thus global objects are being destroyed).

Solution: circumvent destruction of relevant globals altogether.

Change-Id: Ibeab05ed83cc24dc6a5da926fb542f31d8a33af1
(cherry picked from commit 1e961ec457d48198118bbff42204d668d714bccb)

turn DbAdapter into direct member of XmlParser

Change-Id: Ic807550cabd2cad3177c35a6af14d142b29eaf0b
(cherry picked from commit 1c8bd17ca32a21ca16f2bf625fe788eacf2e0a72)

remove unused PolicyIterator

Change-Id: I06de62c3359186b094b51def7beff47752bfa998
(cherry picked from commit b9c833dbfa425b51f93feff09a827b4222bb7820)

circumvent destruction of ::policy_checker, remove destructors that are empty or never called

Motivated by KONA bugs (ex. DF161028-00875). VD product environment
apparently likes calling libdbuspolicy1 public functions while exit
handlers are being called (and thus global objects are being destroyed).

Solution: circumvent destruction of relevant globals altogether.

Change-Id: Ibeab05ed83cc24dc6a5da926fb542f31d8a33af1

turn DbAdapter into direct member of XmlParser

Change-Id: Ic807550cabd2cad3177c35a6af14d142b29eaf0b

remove unused PolicyIterator

Change-Id: I06de62c3359186b094b51def7beff47752bfa998

Removes file xml_policy.hpp. It is part of old implementation and is no longer used.

Change-Id: Ife804864f767ddd92a979e983cd45164e89c6315
Signed-off-by: Krystian Kisielak <k.kisielak@samsung.com>
(cherry picked from commit 9cce1d8723e29a83db77f1d90d8fac74f19eb6cc)

Merge "Removes file xml_policy.hpp. It is part of old implementation and is no longer used." into tizen

Remove a redundant line

Change-Id: Ic41712669be7b626c0eee19a0177b45b495f21d0
Signed-off-by: Michal Bloch <m.bloch@samsung.com>
(cherry picked from commit a4e6142895b50cc6faf03670a579e48696b5bb9c)

Remove a redundant line

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

Merge "Minor fix to failed test output" into tizen

-Wextra -Werror

Change-Id: I1b74a76cff3ecb38b646a3dd9a7327d25e5fa1d7

Minor fix to failed test output

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

Merge changes I097ff109,Ie115904d into tizen

* changes:
  Added doxygen documentation.
  Adds creation of doxygen documentation package.

Added doxygen documentation.

Change-Id: I097ff1090998637d5c51ffbfa6023bfd0c04e405
Signed-off-by: Krystian Kisielak <k.kisielak@samsung.com>

svace:change strcat to strncat

Change-Id: I546599c8f5c850eda6374a792492e7db1e9dcf7a
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Merge "Added description label to libdbuspolicy kdbus connection" into tizen

Merge "Fix: free memory by KDBUS_CMD_FREE ioctl" into tizen

Adds creation of doxygen documentation package.

Change-Id: Ie115904d41dea90d5c26844c483ebc0c87fbaa7e
Signed-off-by: Krystian Kisielak <k.kisielak@samsung.com>

Removes file xml_policy.hpp. It is part of old implementation and is no longer used.

Change-Id: Ife804864f767ddd92a979e983cd45164e89c6315
Signed-off-by: Krystian Kisielak <k.kisielak@samsung.com>

Merge "Bugfix:wrong gid/uid converting" into tizen

Bugfix:wrong gid/uid converting

if input is as below,
     <policy group="%{TZ_SYS_USER_GROUP}">
DbAdapter::convertToUid / Gid function return 0, not -1.

Change-Id: I4a36e0b387bd532b72e37a339c1a308ad8bf55f8

Fixes policy "*" names issue. Adds related test.

Change-Id: I3bb40c1ccc47de02d06837d1e36bb64be6b4b3b7

Fix: wrong data preparation for policy requests

Additional Fix: wrong assertion (name validation)

Change-Id: I79fa2bc27f8a8406f067bc7e4299bdc422c79b37

Revert "revert changes:revert commit id dc3048a8de00c01514c14ef465a87b8ca9a7c704"

This reverts commit 1b4d4f2311ae31440417fafd3233177d39bfd768.

This revert is not nessesary- it requiers only simple fix.

Change-Id: I4591012edd0d039b1eb7bb382cf0b34b8e5e6771

revert changes:revert commit id dc3048a8de00c01514c14ef465a87b8ca9a7c704

previous submit has bug, and revert it
"Prefix tree search for ownership rules, removed virtual methods."
id : dc3048a8de00c01514c14ef465a87b8ca9a7c704

Change-Id: I0a73eab24c95f1a6cc3a4e37a9ffb0d2da943e3f
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Added description label to libdbuspolicy kdbus connection

Change-Id: I3c17aa33ea71f9eab6cd2bd161500b1363804389

Fix: free memory by KDBUS_CMD_FREE ioctl

Change-Id: I6252c1783fa9dd52343bd128fa975b38191247f3

Fix: enable libdbuspolicy for multiple init.

Change-Id: I5ffea43051cb5bd51320333fde273815e12801ae

Merge "Fixes coding style issues." into tizen

Merge "Prefix tree search for ownership rules, removed virtual methods." into tizen

Fixes coding style issues.

Change-Id: Ibbbaab71538906fa123156c3260707b9edef4bcc
Signed-off-by: Krystian Kisielak <k.kisielak@samsung.com>

bug fix:wrong index of kdbus_unique_id

Change-Id: I374bf3de298adf49d67e48b199590a1aa164d0a5
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

kdbus_unique_id fix

Change-Id: Ice25972fdc38e032f8d06ccb66eec8e3f202337a

Prefix tree search for ownership rules, removed virtual methods.

Change-Id: I90909b035da16dce39ac28278c1561eca0396335
Signed-off-by: Krystian Kisielak <k.kisielak@samsung.com>

kdbus: synchronize kdbus interface header between repositories

Change-Id: I8e47edf8c61699a99523dead09bb3ef321894d31

Change in API to return internal errors by libdbuspolicy

Libdbuspolicy now returns error codes via main API
(check_out and check_int functions):
DBUSPOLICY_RESULT_ALLOW 1
DBUSPOLICY_RESULT_DENY 0
DBUSPOLICY_RESULT_DEST_NOT_AVAILABLE -1
DBUSPOLICY_RESULT_KDBUS_ERROR -2
DBUSPOLICY_RESULT_CYNARA_ERROR -3

it is needed by gdbus and libdbus to make better descripted logs.

Change-Id: I060de3ec0a767ee56dfd5ae8f6aa4475dd25ffa4

Merge "Fix copying to empty string." into tizen

Fix copying to empty string.

Change-Id: I0d2e111372cbe62fc3ebe9b4211e90b89a335dbf

Merge "Added test description to make test better described" into tizen

add LICENSE.APACHE2.0

Change-Id: Icb8da89c7d0413a6b2bdb79cb59cb75edb9f9b5c

Added test description to make test better described

Change-Id: I90909b035da16dce39ac28278c1561ecf0396335

svace

Change-Id: I745773662dc8578fce7590bd1b3bc2bde3ede5ed
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Fix: xml parser + minor fixes

Fixed xml parser bug that causes that only first <includedir>
works as it should.

Removed deadlock in init when logs are enabled

Minor refactors

Change-Id: I94c97efff478cf919cc5809f106bca1bb1465889

libdbuspolicyt refactoring to meet Tizen coding style

Change-Id: I9765dd74342d695d50b223185c71a84d7f0f1aa3

Tree policy engine decision checker replacement

Tree policy engine decision checker was replaced by naive
search algorithm. For each check call, libdbuspolicy iterate via
policy rules in the reverse order. First rule that matches request
is used to return decision.

Multiple sender's and receiver's names are proceeded during search
procedure (algorithm try to match current rule for each name). Previous version
search all database for each name.

Added unit tests (make check)

Change-Id: Ie78ae8eec285a7042d275d02333372bca4d76d28

Allow incremental build

Change-Id: Id8c02912d1ddf0d3a86bb5e5f999a1dbce035943