Skip to content

fix(sha512): use 64-bit counters - #27

Merged
LimiNode merged 2 commits into
mainfrom
codex/change-m_tot_len-and-m_len-to-uint64_t
Sep 3, 2025
Merged

fix(sha512): use 64-bit counters#27
LimiNode merged 2 commits into
mainfrom
codex/change-m_tot_len-and-m_len-to-uint64_t

Conversation

@LimiNode

@LimiNode LimiNode commented Sep 3, 2025

Copy link
Copy Markdown
Owner

Summary

  • change SHA512 length counters to uint64_t and handle arithmetic accordingly
  • add regression test for hashing data larger than 4GB

Testing

  • g++ -std=c++17 test_all.cpp sha512.cpp sha256.cpp sha1.cpp hmac_utils.cpp hmac.cpp -lgtest -lgtest_main -lpthread -o test_all
  • ./test_all >/tmp/test.log && tail -n 20 /tmp/test.log

https://chatgpt.com/codex/tasks/task_e_68b8c14560ec832cac29da62a5872fe1

@LimiNode
LimiNode merged commit 8b99d2a into main Sep 3, 2025
4 checks passed
@LimiNode
LimiNode deleted the codex/change-m_tot_len-and-m_len-to-uint64_t branch September 4, 2025 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant