Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Examine TLV bounding #375

Open
dccutrig opened this issue Dec 17, 2024 · 2 comments
Open

Examine TLV bounding #375

dccutrig opened this issue Dec 17, 2024 · 2 comments
Assignees
@dccutrig

Comments

@dccutrig
Copy link
Contributor

Examine TLV bounding in Crypto_Process_Extended_Procedure_Pdu.

Need to ensure that TLV values in a packet are:

  1. Less than or equal to Max TLV Length (done, and fixed otherwise segfault - UTs exist) , and
  2. That TLV values make sense in the context of the length of both frames and space packets received

2 is a little more complicated and I can revisit
@dccutrig dccutrig self-assigned this Dec 17, 2024
dccutrig added a commit that referenced this issue Dec 18, 2024
@dccutrig
[#375] WIP - shows a good example of TLV actually being corrupted and…
a0d1e6b 
… deactivating a bunch of keys. Tests are being worked pre-development
@dccutrig
Copy link
Contributor Author

Behold, an example of what mismatched lengths and corrupted values can currently do - and also the point of this issue:

image

dccutrig added a commit that referenced this issue Dec 19, 2024
@dccutrig
[#375] WIP on UTs for TLV testing, rename tlv 'pdu' name to actually …
8773bcb 
…be tlv_pdu for clarity
dccutrig added a commit that referenced this issue Dec 19, 2024
@dccutrig
[#375] Wording in crypto_print for TLVs
7f941bf
Donnie-Ice added a commit that referenced this issue Feb 10, 2025
@Donnie-Ice
[#375] Added edge checks to Crypto_Key_Update
70ea14c
Donnie-Ice added a commit that referenced this issue Feb 10, 2025
@Donnie-Ice
[#375] Wrap print in debug
829d439
Donnie-Ice added a commit that referenced this issue Feb 10, 2025
@Donnie-Ice
[#375] Expand EP functionality
51e2f45
Donnie-Ice added a commit that referenced this issue Feb 10, 2025
@Donnie-Ice
[#375] Correct error codes in UTs
1f5ba96
@Donnie-Ice
Copy link
Contributor

Need to look deeper into sa_start

Donnie-Ice added a commit that referenced this issue Feb 11, 2025
@Donnie-Ice
[#375] Exercise sa_start function
2caaef7
Donnie-Ice added a commit that referenced this issue Feb 11, 2025
@Donnie-Ice
[#375] UT to test max size on sa_start (WIP)
a161ade
Donnie-Ice added a commit that referenced this issue Feb 12, 2025
@Donnie-Ice
[#375] Expand sa_start debugging
ed13cf8
Donnie-Ice added a commit that referenced this issue Feb 18, 2025
@Donnie-Ice
[#375] Add length checks to OTAR
669a583
Donnie-Ice added a commit that referenced this issue Feb 21, 2025
@Donnie-Ice
[#375] Added Crypto_Interface init debug prints
49755ce
Donnie-Ice added a commit that referenced this issue Feb 21, 2025
@Donnie-Ice
[#375] Added crypto.c max tlv checks, debug prints, and errors
172863f
Donnie-Ice added a commit that referenced this issue Feb 21, 2025
@Donnie-Ice
[#375] Fix wrong length variable used
d700a2b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dccutrig dccutrig

Labels
None yet
Projects
CryptoLib Path Forward
Status: No status
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Donnie-Ice @dccutrig