Fix flatbuffers reverse_iterator

Change-Id: I77233a416d76198175859c19adccdb41b5ab2ea8

refactoring: hide XmlParser

This disconnects XmlParser from __internal_init() and hides it
behind StorageBackendXML. This way XmlParser is local, non static and
non global anymore, a bit more tightly connected to StorageBackendXML,
but more loosely connected to other components.

Change-Id: I712129dfc9f8da357e7a7d66411aaea3590172e7

refactoring: simplify strings management in XmlParser

File name is passed from external API by const char *.
This (at least partially) eliminates construction of std::string where
it is not needed.

Change-Id: I05375820fe2996cb4f95af5be8bbbab5ecd87fc4

refactoring: make assert more informative

Function parseAssert() showed only that an error occurred.
A macro instead shows also the error condition and line number.

Change-Id: Ibd686286c58b657fd578f92bd4328072aa892933

serialization: extract xml-based db implementation

Change-Id: I84ef422a5451192fe18762c16bc14aa7ed4b9d02

fix uid/gid wording

mapGroups are maps that tie uid with vector of gids. This fixes
misleading wording.

Change-Id: I982e8b474a25da189e16b8871f023e9110787b64

serialization: extract mapGroups from pimpl of DB

Previously, mapGroups were incorporated into PolicySets in pimpl
NaivePolicyDbImpl. But they are in fact not a part of policies.
They are gathered from the system and only filtered through policy data.

This commit extracts mapGroups from pimpl back to NaivePolicyDb,
with some required changes to work with current interfaces, and also with
some not required, but fun changes.
The extraction will allow switching pimpl in the future, keeping the same
groups logic for any pimpl.

Change-Id: I3e1dace93bebb7e19c94cacd7fbf7ceb426d315e

refactoring: PolicyAccess uses common policy base

PolicyAccess works in a very similar way to PolicySend and PolicyReceive,
but it has its own class.

This commit makes common base for all three classes.

Change-Id: I63c89f23a6f65b6e5107c9a17a950b99a6664111

serialization: reworked init for serialized backend

Change-Id: Ie72cf3444dfc7bf446bb6035df6a57d04ae0c180

serialization: implement matching

Change-Id: I2d94b5475939e4ccea92def362fe6cc0b1ec98cf

refactoring: reworked matching

This changes matching.
Previously Items had match() function with MatchItems as params.
Now, MatchItems have generic match() functions for use with both types
of Items (XML-based and serialized).
Therefore, the matching code is the same for both.

Change-Id: I68a90bd3b787c66d72fa30d5fc25e16b0634daa2

serializer: add tests

Change-Id: Ibd782e92c88f63caa290a33b13a890f54071619b

serialization: add printing tool

This adds dbuspolicyprinter, which takes serialized file as input
and prints it in the same fashion as libdbuspolicy does with XML policy.

Change-Id: I20c8d5af6c97975251cf48bcf94649ef668b9abf

serialization: add storage backend

Change-Id: I1830ff322a8c14557d46f8f353a7b3d34a43d187

serialization: rework printing content

We want to print content from both serialized and XML.
This makes a common base for printing content in the similar way.

Change-Id: If010b8adb1e39e6ada7b08b7949a0b446470160a

serializer: add dbuspolicyserializer do the spec file

Change-Id: I25616008fd2d4bb43c084deaea79fae0adda0db4

serialization: add dbuspolicyserializer to Makefile

Change-Id: I9b75dcae61188665f01611f124459f0ec44196cb

serialization: add serialization class

Change-Id: I8eb5d24012b24830c1f8e671805eb091b1d2c9a1

serializer: add schema file

Change-Id: I35ab9247800fa2a00d22fb7f44a3390ff4273e3d

serialization: add Flatbuffers

Change-Id: Iafb4e0840c4b80e894af53c0c50b89cecd0f773c

serialization: move TreeNode definition and add getters

Change-Id: I40c7bc6c59f29bfc6b35930893aa54eacbab6fb4

serialization: add interface for getting policies

Serialization needs access to stored structures.
This introduces access functions on Db->Policy level.

Change-Id: I46b21886c6026a7f6de4741ba20f3a0f228a6cb1

refactoring: extract Policy containers

Preparing for serialization.

Serialization needs to iterate over db structures. It needs a bit of insight
into the actual types stored.

This extracts Policy container classes from naive_policy_db.cpp.

Change-Id: I51d1ce5c63eef28c58423b0e64ff9f5a2bc5d09c

Apply the highest priority to constructors

__attribute__((constructor(101)))

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

refactoring: making bus type a type - part 2

This makes bus type a type in one more function in addition to commit 7b3249.

Change-Id: I22a119ffc09b3e111e6409006baba1a05d857a50

refactoring: simplify name string handling in Own

ItemOwn stores names as std::string, but it gives them as const char *.
Then, it is used again as std::string.

This commit removes conversions, by passing const references to std::string
everywhere.

Change-Id: I0d606cecd554e8619848e271e4e1aa48f5dae453

refactoring: simplify privilege string handling

DecisionItem::__privilege is std::string, but it is converted to const char *
and then converted again to std::string.
This removes conversions, passing const reference to std::string and converting
only when necessary.

Change-Id: I86e9a0e91f4c323d743fac6f88349aff37872998

tests: bring back compile-time tests

It appears one-liner 'make all-tests check' does not work the same way
as two lines:
make all-tests
make check
This brings back two lines, because with one line 'make check' is
not executed.

Change-Id: I3f04b72b4421aab74dcc1a70f4146a329db87bfd

refactoring: remove unneeded template param

Since Items got typedef match_type, they are explicitly
tied to their Match counterparts. Therefore, we no longer need
to tie both type categories explicitly in other places.

Change-Id: I63f048787ae857c97b0087ccf091f80401f98ed6

refactoring: remove exec rights from the source code

Change-Id: I99e1c4ada233637ea0810945f866f468ac3dafff

refactoring: move "parse" methods to the "parser"

Change-Id: Ibd543e118e7546d409f1441e4f7de99b964b4d8f

refactoring: remove unnecessary params in parseRule

Removed params policy_type and policy_type_value from
DbAdapter::parsePolicyAttribute() as they are used once and
are references to policy_type and policy_type_value members.

Change-Id: I59850b91fe739cd275d122a3df83ffb988c2cd74

refactoring: templatize addItem

Change-Id: I40429772c49a74e5d7b3420c72882e3bd8c0a12f

refactoring: make bus type a type instead of bool

Change-Id: Idadba8b5d9aa5cb7b801be89b08699c53767905f

refactoring: remove duplicate db access method

This removes duplicated db access method from naive_policy_checker.
Additionally, friend declaration is removed for DbAdapter,
the access method made public, and code that uses removed method is reworked.

Change-Id: Ifac510e13ba0001390c25d5584c7500c89d665be

refactoring: make pimpl in naive_policy_db

naive_policy_db got a bit bloated, including several internal classes etc.
To clean things up this introduces impl pattern, moving all the private
stuff into implementation file, leaving only public nonvirtual interface.

Change-Id: I32dd22bd219826994a2f52baa12aa64649ac0af2

refactoring: remove user and group from access PolicySet

Additionally, this adds specialized method for getting different
policies to get rid of artificial parameterization with PolicyType
and PolicyTypeValue.

Change-Id: Ib22191aafc8d2823cb52af9b5e61ccf31d207acf

refactoring: simplify PolicySet

This splits artificial context array into separate contextMandatory
and contextDefault.

Change-Id: I05a0b3a66d2f3370f73b6b4992de4a911db331f4

refactoring: prepare for PolicySet customization

PolicySets are composed of context, user, and group policies.
However, PolicySet for PolicyAccess does not need user and group
policies.
This commit prepares base classes for PolicySets that will allow
handy customization of behaviour and composition.

Change-Id: Idde9daf06e2caa2975d899b99f091ece49b66113

refactoring: templatize NaivePolicyChecker::check

This converts check() methods for MatchItemOwn, MatchItemSend and
MatchItemReceive to single template method.
The MatchItemAccess check() method stays in place as it requires
different params.

Change-Id: Ibbc2c0498bb9289f6dbce5ffd3206abf93115d17

refactoring: eliminate NaivePolicyDb::getPolicy()

Change-Id: I4378947432a01974f27ebf115e785556e52155d5

refactoring: simplify NaivePolicyDb::getGroups

It affects dependencies, which mostly lose unnecessary ItemType
parameter.

Change-Id: I32056de6f2c8edf389909cd816a0de656b4f3830

refactoring: eliminate template params in getDecisionItem

