Release 0.18.1

* Change release mode log level to warning

Change-Id: I3fd4087d40cf0ecdf14005fda7cf61c0975367fd

Change release mode log level to warning

Debug logs are clearly only for debug mode, but warnings
can be helpful in release mode (VD request).

Change-Id: I86b390fe2838aec3b793ca4bf9b274554a8e6d78

Release 0.18.0

* [helpers] Fix linking with creds-commons
* Add dbus helper API to fetch all credentials in one call
* Add gdbus helper API to fetch all credentials in one call
* Change some logs into warnings

Change-Id: Ic5f161e7410e8f93b607d7bb758d17ba322f5a4b

[helpers] Fix linking with creds-commons

As TARGET_LIB_CREDS_COMMONS wasn't defined on level visible
for credential helpers build, credential libraries were not
properly linked.

Change-Id: I340cb0a312a3f98e9bf11957423a57fc40a35b65

Add dbus helper API to fetch all credentials in one call

Change-Id: I17653b94fe79065354bdb82c52c378d9126dab15

Add gdbus helper API to fetch all credentials in one call

Change-Id: Ie3a0446b60f837bdec42c80323fa6809c7cdddc9

Change some logs into warnings

As requested explicitly by Visual Display Division.

Change-Id: I54d18b426d0fc1f65d92352036e660ed6310736d

Release 0.17.2

* Add additional logs
* Implement better tryCatch
* Put some .cpp-local entities in anon ns
* Remove unused sec_build_binary_debug_enable build macro

Change-Id: I40820f05f1ddbe5a9a7455c2796e5dbd64b56b35

Add additional logs

All done for explicit request of Visual Display division.

Change-Id: I7b4f6a487d1dddf5288c3ab5758249bd3e3c6b5e

Implement better tryCatch

     "Gotta catch them all!"
- from Pokémon

Recently it was reported that there's an issue if not-handled
std::bad_alloc in Cynara. After getting more information
(incl. decompiled version of the module), it was found that
when compiled using gcc, implementation of std::function made on-the fly
as argument to tryCatch() is being allocated BEFORE passing it to
the tryCatch() function.

When compiled using clang9, there is no new() in that place.

This patch changes tryCatch() to a template with function being
an R-value reference with std::forward, solving the issue.

Decompiled version with additional new
(thanks to Konstiantyn Melnik, k.melnik@samsung.com):

int __cdecl cynara_check(void *cynara_struct, char *client, char *client_session, char *user, char *privilege)
{
  int v5; // ebx
  char ***lambda_func_obj; // eax
  char *__loc_privilege_ptr; // [esp+1Ch] [ebp-4Ch]
  char *__loc_user_str; // [esp+20h] [ebp-48h]
  char *__loc_client_session_str; // [esp+24h] [ebp-44h]
  char *__loc_client_str; // [esp+28h] [ebp-40h]
  void *__loc_cynara_struct; // [esp+2Ch] [ebp-3Ch]
  char ***lambda_func_ptr; // [esp+3Ch] [ebp-2Ch]
  int (__cdecl *v14)(void **, void **, unsigned int); // [esp+44h] [ebp-24h]
  int (__cdecl *func_to_run)(void *); // [esp+48h] [ebp-20h]
  unsigned int guard_var; // [esp+4Ch] [ebp-1Ch]

  __loc_privilege_ptr = privilege;
  __loc_cynara_struct = cynara_struct;
  __loc_user_str = user;
  __loc_client_str = client;
  __loc_client_session_str = client_session;
  guard_var = __readgsdword(0x14u);
  if ( cynara_struct
    && *(_DWORD *)cynara_struct                 // if (!p_cynara || !p_cynara->impl)
                                                //         return CYNARA_API_INVALID_PARAM;
    && client                                   //     if (!isStringValid(client) || !isStringValid(client_session))
                                                //         return CYNARA_API_INVALID_PARAM;
    && strlen(client) <= 0x1000
    && client_session
    && strlen(client_session) <= 0x1000
    && user
    && strlen(user) <= 0x1000                   //     if (!isStringValid(user) || !isStringValid(privilege))
                                                //         return CYNARA_API_INVALID_PARAM;
    && privilege
    && strlen(privilege) <= 0x1000 )
  {
    lambda_func_obj = (char ***)operator new(0x14u);
    lambda_func_ptr = lambda_func_obj;
    *lambda_func_obj = &__loc_client_str;
    lambda_func_obj[1] = &__loc_client_session_str;
    lambda_func_obj[2] = &__loc_user_str;
    lambda_func_obj[3] = &__loc_privilege_ptr;
    lambda_func_obj[4] = (char **)&__loc_cynara_struct;
    func_to_run = (int (__cdecl *)(void *))sub_4B80;
    v14 = sub_3930;
    v5 = Cynara::tryCatch(&lambda_func_ptr);
    if ( v14 )
      v14((void **)&lambda_func_ptr, (void **)&lambda_func_ptr, 3u);
  }
  else
  {
    v5 = -4;
  }
  if ( __readgsdword(0x14u) != guard_var )
    terminate_proc();
  return v5;
}

