In chaps (SessionImpl) and attestationd, we have several helper functions which transform the format between
a. TPM defined struct
b. Openssl object
c. DER-coded string/SecureBlob

Currently, we always put the helper function as a member function of the implementation class, even it doesn't need any information in the class.

We have some option to improve that depends on how many codes are duplicate and shared,
a. create/merge to a shared library to reuse these code across the daemon
b. create a utility library to reuse these code in a daemon (ex. chaps_utility)
c. move these function as a static function

 

In chaps (SessionImpl) and attestationd, we have several helper functions which transform the format between
a. TPM defined struct (TPM 1.2 and TPM 2.0 have different struct and serialize/unserialize funcs) 
b. Openssl internal object
c. PKCS#11 format
d. DER-coded string/SecureBlob of public/private key
   d-a. RSA: PKCS#1 and PKCS#8
   d-b. ECC: RFC 5915 and PKCS#8

Currently, we always put the helper function as a member function of the implementation class, even it doesn't need any information in the class.

We have some option to improve that depends on how many codes are duplicate and shared,
a. create/merge to a shared library to reuse these code across the daemon
b. create a utility library to reuse these code in a daemon (ex. chaps_utility)
c. move these function as a static function