doc: overview, purpose, design, implementation

The description of the libdbuspolicy project. It includes
the overview, the purpose, the description of the design and
the implementation, and additionally notes for improvements
made or considered.

Change-Id: I6b0bc9028ed86cb688d0dac9e910c10d3c4d8bc5

Fix some issues found by static analysis.

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

Fix DetachedBuffer self move assignment

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

svace fix:add catch - std::out_of_range

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

svace fix

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

src: fix non-linking with some compilers/linkers

We got a report that the library does not link within some
environments, giving messages like:

 undefined reference to `BusPathResolver::KDBUS_SYSTEM_BUS_PATH'

This is due to missing definition of static constexpr fields.
It is not necessary to give such definition when a field is initialized
inline since C++17, but this is compiled with C++11.

This commit adds definition of static constexpr fields.

Change-Id: Ia354f0e1aa5f1051571bf328fc654627f69c5dbd

printer: fix 'v' argument

':' in getopt list means that the option requires an argument.
'v' option does not require argument.

Change-Id: I63052b80e8ed9cc6df93524b7052f8f1c77a8773

coverity fix

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

Merge "dbuspolicy-printer: add xml format" into tizen

Merge "dbuspolicy-printer: fix segfault due to invalid input" into tizen

dbuspolicy-printer: add xml format

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

dbuspolicy-printer: fix segfault due to invalid input

-i opt expecting serialized file.

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

Migrate to openssl 1.1

Change-Id: I07c62e6cf0e0fd9173a5d92b40b4b34756f05861

change 'constructor with 1 argument' to explicit constructor

* implicit conversion
The compiler is allowed to make one implicit conversion to resolve the parameters to a function.
What this means is that the compiler can use constructors callable with a single parameter to convert from one type to another in order to get the right type for a parameter.

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

Fix a dangling pointer / needless copy

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

Add a missing whitespace

"allowown" -> "allow own"

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

internal: add send index

This adds indexes:
- main index for send rules;
- additional index for send prefix rules;
- indexes for send rules for each user;
- additional indexes for send prefix rules for each user.

Change-Id: I45a4b3625fe67a9fff7525d04ae6ebf8879e14f1

refactoring: remove unnecessary includes

Change-Id: I07f76694014d24d89964b95d834d10b4b3f8b38b

new printing pattern

Change-Id: Ifb3f31c44ba845a35490f26d63e74f05cb8c6658

dbuspolicy-finder documentation

Change-Id: Ia6617d124a4e4da8a1377eefba2b70b971b65c23

added policy rejected message as input

Change-Id: I15f921221f02d9d8d72097d93aa54f92f91895c0

added dbuspolicy-finder utility

It shows filters policies and possible privilege combinations.

Change-Id: Ia4078eb76a7fde3fc457280a08189208089464d2

fix maybe-uninitialized error in serializer.cpp

Function Serializer::serialize(const std::string config_path, size_t &size) can
return nullptr in case of error, thus leaving size to be uninitialized.

[   50s] src/internal/serializer.cpp: In member function 'serialize':
[   50s] src/internal/serializer.cpp:133:50: error: 'size' may be used uninitialized in this function [-Werror=maybe-uninitialized]
[   50s]   output.write(reinterpret_cast<char *>(buf), size);
[   50s]                                                   ^
[   50s] src/internal/serializer.cpp:130:9: note: 'size' was declared here
[   50s]   size_t size

Change-Id: If8b2c735e6b01e73b592827b937e0dfe8f94a615
Signed-off-by: Himanshu Maithani <himanshu.m@samsung.com>

Change the config value of the "RemainAfterExit=" ("true" -> "yes")

Change-Id: I969b67e81bb7db736d1db7a566cb361bbb281a1a

bugfix: LOGE messed with errno

Change-Id: I626c8e4e809791e5a0050a3096b392336f85cc60

refactoring: make less dependencies

Change-Id: I8ec4031716f0a089e62ca0cae6fe1bfc3f5e9529

tests: add 'worst case' performance test

Change-Id: I4d14eeb5bea1e7054674793ebd8d0861c876eb59

tests: add missing deny for build-time tests

Previous commit somehow didn't get additional deny, which
is needed for passing build-time tests.

Change-Id: Id29a11a14e6f2b586d1cab58fd8b27ef576cf713

Merge "extracted helper functions to a separate header" into tizen

Merge "Makefile now conforms to libtool naming standard" into tizen

extracted helper functions to a separate header

Change-Id: I777b45cf0166611e215bf38c275b62db777dd9dd

tests: make denying method calls narrower

After commit 329ae0e41 the cynara test stopped working. It was due to
different order of reading configuration files, as method.tests.conf
denied all the method calls.

This makes method.tests.conf deny only method calls to org.test.*, which
covers all destinations defined in the file.

Change-Id: I9a181bb184d41af83585c000b458fb9edd78cfb9

internal bugfix: handle no-interface messages

No-interface messages should be handled in a special way.
Dbus says in bus/policy.c:

 The interface is optional in messages. For allow rules, if the message
 has no interface we want to skip the rule (and thus not allow);
 for deny rules, if the message has no interface we want to use the
 rule (and thus deny).

For example, a rule of type:
 <deny send_type="method_call" send_interface="foo.bar"/>
should:
- deny method_calls with foo.bar interface;
- deny method_calls with no interface.
A rule of type:
 <allow send_type="method_call" send_interface="foo.bar"/>
should:
- allow method_calls with foo.bar interface;
- not allow method_calls with not interface (the rule should be ignored).

The point is to make libdbuspolicy work as close as possible to how
dbus-daemon works with policies.

Change-Id: I99563d3728047a67fa4719948719a2df9c5d9f97

Makefile now conforms to libtool naming standard

Change-Id: I61f5d06ca5aeb5aa63875cc087c438b6b93ecc9a

add missing PolicyOwn::ItemType

Change-Id: I04fc8c0a51d3ff717834f9985acd883800be074e

serializer: define order of including files in includedir

In the reference documentation <includedir> has an undefined
order of including files from a directory.

This makes the order unoficially and implicitly defined to make
generated serialized files equal for the same set of files.
It does not mean that users may now rely on it.

The order is alphabetical.

Change-Id: I07fd137e6a9646a541387bb0ef205edfc012f5f9

serialization: file version verification

This adds flatbuffers file identifier and uses it during
verification.

Change-Id: I2d49607d8c5ca0b9678e9bc68a9dc6201ef4c7b5

serializer: add SHA1 and FlatBuffers verification

Change-Id: I8c78971f8b6481d1cd00c54dce7b2f0466f1ce61

tizen: Add new package to perform data verification on boot

emergency.target is started on verification failure.

Change-Id: I5c67e624d76440f9667602930aecb5adff233ecc

serialization: change tool name

For readability and easy typing

dbuspolicyserializer > dbuspolicy-serializer
dbuspolicyprinter > dbuspolicy-printer

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

Merge "serializer: add option for checking for updates" into tizen

serialization: make input consistency between printer and serializer

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

serialization: printer: change log level from silent to default

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

xml_parser: modified log to print out filename

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

tslog: change name log to log_debug

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

tslog: modified to support log level

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

serializer: add option for checking for updates

Developers may need to check if they already updated their
serialized policy file. This adds -j option which does not write
anything to the output file, but rather just checks if the generated
file would have the same contents as an existing output file.

Change-Id: I28ab3e992416d25c04206279275f007c0ca08919

refactoring: group helper functions into a namespace

This groups helper functions into an anonymous namespace.

Change-Id: I1bf32e719f4268886014757a1423ef2c24b4c242

refactoring: group path resolving ops into a class

This introduces BusPathResolver class.

Change-Id: I56d22e0b6b5e565415b1ff6d070986efa97c7502

refactoring: group kconn ops into a class

This moves functions dealing with kconn into the kconn class.

Change-Id: I27422ae661071ec33d27bced8b2954ea744d16d0

refactoring: group checker functions into a class

This introduces local Checker class that helps with initializing
and selecting NaivePolicyChecker instances.

Change-Id: Ia859e63903b4ea173fe4cae5de9cfadc1aa94d4a

refactoring: group g_udesc operations into a struct

Change-Id: I474a61e811cb7eb316ad8b57c298b8730f74eb25

refactoring: getNames() -> prepareNames()

prepareNames() is more proper name for the function as it
prepares an array of names in the KdbusConnectionInfo object,
it does not return them.

Change-Id: I9f86ae80c138a2371456c86543757e962d23e25a

refactoring: use checker directly instead BusType in configuration

This makes configuration use pointer to checker in place of previously
used BusType. This way on queries, the object is used throught the pointer
instead of selecting correct checker with yet another 'if' clause.

Change-Id: I79f129be899ff501a01ff1fbf1a95c5d194e3abf

refactoring: make BusType a local type

BusType does not need to be a global type, because in fact
it is needed only to choose a correct policy checker.

In the test cases, or in the command line tools we explicitly know
which checker we need to choose.

Thus, it is only required for the API, where we still keep it
in the configuration.

This will allow further simplifications.

Change-Id: I61663d496279b47a6a0338b93434b5ae42326f33

refactoring: remove the remains of internal.cpp

This removes the remains of internal.cpp by replacing all __internal_init
functions with more direct calls.

This allows removing of internal.cpp.

Change-Id: I064d919d06a25888cae6079bf9d37cd535f9b8b6

refactoring: remove tslog-related functions from "internal"

This removes tslog-related functions from internal.{cpp,h}.
Needed functionality is moved to tslog, the rest is eliminated,
as obsolete.

Additionally, RAII object LogLock is provided to make it look
prettier in places where transaction_guard for __internal_enter and
__internal_exit was used.

Change-Id: I48e5b9c5bf352eb03c50b2a4b4c109c2df0c11bc

refactoring: eliminate __internal_init_sup_group()

This removes __internal_init_sup_group(), which no longer serves
as anything more than a useless proxy.

Change-Id: Ie460c9f2b71413235e7141ec84b5112be27e0f5b

refactoring: clean up tests (a bit)

This uses more understandable expected results of checking, like ALLOW or DENY
instead of true or false.

Aimed to increase readability.

Change-Id: I48be2de6005812778463be537ea3fa475256f91f

refactoring: eliminate __internal_can_* functions

This removes __internal_can_* functions as they do not really
do anything useful anymore...

And, as always, it allows further simplifications.

Change-Id: Id30db12afbf8d60dce87c23355c41cff0d907d05

refactoring: don't copy string_refs for matching

This modifies MatchItemSR to use externally built array of names instead of
copying input arrays. The external array was built anyway in case
names were taken from KDBUS_CMD_CONN_INFO ioctl.

The other case, where the array is built from space separated string is
changed in a way that the building process is performed outside of
MatchItemSR. This allows further simplifications.

Change-Id: I13cfba0940f3347be91c9281614a3c66b8cb11b6

refactoring: reorder includes

Makes it easier to spot duplicated includes

Change-Id: I6f0ef0cd5bf558500020ee715e301f708fffb04d

introduce KdbusConnection and KdbusConnectionInfo classes

Change-Id: Ie9b80624f6f6d09e08338ed68e23657f5dc33480

refactoring: simplify getting conf file names

Change-Id: Ib452c4c11480ab388c69296d6e220ac725e83ee0

refactoring: extract function getting bus owner from path

Change-Id: I967484c5149f8eda1220e4e2045deb54ad84a4ab

kdbus: free cmd earlier while doing hello()

Change-Id: I1b22db1d9296277de188d9e21e790390a24e7627

refactoring: remove 'struct's

Change-Id: I38349fb759a1a23873892d80642698580c7bbbfb

refactoring: generalize and use setting string item

Change-Id: Iabedbee561b73df704f97392ae064cfcc97806a5

refactoring: manage kdbus commands

This introduces helper class kdbus_command which makes it easy
to allocate stack space for kdbus commands, including space needed
for optional items.

Change-Id: If4b9ab6cbc92f4705b21373af3d94f9d88a7fd6a

refactoring: merge 'can' with 'can_multi' functions

Change-Id: I17889df47ac3a5f740eb528f7f56c3e3b8b7edc5

refactoring: replace bool return type with RAII

This changes return type of bus path resolving to shared_ptr<const char> from bool.
This is sufficient to carry all the needed information, and allows
removing of explicit free()s and gotos.

Change-Id: Ic05a734aea49439182fca4ea912086213a6cf91a

refactoring: use string_ref for extracting bus_owner

Change-Id: I4009d4bf06ba09c3e8177c8ffbb0dec6a1d05000

refactoring: remove not needed types

Change-Id: Iff52dbdab0e4b4b45c52d971beb0934cb22044a7

use call_once for initialization

Change-Id: I5dae342a7ba7083903d1fa30a4440700629b6945

API: documentation update

Change-Id: I002b6ce0869677f3a522162d8a68bda536ddbc9e

API: mark dbuspolicy1_init as deprecated

Change-Id: I6ca78933941a57018130bcc67732ff07550fa0d6

cpp-ify libdbuspolicy1.c

Change-Id: I6f0ef0cb5bf558500020ee715e301f708fffb04d

extract kdbus operations to separate file

Change-Id: Ia33922128de417e5457a0daf5f9757a6f130cfa3

simplify string duplication

Change-Id: Ie228964fcabf0f3fea8225e80ed2cdf82d12119d

kdbus: simplify handling names

Change-Id: I4330f299a7bcf1b2325205ac2dd6b54526a2b8c1

simplify freeing kdbus structs

Change-Id: I483b64423c5beddb62254aebb9d8f3e4ea987c35

serialization: printer: check initialization status

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

serialization: check max size of serialized file

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

Add libdbuspolicy-serializer as requirement

We need dbuspolicyserializer to serialize D-Bus policy configuration
during building the image

Change-Id: Id6ab6331930404553b1306968b85e5a66a289cad

svace fix

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

internal: use strerror_r instead of strerror

strerror() is thread-unsafe. This replaces usage of strerror()
with usage of strerror_r().

Change-Id: I80d1888c1ec24584c57c68a7b2531084f616342f

serializer: set output name to .serialized

By default libdbuspolicy may expect serialized policy file
in default location, with XML policy file name modified
by adding .serialized suffix.

This changes serializer default behavior to create output files
with the same naming rules instead of using result.dat.

Of course, if a user specifies its own specific output file name,
the serializer still creates output file with the given name.

Change-Id: I5ca0699027013e23fe2d83c5abd80bdebc165b1c

svace: add noexcept to MatchItemSR::addNames

SVACE complains about non-catched out_of_range exception from
boost::string_ref::substr() called by MatchItemSR::addNames().
The code is written in such a way that no exceptions should be trown.
To indicate this, and possibly make SVACE happy, this adds
'noexcept' keyword to addNames().

Change-Id: I5b31cfd201af2267e08587118d2dd54da225d945

Add license headers

Change-Id: I90a104cb4452a7f6e616e90f84bf417d7e05fe6e

spec: enable parallel building

Change-Id: Ia2d1ea4534bf76aab5ba6cd116d79657d7117165

add support for sharing fd and pool with clients

This adds two new API functions, which can be used for sharing
kdbus fd and kdbus pool with the library clients.

Until now, the library created its own connection each time
it was initialized, effectively allocating 1MB of memory
only for getting small responses to one of kdbus ioctls.

This kind of activity may be performed on any usable kdbus
connection. Thus, the idea of sharing existing connections
with libdbuspolicy.

As it is a change of interface (two new API functions), the version
of library and package has been bumped up.

Change-Id: I7bae7a60a386f1c3bc35c88528c9f2baa2c57e34

refactoring: change logError parameter type

Change-Id: I4fcaf121159d9a503a44e69d281bfd1147b8c5d1

Check if config_name is NULL or empty

config_name must be provided and case in which it is empty or is NULL is
a programmer error

Change-Id: I21274c8331edd118d672802f06f7bec7ac8ddde8

refactoring: simplify StorageBackendXMLImpl

This simplifies StorageBackendXMLImpl by putting members
into a tuple, and using type_lists to select correct
elements of tuple without any need for type helpers.

Change-Id: Ia93fb9bab03613cebade82f1daa29181b7f178ee

tests: add -v (verify) command line parameter

Change-Id: Iaf03cab80eecc12839610121786abfb3a1d102c9

Add warning when serialized file is missing

Change-Id: I604b94cad042a5e65275705f7ca6a88b9fbaa9fb

Add design documentation

Change-Id: I123ed895c624fe712d4bd3f6da8f72c820e3a87f

refactoring: change string to ref-to-string in own tree

This changes std::string in construction of TreeNode to reference-to-string
to avoid additional copy of the string.

Change-Id: I49ac9d8e83f8691c022c9113d427bc60c7463902

refactoring: remove unused code

This removes code which is unused after switching backend
from StorageBackendXML to StorageBackendSerialized.

Change-Id: I2326b24f06c7eae050e190e15d8f1c4740a4d29e

refactoring: remove unused parameters and "using namespace std"

Change-Id: I886c7dfd075fce52bfa5a8d1e7bf6d9aeb301ec9