Change-Id: I5455d03d411b03ed76a81659efb60c6474ceb99b

Put some .cpp-local entities in anon ns

This fixes at least one ODR violtation (FakeStreamForBucketId) which was
defined twice, in test/storage/serializer/{deserialize,serializer}.cpp.
Two unit tests would fail in some environments as a result, particularly
StorageSerializerFixture.dumpBucketsPositive, due to dynamic_cast
failing within gtest (the one from serializer.cpp was being cast to the
one from deserializer.cpp).

Change-Id: Ie0dc0191b4e45d032b68b0df9bfcdf81fca8678f

Remove unused sec_build_binary_debug_enable build macro

Change-Id: I3d8c8e4c936449ff36b7288580d90383ce64a2fe

Release 0.17.1

* Fix UB in predicate-driven PolicyBucket erasure

Change-Id: Iee7c125a927905e68e3477c0126b989951f5366c

Fix UB in predicate-driven PolicyBucket erasure

Change-Id: Ic1559c79c875e503ac165c29671460ad56deaab2

Release 0.17.0

* Automate code coverage measurement
* Add dependency warning on enabling journal logs in cynara-client
* Shrink CapacityCache keys, speed up generation
* Prevent iterator invalidation in CapacityCache
* Fix zero-initialization of class members
* Remove unused code
* Categorize existing unit tests into positive & negative cases
* Add/amend assorted unit tests to increase coverage
* Fix cyad backup command help position and message
* Redo error printing in cyad
* Remove unnecessary copies
* Force googletest to build with c++17 instead of c++11
* Move protocol serialization/deserialization to one place
* Ensure that MonitorCache capacity is of permissible size

Change-Id: I1daf23ca44ec9fb41ebb29ae1e8facd84a14ae0c

Fix debug source directory name in coverage script

Change-Id: I0789ea1b0676438e2f1be3c7e55a652074a69e4e

Fix coverage generation in rpm 4.14.1

Debug source package directories now have different names.

Change-Id: Ic84985a203189b67ce01e0db8be222756e1e2f77

Add dependency warning on enabling journal logs in cynara-client

libdbus-policy is going to be integrated with systemd on
kdbus-aware systems. This, combined with cynara-client
which can be configured to use journal (systemd) logs,
can lead to cyclic build dependency:

systemd -> libdbus-policy -> cynara -> systemd

This patch adds a warning comment in spec file where configuration
of the logging module is done and adds a warning displayed from
CMake during build to check the cyclic dependency.

Removal of journal logging was not performend as older cynara
releases used to take advantage of it and the implementation
may be still useful.

Change-Id: I474ae7d6f2724dbaf855adb2c49cea38168d85a3

