Signed-off-by: Ngao_harmony <gaofeng44@huawei.com>
Change-Id: Ib76ee2cd8e4eefc6ab2a05114b966a5e4f236a54

Change-Id: I28596fee91e500a25307d608f238871ccf938459
Signed-off-by: Ngao_harmony <gaofeng44@huawei.com>

Change-Id: Ic48db9b6b4aff165c91d1214a6b8b83747a6b841

Change-Id: If61683f8fd4fe2a6236008c9fb8b27a4e7c272e9

Change-Id: I997f611b09917e657b063111ce04524281b2df3f

Signed-off-by: NRobert Drab <robert.drab@huawei.com>

localtime can return null on error, therefore we need to check it
before dereferencing.
Signed-off-by: NRobert Drab <robert.drab@huawei.com>

Size of the buffer was smaller then max size used in strftime
and alike function calls.

Moreover, according to man strftime contents of the buffer
is undefined in case in which actual string is longer than the
specified max size, therefore buffer is zeroed and last byte
left unused.
Signed-off-by: NRobert Drab <robert.drab@huawei.com>

In many places checking for nullptr is done using EXPECT test macros
that doesn't return from the function when condition is not met
(i.e. they are non-fatal). Lack of further error handling results
in a subsequent segfault. To avoid that in most places it's sufficient
to change macro type to ASSERT.
Signed-off-by: NRobert Drab <robert.drab@huawei.com>