Affects dependencies, which also lost unnecessary params.
Policy classes can be now private, it allows single private
and single public section in NaivePolicyDb.

Change-Id: I5f01d2e43f2ffd3f58ef2aa76b1cd06d83365b7b

refactoring: simplify NaivePolicyDb::getPolicy()

Change-Id: I3be0a3682dd7be9510a0145d16cd3b750ee129c9

refactoring: make Policy classes more private

Change-Id: If2724b9f8de024da28e04710a81ba8435dcdf515

refactoring: group methods from same classes

Change-Id: Id6f8124fc7b5423ee4b9922b017f153b99b249b4

refactoring: make own_tree more const-correct

Change-Id: I4c6f7cb72f743e597f6fa295cb4731e01cfcda55

refactoring: make parsing lazy

We don't need to reset all the potential items on every rule.
This commit resets items on first encounter of given type.

Change-Id: I9bd1abe38516c188dd295697109bfe1a337eaf0b

refactoring: get rid of unnecessary MessageDirection

Things done to accomplish removing MessageDirection:
- used ItemSend/ItemReceive instead of ItemSendReceive
- used MatchItemSend/MatchItemReceive instead of MatchItemSR
- used PolicySend/PolicyReceive instead of PolicySR
- database no longer contains pointers

Change-Id: I1c5957dad2181a6c1c42dee0e77be3e8b0ccc471

refactoring: introduce Send/Receive split types

This will help with getting rid of some unnecessary switches/ifs and types.

Change-Id: I784e133063c78f2b14ee50c81d526253ec173f0c

refactoring: move addItem() declarations

This is a move towards making single public and single private
section in NaivePolicyDb.

Change-Id: Ib300da409a4442b1578f3a48ae888c1982de24fa

refactoring: group maps moved into Policies

Change-Id: I5def57a4ed74a2a72ab76a527091c4e56d5c1b67

refactoring: make common interface for Policy classes

Change-Id: I02efdddbafcce4bbbd8e7f7630cececaa1b362ed

refactoring: move PolicySet methods into proper class

Change-Id: Ia9c72791fd592017ac3e0284f1e72f753f398f10

refactoring: introduce MatchItemOwn for matching

Change-Id: I0c01db965a1b4d61cea0ac7f0b825d93cb331ed9

refactoring: remove params from ItemSendReceive()

Params for ItemSendReceive are never used.

Change-Id: I0c4c6bbaf68a4f65472267b3498c14f335561d12

refactoring: replace char* with C++ strings

Change-Id: I2554b13d0d00ed1b8ce5dbdfba7cb5f9cb6db6c7

refactoring: replaced snprintf with to_string

Moved string conversion to cynara.cpp.
Eliminated snprintf, char array in favor of single call to to_string().
Reworked ifs to switch.

Change-Id: Ib57f459183328f55a16412af84214e77f7750f58

tests: make tests passing again

Do not confuse with 'make tests great again'.
But at least they should work and give results now.

Change-Id: I7f0c69f329e7ef1a1813c23eb51387494c717226

tests: build tests in a single build

Spec file makes two builds: one with --enable-tests, and second
without tests. This was used to add a special function for changing
creds and checking config file.
This commit exports the behaviour which is different in both versions
to separate files and uses those files for linking with proper versions.
It also makes the special function not exported anymore, as it is not needed
for static linking with test programs.

Change-Id: I72cc44e35c17da85e73ac503bbd447a4ee24e6cc

internal: fix mistaken getgid()->getuid()

Change-Id: I97a0514e779c3a5efc3e0dc1aae3e770d8769102

svace fix

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

internal: make logs less cluttering

Many logs are under an if-clause, making them take more space
than necessary. This replaces most of if-clause-and-logs
with one-liners to improve readability.

Change-Id: I7e9ada35160974c8cc38de1c6e0217de15912c6e

internal: replaced manual reverse iterator with STL-provided

PolicyConstIterator's purpose is identical to vector::const_reverse_iterator
from vector. This removes the unnecessary code, replacing it with
the usage of vector::const_reverse_iterator.

Change-Id: Ib3063eb6eb2a4fd23f8da8a2cce75c808abb3c00

internal: make debug logs small to be more readable

The logs take relatively large space within actual source code.
This makes them smaller to help with avoiding distraction.

Change-Id: Iafca91f5e30f4446e28215cafbf78325ac9287bd

