Fix GCC 13 incompatibilities

Change-Id: I4a1f4098a3fadba81c9c44801f1ec80dc8ce2c41

packaging: use linux-tizen-modules-headers

Change-Id: I07e4534ca1e152201e97be447a608f88f6f58352

tests: add generated performance tests

Enhancing performance test by incorporating VD Tizen
image specific Item Send rules in default context to
performance test at runtime.

Verification Results:

sh-3.2# ./stest_performance --system -r 10
SERIALIZED:
Default ItemSend Rule Count: 578
run: 0.001137

Change-Id: I7050d19a3ef50119254e417a2ef05b5bf4ecd911

Fix build errors - kdbus.h

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

kdbus: add kernel linux/kdbus.h header

Apparently, linux headers package may be not a good place
to keep linux kdbus header, as we can't rely on the
header being there when packages that optionally use it
are built (e.g. systemd, dbus, glib and libdbuspolicy).

However, this package, libdbuspolicy, is very tightly coupled
with kdbus so far. In fact, it is so tightly coupled
that maybe in the future the kdbus bindings will be moved here,
and the package will be renamed to libkdbus? Unlikely,
there are not many people out there that would like to touch it
anymore.

Anyway, anyone who wants to use kdbus on Tizen, should also use
libdbuspolicy. That's why the kdbus.h header goes here, and nowhere else.

Use it with libdbuspolicy or don't use at all.

'devel' package is the one for distribution of kdbus.h. Additionally,
kdbus.h has LGPL-2.1 license, so the 'devel' package also gets it.

Change-Id: Ie632e7ecff81d54044f2721138fc9656f3af8412

stest_performance: add some error checking

Change-Id: I95f8ec6be9597d05d5ebb3a847f241d5cb9e8dc9

Merge "tests: allow reading from file in perf test" into tizen

refactoring: move getter out of loop and lambda

Change-Id: Iab7e249849a2ae3f61642b4ead9a973071287401

tests: allow reading from file in perf test

Change-Id: I1660ed2a0248efe139590ac278fbf29d4db2d44c

serialized: don't copy strings in policy lookups

Direct serialization allows more optimizing. Now, that other
backends are gone, we can slightly modify the interface.

This commit makes send index searching pass boost::string_ref
to the backend instead of copied C-string. The backend interface
is modified to accept string_ref. This way we can remove
string copying which was necessary for having null-terminated C-string.

Change-Id: I5a1f9657efd050d93023b5256f13afd6d150c369

fix: add missing '[]' in declaration of SerializerDirect::_data

Change-Id: I161ff679ad19f74d8815554958f8fa0755ac4445

refactoring: inline StorageBackendSerialized private methods

Change-Id: I0483dceef8afa5ec4d160e6a1a785142490fb01a

Release 1.2.0

Replacing flatbuffers with own serializer really deserves separate
version.

Change-Id: Ibbcd300c29f3cbc30f6dcede7a346c421f1f31c4

serialization: add simple file verification method

Change-Id: I95d3501bcfffa0e4601ee2c3738b38c41b366a44

license: eliminate dual-licensing in favor of MIT

Change-Id: If1cddafdf0f25c7e42120d0ddec1897bf079f4b3

serialization: remove flatbuffers usage

Change-Id: Ic334d20bf00cd50854af767efa9c9e0bce6111b8

serialization: add direct backend

Change-Id: I5c9345310f58fa62c7fec3d8a1ccc56978e4c53a

printing: print send index

Change-Id: Ib01e7eced91c1cf91c147361107946162b6ac2d5

tests: switch to the new extracting interface

Change-Id: I2167e014c951099ff3ac56af7d2e3f64f5ee102f

refactoring: create indexes in StorageBackendXML

This moves creating of indexes from serializers to StorageBackendXML.
This way it's standardized, in a single place.
Serializers now serialize the indexes from the StorageBackendXML instead
of creating their own.

Change-Id: Ie2aec8a8cbace425f2e74c402cb99c25f21a27d1

refactoring: move generalized operations to StorageBackendSerialized