Move CapacityCache::keyToString() to anonymous namespace

Change-Id: I528688316856505b734109fa5fb6e263f9752def

Shrink CapacityCache keys, speed up generation

Change-Id: Ifc8806b934e7a44a81a3e58e850755ad8baf3765

Prevent iterator invalidation in CapacityCache

If resultIt happens to point to the element that gets evicted, it will
be invalidated. Perform lookup after possible eviction.

Change-Id: I4d536c6779494f85077a247e3356374346fa35b2

Automate code coverage measurement

To gather unit tests coverage report:
- use COVERAGE build_type,
- instal cynara-coverage rpm,
- run cynara-coverage.sh script.

Change-Id: I96e113e2fff06e126ea20a1a2bb06dfdd98ae0db

Fix zero-initialization of class members

Some class members are not zero-initialized, which can cause seemingly
innocent initializations to result in uninitialized memory, which can
cause UB. While unlikely, this shows up as a critical issue in SVACE.

I have changed the constructors so that they explicitly initialize
members in order to fix the issue.

Change-Id: I16e1f253985e1c37f3b5ee67d3d9c91696706054

Remove unused files

Change-Id: I7751179a91f0b468e67ff6ff621028bfdfbf5f13

Categorize performance unit tests

Categorize into positive/negative test cases.

Change-Id: I19e49df3f26e84f368e744aae235cc3725cdeae6

Categorize EntriesQueue unit tests

Categorize into positive/negative test cases.

Change-Id: I296ea8d0e768eb86eacdfe986ae7e9764e33deb4

Categorize PolicKey unit tests

Categorize into positive/negative test cases.

Change-Id: Ic00e9a906c50f40eb92bfede740e73ef8daa0650

Categorize StorageDeserializer and StorageSerializer unit tests

Categorize into positive/negative test cases.

Change-Id: I3c1c1f295993b9fa511b7401f8929df6bb93e227

Categorize BucketDeserializer unit tests

Categorize into positive/negative test cases.

Change-Id: I6986240e763cc1ea274415b0abaaa4496be0a446

Categorize InMemoryStorageBackend unit tests

Categorize into positive/negative test cases.

Change-Id: Ie5212cef2761849dfbee1430293b7ea2f07215b0

Categorize Storage unit tests

Categorize into positive/negative test cases.

Change-Id: I33b618de130e7d496848f7213ac392279e0f854a

Categorize ChecksumValidator unit tests

Categorize into positive/negative test cases.

Change-Id: I67e876a1b6205bf64e0261278732f95e9593274b

Categorize EntriesManager unit tests

Categorize into positive/negative test cases.

Change-Id: I24f66ec022f60434399567880659b87fca53e4df

Categorize Cynara cmdline parser unit tests

Categorize into positive/negative test cases.

Change-Id: I13a7d04717131024a151b2dd8f321800927353a7

Categorize AdminPolicyParser unit tests

Categorize into positive/negative test cases.

Change-Id: Ia86512bfc7fa91ff1f941c93f4bb279cc5b5a637

Categorize CynaraAdminPolicies unit tests

Categorize into positive/negative test cases.

Change-Id: Ied6728a02db0f05d741afbee24fad0d60e14d91e

Add missing test categorization in ProtocolAdmin negative unit tests

Change-Id: I152549b18c914c5c807484caf90ee6f94b204997

Redo cyad commands dispatcher unit tests

* Minor refactorin
* Categorize into positive/negative test cases

Change-Id: Ib449c1b76863d84cda543db752d965a1b5abbf05

Categorize CmdlineOpts unit tests

Categorize into positive/negative test cases.

Change-Id: Ifa0d505ab9fb2e678f69ea5664f15a1922e5fd5c

Redo Cyad commandline unit tests

* rename positive/negative test cases
* add more negative test cases

Change-Id: I204814abcb11cfa41904ed5f3fa45223421cade5

