Signed-off-by: Nnan-xiansen <nanjinbin@huawei.com>
Change-Id: Ic43d8abb7ae64da752598ffc1e3b60cbc4052402

Signed-off-by: Nnanxiansen <nanjinbin@huawei.com>
Change-Id: I0e20e79e68d7052384584ca9c590cedcdc7e860f

Signed-off-by: Nnanxiansen <nanjinbin@huawei.com>
Change-Id: I30d4eaee067e0feeff8ff48b29ae224a24e760a5

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>