Fix Memory Leak in serialize_indirect_attribute()

[hoyong2007]

Add release_serial_object(obj2) to release obj2
before serialize_indirect_attribute() returns to
prevent memory leakage.

[hoyong2007]

This is a PR for issue #387

[etienne-lms]

Could you format the commit message in a more standard way? A header line (74 char/max, starting with libckteec: + a short description + your Signed-off-by: tag. Please also add a Link: tag to reference the Issue entry this change fixes, as well as a Fixes: tag referencing the faulty commit. Something like:

libckteec: fix memory allocation leakage on template serialization

Fix memory allocation leakage with a call to release_serial_object()
to release obj2 before serialize_indirect_attribute() returns.

Link: https://github.com/OP-TEE/optee_client/issues/387
Fixes: e88c264ba358 ("libckteec: helper function to serialize a attribute template")
Signed-off-by: REAL-NAME <E-MAIL-ADDRESS>

There are few indications in the contribution documentation. See also the other commits in optee_client.git history.

[etienne-lms]

Thanks for the fix. Could you squash the 2 commits into a single one.
You can apply my R-b tag: Reviewed-by: Etienne Carriere <[email protected]>.

[etienne-lms]

@jforissier, could you enable CT tests for this P-R?

[etienne-lms]

Looks all good to me.
@jforissier, could you enable CI tests for this change?

[jforissier]

@jforissier, could you enable CT tests for this P-R?

Sure, done.

[etienne-lms]

Thanks.
All good. Ready to merge.