Change-Id: I9abb90ff1d81970d939ca3f80b784c9b7b60ab23

refactoring: generalize operations

Change-Id: I6af4d846397b73dc3dfaf45ccf5ebb0fd1ceef46

serialization: use new interface for operations

Change-Id: Idfd52382dd4b3acd4d158d7604468e9fcc297a9b

refactoring: move makeDecisionItem, and use new interface

Change-Id: Ibb64103c318ff1cf404dd23bdf837c28da06a120

printing: use generic data extracting interface

It eliminates any konwledge of Flatbuffers structures, types, and methods
of data extracting from the printing module.

Change-Id: I5da411f3878b8cb13e9962a7f6c994063446304a

refactoring: move FB_ID_* constants

Change-Id: Ia5a8b74612361cdbc384f251ec45c4d02a7f4032

printing: add generic data extraction interface for XML db

Change-Id: Icf05d2d7a8d5077b9efa23aee7c99af8b9264242

refactoring: move local functions to the class

Change-Id: I98e3f0ab900be2b487df47f06d131299507cd7a3

serialization: add new Storage extracting interface

Change-Id: I947a1c73e0396034ebd0038fb483f5208a43bba3

refactoring: eliminate StorageBackendFlatbuffersImpl

Change-Id: I73f2a898483fb336518f8b9ef28338ef91e55796

refactoring: decouple StorageBackendSerialized and Serializer

Change-Id: Ib4fbd8db4a2585694d50bfce33432b170c2cdb96

refactoring: print 'serialized' instead of 'flatbuffers'

Change-Id: I262f6345277133bb1c0ea16d270a38bfa066695b

serialization: add SerializationBackend type

Add SerializationBackend type, which is a customization point
for choosing serialization backend. At the moment it is set
to Flatbuffers. It is the place to select another backend, when it shows up.

Change-Id: I2a54f643d5c5bda065fcd193c51ca928f20df203

refactoring: extract common file operations to StorageBackendSerialized

Change-Id: I0a3fd26342482d747f44aae08ece46bb2b40a4fc

refactoring: extract common file operations to Serializer

Change-Id: I87e8dd9daa91a35c2748001fba71178d608d2024

license: add license boilerplate where it's missing

Change-Id: I60be356b594cde13e7b910a333ec7b48c6409fd1

refactoring: remove unused enum

Change-Id: I0ebeb4c599bc05ebe9c92263fd75ccaf8cfd9486

refactoring: rename StorageBackendSerialized to StorageBackendFlatbuffers

Change-Id: I1d6a8e089b31730e7a8e2f54ea0a3b2fe8548401

refactoring: generalize serialized enums conversion

Change-Id: Ia1fe8089c03605fe329bc3722bf7d99fe2117b21

refactoring: rename Serializer to SerializerFlatbuffers

Change-Id: I52d6c22f44d7730dea221c4b9d81df9b7b3af291

refactoring: add size argument to initFromData()

Change-Id: I0fea35816cbbb7dc898e495adab73d9c28729bc3

refactoring: make serialized data const

Change-Id: I1e989504323c18754c9117d903ba8f4b951d8377

refactoring: move Serializer to ldp_serializer namespace

Change-Id: I75fb678335aee17bcf6c13792f5124da4d0ad422

refactoring: hide FB::File from interfaces

Hide FB::File from interface of StorageBackendSerialized. This
generalizes the StorageBackendSerialized interface. And that allows
other implementations.

Change-Id: I84631685e6ead50b33e7254c73dfb9af18bb9c4d

own_tree: make ordered map, we need it sorted anyway

Change-Id: Iaac12479580b1502bc459387b64b7dda12ce9bef

finder: improve printing

Add actual attribute names to printing instead of printing just "send"
or "receive".

Change-Id: Ib117847d44a57abbf1fd1217a786c1d2d8b5d218

xml-parser: don't clear list of included files in each dir

The 'includedir' directives should make the parser include all
the files in the directories introduced with the directives.

This commit removes clearing of the gathered file list when
the directive is encountered.