internal: make nondestructible global static_parser

This should help with destructing global variable
while using it in other thread.

Multi-threaded daemons often kill the main thread before killing the other threads.
This patch prevents instance memory release when the main thread exits.
, and thus required to avoid crash on released memory.

.#0  ldp_xml_parser::ItemBuilder::~ItemBuilder (this=0xb67a980c <ldp_xml_parser::static_parser>, __in_chrg=<optimized out>) at src/internal/policy.cpp:510
.#1  0xb67a5de8 in ldp_xml_parser::XmlParser::~XmlParser (this=0xb67a980c <ldp_xml_parser::static_parser>, __in_chrg=<optimized out>) at /usr/lib/gcc/armv7l-tizen-linux-gnueabi/6.2.1/include/c++/ext/new_allocator.h:110
.#2  0xb695806c in __cxa_finalize (d=0xb67a9438) at cxa_finalize.c:83
.#3  0xb679bcea in __do_global_dtors_aux () from /lib/libdbuspolicy1.so.1
.#4  0xb6fdfa1c in _dl_fini () at dl-fini.c:235
.#5  0xb6957a44 in __run_exit_handlers (status=<optimized out>, listp=<optimized out>, run_list_atexit=run_list_atexit@entry=true, run_dtors=run_dtors@entry=true) at exit.c:106
.#6  0xb6957b6c in __GI_exit (status=<optimized out>) at exit.c:137
.#7  0xb6940640 in __libc_start_main (main=0xbefffe44, argc=-1230635008, argv=0xb6940640 <__libc_start_main+280>, init=<optimized out>, fini=0x7f55f69c <__libc_csu_fini>, rtld_fini=0xb6fdf7e4 <_dl_fini>, stack_end=0xbefffe44)
.    at libc-start.c:323
.#8  0x7f5589e0 in _start () at ../sysdeps/arm/start.S:110

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

tests for send_destination_prefix

Change-Id: Ic7943971b11eeae8dae55ccd50b884beb84dba0a

support for send_destination_prefix

This commit introduces support for D-Bus policy extension
send_destination_prefix.

Change-Id: I671560cb71aae96fa3bd47579e5991f08509a6bd

Merge "Secure get_test_id against buffer overflows" into tizen

Merge "Fix Asan Issue buffer overflow" into tizen

Fix Asan Issue buffer overflow