Fix cyad backup command help position and message

Change-Id: I7c8f0de363e8d3b64922ea9d3947cbf37c04b931

Redo error printing in cyad

* remove unused error message generators
* systematize unknown option, missing option and missing argument
  errors printing for all main and sub options

Change-Id: I59368ffc0402b5f07d58616468a045c47fa3596d

Redo Credentials Config Parser unit tests

* Categorize into positive/negative test cases
* Split mixed test cases

Change-Id: I069b535d526aad4496dc6ef2befe7c96e133b8ce

Categorize string validation unit tests

Categorize into positive/negative test cases.

Change-Id: I4675e0a5c8d0d8f5682d3d54667a265cd6ac03f4

Categorize PolicyBucket unit tests

Categorize into positive/negative test cases.

Change-Id: I7f289446a33303cd1e2f1845a925176b6f294aef

Remove unnecessary copies

Change-Id: Iea05840fc9366319f17ccc98919efbbf4c350a99

Categorize PolicyBucketFixture unit tests

Categorize into positive/negative test cases.

Change-Id: Ic78e793e3d9d1ffe8e969f1f2054af2f83dc41ac

Redo unit tests for MonitorGetEntriesRequest in ProtocolMonitorGet

* squash positive test cases

Change-Id: Ida7226e6692548d181cff18f18633d10ce2f6f36

Fix: Add missing using for ::testing::DoAll

Change-Id: I2f1c9b822fba93de3f513854a071b8684e563499

Force googletest to build with c++17 instead of c++11

Newest googletest builds explicitly with c++11. This makes this
standard propagate to cynara-tests compile options and causes
a build break. This should be only a temporary fix, as cynara CMake
build files should be rewritten properly.

Change-Id: I8afa84921fb2eb2979afc6a7e7b072428bdb258f

Redo unit tests for MonitorGetEntriesResponse in ProtocolMonitorGet

* squash positive test cases
* add invalid protocol test cases

Change-Id: I2172a1a8814a924754411704b1d166df681ba92c

Redo unit tests for MonitorGetFlushRequest in ProtocolMonitorGet

* squash positive test cases

Change-Id: I848f90d1e9d565ee186f2728d37951144c297900

Add negative test case to RemoveBucketRequest unit tests

Add invalid protocol test case.

Change-Id: I06ddadc7acbd57c7f2bf52746a7770f23d0ad37f

Redo unit tests for ListResponse in ProtocolAdmin

Change-Id: Ib012cc489597013c1b1af260d1c890af17e6673e

Redo unit tests for ListRequest in ProtocolAdmin

* squash positive test cases
* add invalid protocol test cases

Change-Id: Iff443af42933c5ca8b942844b06c259a533b7627

Add negative test to InsertOrUpdateBucketRequest unit tests

Add invalid protocol test case.

Change-Id: I40928c4db573adaba9c5cc706033c37a7423f93d

Redo unit tests for EraseRequest in ProtocolAdmin

* squash positive test cases
* add invalid protocol test cases

Change-Id: I4e3d65668c3bd42ce56c8b3107b631affe7afd70

Redo unit tests for DescriptionListResponse in ProtocolAdmin

* squash positive test cases
* add invalid protocol test case

Change-Id: I798369d477bf4feefcea93e9ac54423357cf2f26

Categorize unit tests for DescriptionListRequest in ProtocolAdmin

Categorize into positive/negative test cases.

Change-Id: I0a1c0b6b9e53ff31d3000d6994cd665df0d6905c

Redo unit tests for AdminCheckResponse in ProtocolAdmin

* squash positive test cases
* add invalid protocol test case

Change-Id: I2a3e46e2df2942e091aa0620345c91c4e7d10bbb

Redo unit tests for AdminCheckRequest in ProtocolAdmin

* squash positive test cases
* add invalid protocol test cases