Change-Id: Id14be322e8696bd85ddfbb0ba2360a7b4b5bcda4

tests: test multiple includedirs directive

This adds a test which checks if the files in all the directories
specified in all the <includedir> directives are parsed.

It breaks the checks during building as it exposes a bug.
The subsequent commit fixes the bug.

Change-Id: I020246138586357717dbee73617182f79176eac9

Fix memleak issue reported by Dexter tool.

Change-Id: I24e2d2d6b6ecb3e3daf0edd94fe67866f369e87a

coverity: Fix coverity issue

constexpr function should return LiteralType

Coverity: #1050604 Parse warning
constexpr_return_not_constant

Change-Id: I608c84748dc515778bf96c6b65e59f1e389359c3

Remove warning for GCC-9

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

Merge "DF191023-00350:[Secure option] file secure option application request (PIE, etc)" into tizen

DF191023-00350:[Secure option] file secure option application request (PIE, etc)

Change-Id: I65bbebeb5b6f8399dce4bc622c7a59b5963540ce
Reviewed-by: Himanshu Maithani <himanshu.m@samsung.com>
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

Merge "printer: print FileIdentifier" into tizen_5.5

Merge "serialization: serialize send index" into tizen_5.5

Merge "refactoring: move local functions to an anonymous namespace" into tizen_5.5

Merge "doc: overview, purpose, design, implementation" into tizen_5.5

Merge "Fix some issues found by static analysis." into tizen_5.5

printer: print FileIdentifier

Change-Id: Ia993cd3739f74920e3dedceeb4a58de1e967b326
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>
(cherry picked from commit 87cae30fd5842acc83ababd378873b3182c11f02)

serialization: serialize send index

This removes creating send index after reading a serialized database.
Instead, the send index is created while serializing the database,
and written along with the database. Then, the index is used
as before, but in a bit more generalized way, being a part
of PolicySend now.

Therefore, this extends the database with the new tables,
making the database version up to LDP2. The previous version
LDP1 is still compatible with new code, however the send index
is not supported for old databases.

This results in the following:
- with serialized database, version LDP2, the send index
  is kept in serialized database file, and used accordingly
  for all lookups to "send" part of the database;
- with serialized database, version LDP1, there is no send index,
  the lookups are performed in an old-fashined, linear manner;
- with no serialized database, only XML files, the index is created
  in memory, along with the serialized database. Thus, running
  fast, but at the cost of memory.

Change-Id: Ib918fa96eb60d216512faf60f8e20692c8a5d3a3
(cherry picked from commit 783b7e7315959b625a613b959c19b12a381e5831)

refactoring: move local functions to an anonymous namespace

Change-Id: I9d9cf07877d59df88305e8847c3156a73e82efd8
(cherry picked from commit c641a5cb8a025f7a75f7b88015d85e1a5cc6813a)

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
(cherry picked from commit 92c61f4ae8c64d2be5adf809c1ba10a1cdb5e905)

Fix some issues found by static analysis.

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

Fix DetachedBuffer self move assignment

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

printer: print FileIdentifier

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

serialization: serialize send index

This removes creating send index after reading a serialized database.
Instead, the send index is created while serializing the database,
and written along with the database. Then, the index is used
as before, but in a bit more generalized way, being a part
of PolicySend now.

Therefore, this extends the database with the new tables,
making the database version up to LDP2. The previous version
LDP1 is still compatible with new code, however the send index
is not supported for old databases.

This results in the following:
- with serialized database, version LDP2, the send index
  is kept in serialized database file, and used accordingly
  for all lookups to "send" part of the database;
- with serialized database, version LDP1, there is no send index,
  the lookups are performed in an old-fashined, linear manner;
- with no serialized database, only XML files, the index is created
  in memory, along with the serialized database. Thus, running
  fast, but at the cost of memory.

Change-Id: Ib918fa96eb60d216512faf60f8e20692c8a5d3a3

refactoring: move local functions to an anonymous namespace

Change-Id: I9d9cf07877d59df88305e8847c3156a73e82efd8

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