The following bt was obtained on running asan tool on libdbuspolicy before patch fix:

 sh-3.2# cat asan.log.nagsam-daemon.599
 =================================================================
 ==nagsam-daemon==599==ERROR: AddressSanitizer: stack-buffer-overflow on address 0xbe4a5720 at pc 0xb6266535 bp 0xbe4a4900 sp 0xbe4a4948
 READ of size 1083 at 0xbe4a5720 thread T0
 #0 0xb6266533 in printf_common(void*, char const*, std::__va_list) (/usr/lib/libasan.so+0x6f533)
 #1 0xb6267cd3 in vsnprintf (/usr/lib/libasan.so+0x70cd3)
 #2 0xb6267f49 in __snprintf_chk (/usr/lib/libasan.so+0x70f49)
 #3 0xb581c9fd in dbuspolicy1_init (/lib/libdbuspolicy1.so.1+0x89fd)
 #4 0x41abead3 in _dbus_transport_open_kdbus (/lib/libdbus-1.so.3+0x41abead3)
 #5 0x41aabf8b in _dbus_transport_open (/lib/libdbus-1.so.3+0x41aabf8b)
 #6 0x41a93e63 in _dbus_connection_open_internal (/lib/libdbus-1.so.3+0x41a93e63)
 #7 0x41a8eb73 in internal_bus_get (/lib/libdbus-1.so.3+0x41a8eb73)
 #8 0x4488a195 in TCIPCConnection::m_CreateConnection() (/lib/libtvs-api-ipc.so+0x4488a195)
 #9 0x4488a25b in TCIPCConnection::m_Connect(bool) (/lib/libtvs-api-ipc.so+0x4488a25b)
 #10 0x4488a48b in TCIPCConnection::TCIPCConnection(std::_cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::_cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, bool) (/lib/libtvs-api-ipc.so+0x4488a48b)
 #11 0x4488a59f in IPCConnection__Create (/lib/libtvs-api-ipc.so+0x4488a59f)
 #12 0x445282c1 in TVServiceAPIConnection::TVServiceAPIConnection(std::_cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::_cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, bool) (/lib/libtvs-api.so+0x445282c1)
 #13 0x444e3f5f in TVServiceAPI::CreateSOSpecApp(EProfile, int, ISOSpecApp**) (/lib/libtvs-api.so+0x444e3f5f)
 #14 0xb6090889 in NAGSAM_MW::CSystemInfo::CSystemInfo() (/lib/libnagsam-mw.so+0x301889)
 #15 0xb6097053 in NAGSAM_MW::ISystemInfo::Instance() (/lib/libnagsam-mw.so+0x308053)
 #16 0xb60c34ed in NAGSAM_MW::TizenSupportManager::TizenSupportManager() (/lib/libnagsam-mw.so+0x3344ed)
 #17 0xb60c4c05 in NAGSAM_MW::ISupportManager::Instance() (/lib/libnagsam-mw.so+0x335c05)
 #18 0xb60c4c53 in NAGSAM_MW::MainEntryPoint() (/lib/libnagsam-mw.so+0x335c53)
 #19 0x111f3 in main (/usr/bin/nagsam-daemon+0x111f3)
 #20 0x41096867 in __libc_start_main (/lib/libc.so.6+0x41096867)

 Address 0xbe4a5720 is located in stack of thread T0 at offset 1184 in frame
 #0 0xb581c35f in dbuspolicy1_init (/lib/libdbuspolicy1.so.1+0x835f)

 This frame has 4 object(s):
 [32, 64) 'cmd_free'
 [96, 106) 'user_suffix'
 [160, 1184) 'buf'
 [1216, 5312) 'resolved_path' <== Memory access at offset 1184 partially underflows this variable
 HINT: this may be a false positive if your program uses some custom stack unwind mechanism or swapcontext
 (longjmp and C++ exceptions are supported)

 SUMMARY: AddressSanitizer: stack-buffer-overflow (/usr/lib/libasan.so+0x6f533) in printf_common(void*, char const*, std::__va_list)
 Shadow bytes around the buggy address:
 0x37c94a90: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
 0x37c94aa0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
 0x37c94ab0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
 0x37c94ac0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
 0x37c94ad0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
 =>0x37c94ae0: 00 00 00 00[f2]f2 f2 f2 00 00 00 00 00 00 00 00
 0x37c94af0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
 0x37c94b00: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
 0x37c94b10: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
 0x37c94b20: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
 0x37c94b30: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
 Shadow byte legend (one shadow byte represents 8 application bytes):
 Addressable: 00
 Partially addressable: 01 02 03 04 05 06 07
 Heap left redzone: fa
 Freed heap region: fd
 Stack left redzone: f1
 Stack mid redzone: f2
 Stack right redzone: f3
 Stack after return: f5
 Stack use after scope: f8
 Global redzone: f9
 Global init order: f6
 Poisoned by user: f7
 Container overflow: fc
 Array cookie: ac
 Intra object redzone: bb
 ASan internal: fe
 Left alloca redzone: ca
 Right alloca redzone: cb

Command: /usr/bin/nagsam-daemon

#################################################################

No such issues were reported after NULL terminating the buffer.

Change-Id: I7afccaab7448119621fdd36bd5a354da96eb9a4a
Signed-off-by: Kushagra K <kushagra.k@samsung.com>

Remove kdbus interface header

kdbus.h is now provided with other linux kernel headers.

Change-Id: Ida7d06aa1f27d88040f949ddfd73f0d6cfd5f23e

Secure get_test_id against buffer overflows

So far it's always called safely so this is mostly
for future-proofing and silencing static analysis.

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

fix svace issues

1. memory leak in dbus_daemon.c
2. Return value of getgrouplist overwritten without checking

Change-Id: I498d2ea22349bace86f3a2a5fc7adec9fd8d5b77

Add .gitignore

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

Add %license LICENSE.APACHE2.0 in the spec file

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

Fix a file descriptor leak.

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

Refactor groups proxy

Fixes container misuse. Happens to remove a false positive
SVACE static analysis warning in the process.

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

remove unused variable and redundant comparison

Change-Id: I9dff1a2de258dad5d08d4d99ee360bc2ee5c3de4

fix string usage

Change-Id: I768140827bf4928cc0a1220f29f35b0a1bb39b62

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>