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.

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

Add XMSS secret key objects APIs, sync with LMS #1587

Closed
wants to merge 18 commits into from

Conversation

ducnguyen-sb
Copy link
Contributor

No description provided.

dstebila and others added 18 commits August 18, 2023 10:20
* init external and parameters

* fix the size of pk and sk

* add cmakelist for xmss_sha256_h10

* add comment

* fix format err

* fix compiler warning of size_t

* update to match local

* add remain and total API

* add sigs_remaining and sigs_total

* add const to API

* add kat_sig_stfl

* to pass the format test

* fix typo

* verbose error

* let's see if this work

* use UINT64_MAX constant

* goto err to avoid memory leaks

* safe arithmetic using unsigned int and initialize lengths array

* fix iteration type to match its comparison

* using unsigned int instead of uint32_t to match with comparison

* use memset to initialize default value

* convert to unsigned int

* propagate unsigned int

* clean up

* remove randombytes, use OQS_randombytes instead.

* use calloc instead of malloc for secret_key memory initialization

* remove randombytes from CMakeLists.txt

* remove makefile

* using namespace to separate core_hash.c

* rename katfile

* add test and kats for test_kat.py

* add compile definition for core_hash.c

* add type for t

* fix typo
* Re-add OQS_SECRET_KEY

* Updates per review and formating changes

* Set function callback for 'free'.

* Address escaped PR comment

* fix formatting

* Update src/sig_stfl/sig_stfl.h

Co-authored-by: Douglas Stebila <[email protected]>

---------

Co-authored-by: Norman Ashley <[email protected]>
* Add base LMS library

* ignore use of free() by adding // IGNORE free-check

* ignore use of free() by adding // IGNORE free-check
* populate all 28 XMSS parameters

* clean up

* remove wanrings in scanbuild

* change free to OQS_MEM_insecure_free

* fix build warning

* fix integer in i386 platforms

* proper type for sigs_remain and sig_maximum

* remove size_t in signature remain and total

* make scan-build happy
* Support LMS H5_W1

* Fix style check

* Rename CmakeLists.txt CMakeLists.txt

* Add namespace

* Address issues from scan results

* Address SA issue

* Fix formatting

* Fix formatting

* Commit Duc's SA fixes

* Fix mem leak, and compiler warning.
* Convert API to use OQS_SIG_STFL_SECRET_KEY

* Update formatting
* Initial addition of sig_stfl API and dummy XMSS variant

* add secret key object

* allocate and free using wrapper function instead of malloc/free

* cleaner function signature

* Fix comment

* Delete old file

* Missing newline

* Missing newlines
* Use secret key struct in LMS. Update de/serialize sk API

* Updates per comments

* Update per comments

* Fix mem leak

* Address scan bild issue

* Removed unused variable

* Remove unused struc member

* Address macOS-noopenssl build failures
* Add serialize and deserialize to XMSS
---------

Co-authored-by: Norman Ashley <[email protected]>
* Callback implemention updating secret key.

* Block XMSS from secret key tests until after support code has been added.

* Remove / from test file names

* Format

* Address SA issues

* Fix mem leak

* Fix mem leak

* Address various comments

* Fix SA issue
* Add mutex protection around access to stateful secret key

* Formatting.

* Clean up warnings

* Exclude XMSS from some tests temporarily

* Remove commented code.

* Document use of callback functions for secret key thread safe protection and storage.
* Added functions to query the total, as well as, the remaining numbers of signing operation for a given secret key.

* Cleanup unused variable

* Fix code style
* Added new LMS varients. Removed unneeded vector functions from secret key data struc.

* Add LMS variants

* Fix formatting
@ducnguyen-sb ducnguyen-sb deleted the add_secret_functions_call_to_xmss branch October 20, 2023 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants