Commit Graph

329 Commits

Author SHA1 Message Date
surtur cf0e548315
pre-commit: exclude the lib folder
[skip ci]
2021-12-05 03:10:50 +01:00
surtur 6f6b3cb8e2
generator: wrap the initialization in a try-catch
All checks were successful
continuous-integration/drone/push Build is passing
2021-12-05 02:00:25 +01:00
surtur a1cbbb209e
handle PRNG state in R_state + accumulator basis
All checks were successful
continuous-integration/drone/push Build is passing
* handle the PRNG state with R_state nested class
* add a private property R holding PRNG state to Fortuna
* add R_state properties as defined in Cryptography Engineering:
  * a generator instance
  * a reseed counter
  * 32 pools that the collected entropy is to be distributed over

* add initial definition of the Pool object and its initialization

* attempt to initialize PRNG in Fortuna constructor. wrap the
  initialization call in a try-catch block like a cultured person
* erase the string used to print data from random_data() after it's been
  used
2021-12-04 00:40:39 +01:00
surtur 72bb378709
pre-commit: add 'make san' step
All checks were successful
continuous-integration/drone/push Build is passing
making sure to build with sanitize flags before committing
2021-12-03 20:34:45 +01:00
surtur 77f1fe4c56
cmake,makefile: sanitize!
All checks were successful
continuous-integration/drone/push Build is passing
as per #3
2021-12-02 18:34:07 +01:00
surtur e72aa06813
executable hardening v2
All checks were successful
continuous-integration/drone/push Build is passing
* enable PIE
* pass flags to linker based on the build type
* enable LTO
2021-12-02 18:25:15 +01:00
surtur bb12cf7b56
cmake: always set -Wall
All checks were successful
continuous-integration/drone/push Build is passing
2021-12-02 14:50:39 +01:00
surtur b15e1e9a55
harden executable
All checks were successful
continuous-integration/drone/push Build is passing
* fortify source
* link with "-pie"
* split stack
* set stack-protector to all
2021-11-30 15:52:17 +01:00
surtur 0c2a1c6744
finalise generator
All checks were successful
continuous-integration/drone/push Build is passing
commit a64b52e5a4
Author: surtur <a_mirre@utb.cz>
Date:   Sun Nov 21 23:39:30 2021 +0100

    finalise generator

    this commit adds a (nearly) complete implementation of the generator.

    * wrap calls to generator in a fortuna class method random_data
      * calls generator's method generate_random_data, that internally calls
        generate_blocks
    * use a proper 256bit key in G_state
    * add reseed method implementation
    * call a reseed in initialize_generator
    * do_sha returns proper digest now
    * add proper do_crypto implementation
      * call generate_blocks internally
      * handle re-keying
    * optimise header includes

    TODO: there are still many commented (enabled on demand) debugging statements
    -> TO BE REMOVED
2021-11-30 14:09:39 +01:00
surtur d942e06263
cmake: compile PIC with "-fPIC"
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-20 22:31:55 +01:00
surtur e531aa6289
cmake: print version
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-20 22:31:55 +01:00
surtur fad32c0a41
makefile: cppcheck suppress "missingIncludeSystem"
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-20 22:31:55 +01:00
surtur e00bc4d277
makefile(valgrind): add --leak-check=full
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-20 22:31:55 +01:00
surtur bcd33c2f33
chore: direct braced string initialization
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-20 22:31:55 +01:00
surtur 4cea227cdd
generator: prefer preincrement
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-20 22:31:54 +01:00
surtur 444660bf89
feat(generator): add constructor/destructor
All checks were successful
continuous-integration/drone/push Build is passing
* initialize_generator() is now a private method
* call initialize_generator() inside the constructor of the Generator
* move full struct declaration to generator.h
* have a private field named G of type G_state
* remove explicit instantiation of Generator from fortuna constructor -
  Generator is a private field of fortuna
* remove direct initialize_generator() call from fortuna constructor
* remove get_state() calls as G_state G is now a private field
* refactor get_state() to simply return the *private field G_state G*
* refactor do_crypto() - I know, this should have been split to a
  separate commit - remove function parameters and useless assignments
  inside its body to reflect that once it accesses k and ctr (of the
  G_state G), it won't be necessary for them to be passed as parameters,
  since *G_state G is now a private field*...
