tree e19308e49136c41852144b3b76813162579d1cda
parent 2d5f169baeebd01e4d0addf12060d08df16ade83
author Julian Hall <julian.hall@arm.com> 1643370071 +0000
committer György Szing <gyorgy.szing@arm.com> 1644236468 +0100

Correct handling of missing salt protobuf parameter

Asymmetric encrypt/decrypt operations take an optional salt
parameter. With the PB serialization, omitting optional
parameters is a legitimate way to tell the service provider
to assume a default. This wasn't handled correctly in
pb_crypto_provider_serializer.c. This change modifies
the behaviour of the cpp crypto client to omit the salt
parameter rather than explicitly encoding a zero length
salt. With the existing asymmetric encrypt/decrypt
service tests, operations with and without a salt
are already covered. With the change to the cpp client,
the invalid handling within the pb deserializer lead to
the expected test failure. A fix is implemented in
pb_crypto_provider_serilaizer.c that resolves the failure.

Signed-off-by: Julian Hall <julian.hall@arm.com>
Change-Id: I3ef89d992092009abf4d1cff5e6e9e032e23a2bc