cisco / libacvp

The libacvp library is a client-side implementation of the draft ACVP protocol (github.com/usnistgov/ACVP).
Apache License 2.0
67 stars 69 forks source link

Many small edits; Comments, typos, and data validation #834

Closed RandallSteck closed 4 months ago

RandallSteck commented 4 months ago

Many fixes/cleanup to comments, extra CRLFs, and some to data output

I have addressed several data validation issues where libacvp differed from ACVP protocol min/max values. I've altered max values for some buffers/arrays (e.g. filename max length to better accommodate windows long filenames) and increased some fields marked "arbitrary" where they length proved insufficient in some cases.

I have been compiling libacvp on a wide variety of systems, include small/embedded/IOT, and along the way I've made minor fixes to address compiler warnings and hardware/compiler "quirks". One operational environment failed Hash/Shake MCT testing, and it is due to a compiler's misinterpretation of the Rightmost Outer Bits (ROB) calculation. I've change it to be explicit and avoid "pointer math".

I'm happy to change/amend as needed. I have more "feature" oriented changes in future pull requests.

abkarcher commented 4 months ago

Merged. Thanks again!