dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/onomondo-eim/+/43502?usp=email )
Change subject: crypto_utils: extend TODO for better understanding ......................................................................
crypto_utils: extend TODO for better understanding
Related: SYS#8100 Change-Id: I411a3b4216b3c0d707d571b7a0c178e001931139 --- M src/crypto_utils.erl 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/onomondo-eim refs/changes/02/43502/1
diff --git a/src/crypto_utils.erl b/src/crypto_utils.erl index cb50415..c3cd1b1 100644 --- a/src/crypto_utils.erl +++ b/src/crypto_utils.erl @@ -31,7 +31,7 @@
%Encode the association token as BER TLV IE enc_association_token(AssociationToken) -> - %TODO: replace this with a proper ASN.1 encoding function. + %TODO: Define an ASN.1 type for the AssociationToken and use it to get rid of this half-baked encder function. AssociationTokenBinary = utils:integer_to_bytes(AssociationToken), AssociationTokenLength = utils:integer_to_bytes(byte_size(AssociationTokenBinary)), utils:join_binary_list([<<132>>, AssociationTokenLength, AssociationTokenBinary]).