2021-11-20 22:31:54 +01:00
surtur 2fa7e135cb
chore(fmt): follow-up of bd42129
All checks were successful
continuous-integration/drone/push Build is passing
this time replace printf's
2021-11-19 22:32:03 +01:00
surtur 225952a7e2
fix(fortuna): add end-of-namespace comment
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-19 20:59:12 +01:00
surtur bd421295c4
feat: use fmt instead of iostreams
All checks were successful
continuous-integration/drone/push Build is passing
switch to fmt - "A modern formatting library" - for fun and profit,
currently in header-only configuration.

ref: https://github.com/fmtlib/fmt

closes #5
closes #4
2021-11-18 22:05:05 +01:00
surtur 9fd35eee93
cmake: require the declared cxx standard
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-18 19:49:15 +01:00
surtur 8d8129dc40
refactor(generator): use solely SHA3
All checks were successful
continuous-integration/drone/push Build is passing
still in test mode (see the assert) at the moment
2021-11-17 23:40:38 +01:00
surtur 85907e7178
generator: add a meaningful assert message
All checks were successful
continuous-integration/drone/push Build is passing
should help with debugging...
2021-11-16 00:17:16 +01:00
surtur c82d3912f5
generator: add CTR-mode Serpent to random_data()
All checks were successful
continuous-integration/drone/push Build is passing
operate on a fixed string with randomly generated key and iv block on
each run.

this is neither finished nor correct implementation, it currently also
calls the Decryption routine to prove that it's working properly with
the input given ("Oh, I am fortune's fool!" seemed fitting for fortuna).
2021-11-15 23:37:04 +01:00
surtur b934b11aa9
cmake(release): harden the build
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-15 23:07:59 +01:00
surtur 0199fbd675
cmake: optimize for debugging with -Og
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-15 23:04:37 +01:00
surtur 9a9b81769f
chore(generator): directly reference std methods
All checks were successful
continuous-integration/drone/push Build is passing
instead of using "std::method" for cout and endl
2021-11-15 23:02:49 +01:00
surtur 2f2f47da01
generator: silence pedantic warnings for __int128
All checks were successful
continuous-integration/drone/push Build is passing
in fortuna, a use of 128bit integer is necessary, which also violates ISO C++:
"warning: ISO C++ does not support ‘__int128’ for ‘ctr’ [-Wpedantic]"

enclosing only the problematic blocks in ignore is chosen as a superior
course of action, compared to completely removing "-Wpedantic" from
CXX_FLAGS, as proposed in #2, as that would prevent any further
"pedantic" warnings from any further code to be shown.

this way, it is assured that the warning "ignore" is activated after
push the push, after which the environment is returned back to its
previous state with a pop, allowing any further warnings to appear.

closes #2
2021-11-15 22:47:22 +01:00
surtur c0012e36f9
chore: remove unused #include <algorithm>
All checks were successful
continuous-integration/drone/push Build is passing
* "include what you use"
2021-11-15 22:41:22 +01:00
surtur 3f411f687e
chore: header-guard encompass files properly
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-15 22:13:00 +01:00
surtur 20c22fef47
chore(main): don't directly work with Generator
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-15 22:05:07 +01:00
surtur 2cd8345c4e
refactor: use HexEncoder's built-in lowercase
All checks were successful
continuous-integration/drone/push Build is passing
remove the custom-defined strtolowerpls from generator
2021-11-14 22:23:50 +01:00
surtur d8adb1af87
refactor(all): using namespace considered harmful
All checks were successful
continuous-integration/drone/push Build is passing
* project-wide refactor to accomodate removal of "using namespace xyz"
  to stop polluting top-level namespace with "std" as recommended by the
  Google C++ style guide.
* use a pre-commit hook to enforce this