Change-Id: Icfbceb6ba42d3ebf96855caf7a44482058fbb883

Categorize BucketRecordCorruptedException unit tests

Categorize into positive/negative test cases.

Change-Id: If2dc7634bfe8eed65265685fb2e7f9407423e252

Add unit tests for CancelResponse in ProtocolClient

Change-Id: If34ce28977fdf13e26a7d89c7a11cd9ce20a287a

Add unit tests for CheckResponse in ProtocolClient

Change-Id: Ibfb00e00db21ca10ba18c6fdcfa5b76fd64ea4f1

Add unit tests for MonitorEntryPutRequest in ProtocolClient

Change-Id: Idb17089a5061fb81fdd2b842c7366f2dbf6821c1

Remove unused code

Change-Id: Ifc6230cc0fdd4044a8967cf619a6e29585ca6228

Add tests for ProtocolSerialization

Change-Id: I9afce8b2635e81c3ffaae45804c1d4b93251b314

Add unit tests MonitorEntriesPutRequest in ProtocolClient

Change-Id: Iab7265f104716cb3607e1b5dd10bb0d3ea53a9f6

Move protocol serialization/deserialization to one place

Move serialization and deserialization of common types to
ProtocolSerialization and ProtocolDeserialization respectively.
This eliminates code redundancy, reduces the possibility of
wrong order of serialization/deserialization and moves protocol
size limits checks to one place.
Currently max container size is 4096. As there is no valid case
when cynara can be passed a 4096-long container, this limit will
be applied, where it hasn't been before, except for
ProtocolMonitorGet, which utilizes different buffer size - 65535.

Change-Id: Id7469015712204b3be8a1aed255dcbfa91975be3

Ensure that MonitorCache capacity is of permissible size

When MonitorCache is flushed its whole content is sent to
Cynara service in one batch. Cynara service should not accept
vectors bigger than CYNARA_MAX_VECTOR_SIZE. Currently this is true,
but this commit adds a compile-time assertion for this limit,
so it won't be changed without realization of the consequences.

Change-Id: I5ae28882ea4c50a7455d2d0d3a7ff76d7ec7a38a

Add unit tests for SimpleCheckRequest in ProtocolClient

Change-Id: Ia797842b01e843f3273d3c436f6d3b38b857cf92

Add negative unit tests for ProtocolClient

Add unit tests for:
* requests and responses not implemented in ProtocolClient
* requests and responses with wrong OpCode
* requests and responses with unfinished frame

Change-Id: I3334a887b6aacac4d4924285da401fec41d7b89f

Add unit tests for CheckRequest in ProtocolClient

Change-Id: I7dec5ab1ae3c78a4c01f00cfd72eb1eb9e2efd1e

Add unit tests for CancelRequest in ProtocolClient

Change-Id: I6644438151faf09eb4aacf5c53e299048c1714fc

Add unit tests for AgentRegisterResponse in ProtocolAgent

Change-Id: Idbdef1ab3315dc4e3238e7fc57f334acdc7583e3

Add unit tests for AgentActionResponse in ProtocolAgent

Change-Id: I88d7ffa5851727d0b0f7905258a40697399bfdfd

Add unit tests for AgentRegisterRequest in ProtocolAgent

Change-Id: I66741df415ced3f8cae240b49aaf276c9097d952

Add unit tests for AgentActionRequest in ProtocolAgent

Change-Id: If94172c87fa6fc6cac6f93ce17edf38e31e2c69f

Release 0.16.0

* Remove logs ifdef from spec files
* Revert "Partially remove libdlog dependency"

Change-Id: Id0e4774c9c50bdce31b161e9d9f9d010e859fdf4

Remove logs ifdef from spec files

Logs backend choice became more complicated, than simple Tizen
version check. This ifdef is no longer accurate and needed.

Change-Id: Ic62497af4753bbd53253a7bf1ec13ff30575d389

Revert "Partially remove libdlog dependency"

This reverts commit 426f9b677c0b6b000ad369e5086726cbdb94e3a1.

Change-Id: I55eedc3b0ea2e9a6772d452c018e3d58b905ebdc

Release 0.15.0

* Dual license cynara under Apache 2.0 or MIT
* Partially remove libdlog dependency
* Add unit tests and redo existing ones to categorize
  them into positive/negative test cases
* Minor fixes and refactoring for new unit tests

Change-Id: Ic49ea50bf625bc222bdb8c3d4633a554c7aacd05

Dual license cynara under Apache 2.0 or MIT

Add licenses to devel packages.

Unify the copyright and license notice formatting.

Change-Id: I0cbd7f9d2fca963737cc2a7df4f9c9bf4c29a9a6

Partially remove libdlog dependency

Switch to journal logging by default in all modules except cynara service.
This is to avoid libdlog dependency in client libraries.

Only cynara service may depend on libdlog and use it by providing a logging
callback to libcynara-commons.

Minor fixes and refactoring.

Change-Id: I15cf8dae915bd5863db79d27cad539cd7103ea34

Change compare() not implemented assertion to compile time

Change-Id: I2c79edb3cd5d962784490139a996e14638fa33fb

Add negative unit tests for ProtocolAgent

Add unit tests for:
* requests and responses not implemented in ProtocolAgent
* requests and responses with wrong OpCode
* requests and responses with unfinished frame

Change-Id: Ibb3882764fc58d307c4f9ff49d75a116512a9c35

Add negative unit tests for ProtocolAdmin

Add unit tests for:
* requests and responses which are not implemented in ProtocolAdmin
* requests and responses with wrong opcode
* requests and responses with unfinished frame

Change-Id: Ifa0fa83b6caeaa32315c2abf14ef231005bfe573

Sanitize test data collection

* add template generation for numeric data
* remove redundant static specifier

Change-Id: I6f64988d98c633e55d027bb5c9f5bf0b8b2597a0

Add unit tests for CodeResponse with ProtocolAdmin

Change-Id: I3aad98d36969b27908cca5a1833531808b0cb6bc

Add SetPoliciesRequest unit tests

Change-Id: Ia4c6d8c9a48650ac472adc613c1c70707e71285b

Add RemoveBucketRequest unit tests

Change-Id: I2b0653f614d516f1b701cde904d8fc81c2a4bb7c

Add InsertOrUpdateBucketRequest unit tests

Change-Id: I5c5eae53ee460a12fe0215a08d1fd02434e888eb

Add tryCatch and exceptions messages tests

Change-Id: I1d82762ca73e491821336b069570ac961e38c8a8

Add parameters information to exceptions messages

Fix UnknownPolicyTypeException and BucketNotExistsException

Change-Id: I3e4ef71e9050781d48d9537cf44111d1727bc37e

Let tryCatch be used in multiple translation units

Move tryCatch() implementation to separate source file.

Change-Id: Ia4614a032a022765c386144e76c58c2eec6ab6b6

Add safeStrError function unit tests

Change-Id: I4f54c082fe854b07d63a7a89cb4f0c13b224427d

Add BinaryQueue unit tests

* add unit tests
* name test cases so they contain positive/negative information

Change-Id: I085c416bcc8fdd1cda1eeaed6778306f24ce5f52

Fix parameter checking in BinaryQueue

* check raw pointers for nullptr
* check for empty buckets in all methods
* don't perform redundant checks in Bucket class

Change-Id: Ia5607df736460881aea5e8b1020ab6c5391b22d4

Add MonitorCache unit tests

* add unit tests
* rename test cases to contain positive/negative information

Change-Id: I663957de5d25b646da4bab33f4f8de3f554243f1

Add SequenceContainer unit tests

* add new tests
* rename test cases to contain positive/negative information

Change-Id: I892fb8fa3c74075ec12d9f340c8e78443063f879