ref:
https://google.github.io/styleguide/cppguide.html#Namespaces
2021-11-13 22:13:50 +01:00
surtur bad86c31df
refactor(-lcryptopp): use cmake's facilities
All checks were successful
continuous-integration/drone/push Build is passing
this means that all builds configured with cmake (not just those called
with our custom development makefile) are properly linked with cryptopp
2021-11-13 00:13:19 +01:00
surtur 1e7395f488
chore: rm bogus reseed() call, use asserts
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-11 04:13:04 +01:00
surtur f16c630ae7
generator: implement do_sha()
All checks were successful
continuous-integration/drone/push Build is passing
* link against cryptopp
* use both sha2 and sha3 generation functions
* try calling do_sha() directly
* call reseed() that in turn calls do_sha()
* return a bogus number, not the proper digest for now (see TODOs)

cryptopp needed to be installed to archlinux for the valgrind step to
pass successfully
2021-11-11 04:11:40 +01:00
surtur 73f8bd75a2
generator: add strtolowerpls method
All checks were successful
continuous-integration/drone/push Build is passing
a convenience method, transforms strings to lowercase
2021-11-11 04:10:07 +01:00
surtur b1fcc5d055
fix(makefile): properly use -DCMAKE_CXX_FLAGS
All checks were successful
continuous-integration/drone/push Build is passing
..even if momentarily empty (no additional flags are being set from the
makefile directly, everything is set in CMakeLists.txt.
2021-11-11 02:18:01 +01:00
surtur ea11801789
add fortuna class
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-10 23:58:42 +01:00
surtur 40ae062657
generator: the number of blocks has to be >= 0
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-08 12:39:56 +01:00
surtur d986b6b482
generator: make generate_blocks() public
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-08 09:45:30 +01:00
surtur 74f42935ed
makefile: add informational echos
[skip ci]
2021-11-08 09:41:41 +01:00
surtur f64295f052
move to oop paradigm
All checks were successful
continuous-integration/drone/push Build is passing
squashed the following:

commit a28354d585
Author: surtur <a_mirre@utb.cz>
Date:   Wed Nov 3 02:14:35 2021 +0100

    chore: make G_state a private structure

    * make G_state private
    * move protected section behind private
    * also do cleanup a little -> k and ctr won't exist as part of the
      generator outside of the G_state struct

commit 926216fad3
Author: surtur <a_mirre@utb.cz>
Date:   Wed Nov 3 01:37:28 2021 +0100

    chore: make methods aware of objects

    as a consequence we stopped returning the generator state from every
    function and equally stopped using it as a parameter to every generator
    method that handles the state in any way

commit 01eadae32f
Author: surtur <a_mirre@utb.cz>
Date:   Tue Nov 2 05:10:58 2021 +0100

    feat: move from structural to object paradigm

    * declare and implement Generator class and member methods
    * create an instance of Generator in main
    * call initialize_generator() from main
2021-11-03 02:57:23 +01:00
surtur f2ed6918c9
chore(makefile): add 'clean' target
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-02 04:40:08 +01:00
surtur 253acfa7e4
chore: increment using ++
All checks were successful
continuous-integration/drone/push Build is passing
2021-11-02 04:40:08 +01:00
surtur b0eaf5e49e
generator(reseed): concat k with seed, ctr++
All checks were successful
continuous-integration/drone/push Build is passing
also got rid of the redundant ctr variable as we only need the one that
is part of G_state.
2021-11-02 04:40:08 +01:00
surtur 7d123ce639
main.cpp: switch to a using declaration
All checks were successful
continuous-integration/drone/push Build is passing
..instead of pulling the whole namespace
2021-11-02 04:40:08 +01:00
surtur 7a133ab2dc
namespace-wrap the generator function block
All checks were successful
continuous-integration/drone/push Build is passing
* the purpose is to subdivide the global scope
* use 'fortuna' as the top-level namespace
* use 'generator' namespace for the generator functions
* wrap the entire file after includes
2021-11-02 04:40:07 +01:00
surtur 2bab9e7afa
cmake: use lld if available
All checks were successful
continuous-integration/drone/push Build is passing
the find "query" is a little naive but hey, we're looking at the usual
place. the purpose of this change is iteration speedup and as lld is the
quickest kid on the block, we're using it.
2021-11-02 04:40:07 +01:00
surtur 87b13e1e1e
chore(cmake): print compiler id
[skip ci]
2021-11-02 04:40:07 +01:00
surtur c968d8c4ba
chore(cmake): force compiler colours
I know, right...

[skip ci]
2021-11-02 04:40:07 +01:00