Merge pull request #183 from chipitsine/master

travis-ci: enable coverity scan

travis-ci: enable coverity scan

Merge pull request #182 from outspace/master

Fix leaks in priv_encode_gost()

Merge pull request #181 from vt-alt/issues/174

Issues/174

Fix leaks in priv_encode_gost()

gost_ec_keyx: Remove redundant code

This `if' statement is redundant since ce40d60 ("Bugfix.")

get_gost_engine_param() have no side effects and `param' is set to
`gost_cipher_list' only if already equal to `gost_cipher_list'.

gost_ec_keyx: Fix CID 253282 Dereference null return value

`param' is used unconditionally in pkey_GOST_ECcp_encrypt() so we can
unconditionally check (vs in any particular `if' branch) for it's being
non-NULL. This is similar to how pkey_GOST_ECcp_decrypt() works with
`param'.

Closes #174.

Fix potential memory leak

Fixes #180

Merge branch 'master' of https://github.com/gost-engine/engine

Fix of potential memory leak

Fixes #178

Merge pull request #176 from lumag/gost512-test-param

Add support for NID_id_tc26_gost_3410_2012_512_paramSetTest

Add support for NID_id_tc26_gost_3410_2012_512_paramSetTest

Add support for 512-bit curve parameters.

Signed-off-by: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>

Merge pull request #175 from vt-alt/scan20190918

Fixes for Scan 20190918

gost_ec_keyx: Fix CID 253283 Unchecked return value in VKO_compute_key

Calling EC_POINT_get_affine_coordinates() without checking
return value.
Some fixes for Coverity Scan from issue #174.

test_grasshopper: Fix CIDs 253281, 253280, 253279, 253275

Unchecked return value (CHECKED_RETURN)
Some fixes for Coverity Scan from issue #174.

Fix OOB read. CID 253274

Fix resource leak. CID 253277

Coveruty CID253284

Fix fb infer complain

Fix fb infer complain

Merge pull request #170 from hackomatic/wip

Fix compilation with Microsoft Visual C.

Make VKO_compute_key public.

Fix compilation with Microsoft Visual C.

Get rid of EVP_MD_CTRL_MAC_LEN

Merge pull request #169 from levitte/omac-ctx-size

For Grasshopper OMAC_ACPKM, OMAC and MAGMA, support XOF

For Grasshopper OMAC_ACPKM, OMAC and MAGMA, support XOF

XOF (eXtendable-Output Function) is supported in OpenSSL since version
1.1.1.  Unfortunately, that support is limited, as it requires calling
EVP_DigestFinalXOF().  However, the grasshopper code wasn't compatible.

This changeset adds XOF support in grasshopper

This changeset removes the _meth_ calls from test_grasshopper.c (they
should never have been there, but considering support for XOF came so
late in OpenSSL, their presence is understandable) and uses supported
functionality for this sort of thing instead.

Merge pull request #166 from levitte/cleanup-tests

Cleanup tests

test/*.t: Don't check OPENSSL_ENGINES

It's set correctly when these tests are started with ctest, and
otherwise, we leave it to the user to decide.  Either way, we set it
to a bogus value when it isn't previously set...

CMakeLists.txt: make sure the engine tests get OPENSSL_CONF too

An OPENSSL_CONF from the user may cause trouble

Merge pull request #164 from levitte/run_tests-verbosity

test/run_tests: when ctest is verbose, make TAP::Harness verbose too

test/run_tests: when ctest is verbose, make TAP::Harness verbose too

Merge pull request #161 from levitte/fix-146-2

test/run_tests: extend support for more operating systems

test/run_tests: extend support for more operating systems

Not all systems recognise LD_LIBRARY_PATH, and we therefore need to
set other environment variables.

Currently set:

- LD_LIBRARY_PATH (Linux and ELF HP-UX)
- DYLD_LIBRARY_PATH (MacOS X)
- LIBPATH (AIX, OS/2)

More can be added as the need arises.

Fixes #146

Merge pull request #160 from levitte/fix-146

test/run_tests: Always set paths when OPENSSL_ROOT_DIR is defined

.cirrus.yml: install pkgconf

That will add the pkg-config on FreeBSD

test/run_tests: Always set paths when OPENSSL_ROOT_DIR is defined

When the environment variable OPENSSL_ROOT_DIR is defined, we need to
ensure that PATH and LD_LIBRARY_PATH are properly affected.

This is a little bit complicated, since OPENSSL_ROOT_DIR could point
at an OpenSSL build directory as well as an OpenSSL installation
tree.  Fortunately, it's easy to detect the difference; if it's a
build tree, there's a sub-directory 'apps'.

For installation trees, it's further complicated by OpenSSL's
'multilib' target attribute, which exists for platforms that support
directory separated libraries for differing bitness (typically 32-bit
and 64-bit libraries).  We trust pkg-config to give us the answer.
Binaries are easier, they live in $OPENSL_ROOT_DIR/bin.

Fixes #146

Merge pull request #159 from levitte/fix-154

gost_pmeth.c: mitigate for constness difference in copy function

gost_pmeth.c: mitigate for constness difference in copy function

The second argument for the pkey_copy function has been constified in
OpenSSL to be 3.0.  This is normally not a problem, but when passing a
pointer to such a function, the compiler does complain.

This change solves the issue by looking at the macros
OPENSSL_VERSION_MAJOR, which is new in OpenSSL to be 3.0, and set
constness based on that.

Fixes #154

Merge pull request #141 from chipitsine/ppc64le

travis-ci: add linux-ppc64le build

travis-ci: install cmake-3.15.1 on linux-ppc64le

travis-ci: add linux-ppc64le builds

Merge pull request #152 from vt-alt/fix151

Fix test_grasshopper and test_keyexpimp engine init

Try to fix Travis-CI build installing cpanimus

Try this approach:
https://stackoverflow.com/questions/32358197/travis-ci-install-perl-module-with-non-perl-language

Fix test_grasshopper and test_keyexpimp engine init

Make proper engine initialization for tests instead of
EVP_add_cipher/EVP_add_digest hack. Fixes #151.

Merge pull request #147 from chipitsine/clean_osx_travis_workaround

travis-ci: cleanup "so/dylib" mangling, not needed

travis-ci: cleanup "so/dylib" mangling, not needed
after https://github.com/openssl/openssl/pull/8951

Make build against current OpenSSL with --no-deprecated

Revert previous commit

Try to fix --no-deprecated build

Merge pull request #140 from chipitsine/master

travis-ci: add osx builds

travis-ci: add osx builds

Merge pull request #137 from DJm00n/master

Fix MacOS building (master branch)

Fix MacOS building

Fixes #122

Merge pull request #138 from Voev/patch-1

Не учитывается значение carry

Не учитывается значение carry

Update README.md

Update README.md

Merge pull request #136 from chipitsine/master

initial cirrus-ci support

initial cirrus-ci support

Merge pull request #135 from vt-alt/test_tls2

test_tls: use socketpair instead of network connection

test_tls: use socketpair instead of network connection

May fix build on Cirrus-CI, issue #134.

Merge pull request #132 from vt-alt/fbsdfix

Fix compilation on FreeBSD

Fix compilation on FreeBSD

Otherwise `struct sockaddr_in` is undefined for test_tls.

Reported in #123 by f-andrey.

Merge pull request #130 from vt-alt/align2

test_digest and alignment fix for Streebog

Fix Streebog alignment problem

Some architectures in some circumstances do not allow unaligned
memory access (such as ARM, MIPS) triggering SIGBUS. This patch
very crudely fixes this issue.

The issue is found and original fix is proposed by Eric Biggers:

  https://patchwork.kernel.org/patch/10878865/

test_digest: simple digest tests

Especially test alginment problems on MIPSel.

Cofactor should be taken into account in VKO

Merge pull request #127 from chipitsine/master

travis-ci: switch to xenial

travis-ci: switch to xenial

Fix one more constification

Fix constification

Partially fix #125 (Memory leak in pub_encode_gost_ec)

Merge pull request #121 from vt-alt/test_tls

test_tls: simple TLS connection testing

We cannot ensure ALIGN(16) of externally allocated data

test_tls: simple TLS connection testing

Generate cert for every parameter, set it up on s_server, connect
s_client, and transfer 1M of data to both directions.

Merge pull request #96 from abbra/support-Test-Stream

Migrate test framework to Test2

Add Test2::V0 to Travis CI

Migrate test framework to Test2

At least Fedora 30 will have no Test::More anymore. Test::More is
deprecated, as well as Test::Stream and Test2 is a recommended
replacement.

Luckily, gost-engine's use of Test::More is limited to functions which
are compatible with implementations in Test2, so only module import and
test plan announcement need to change.

Also, on Fedora 29 perl 5.28 is very picky on undefined variables.

Merge pull request #120 from vt-alt/test_engine

tests: Load just built engine from the output directory

tests: Load just built engine from the output directory

Define ENGINE_DIR for all binary tests. Instead of auto-loading engine
using `example.conf' load it directly from the OUTPUT_DIRECTORY for
curves tests.

Clang options

Merge pull request #119 from vt-alt/lrt

cmake: benchmark/sign - link with rt for clock_gettime when needed

cmake: benchmark/sign - link with rt for clock_gettime when needed

Test if clock_gettime() requires linking with -lrt and link sign
benchmark with rt if needed. clock_gettime() requires -lrt only for
glibc versions before 2.17.

rt library for clock_gettime

Merge pull request #117 from vt-alt/asan

travis: Enable ASan build

travis: Enable ASan build

Merge pull request #115 from vt-alt/leak

Fix leaks in tests

Fix memory leaks in the tests

In preparation for memory leak detector fix them in the tests.

test_curves: Fix forgotten comparison in test

Merge branch 'master' of https://github.com/gost-engine/engine

Fix to #110

Merge pull request #111 from vt-alt/check

Implement EVP_PKEY_{,public_}check

Implement EVP_PKEY_{,public_}check

Very simple redirect to EC_KEY_check_key() which is doing all basic
checks (see crypto/ec/ec_key.c:ec_key_simple_check_key()).

Merge pull request #109 from vt-alt/test_params2

 test_params: remove calls to internall API and add more tests

test_params: additional tests

- Manually verify cert using EVP_Verify API. (In addition to
X509_verify).
- Add test using EVP_DigestVerifyInit API for raw vectors.

Merge pull request #108 from vt-alt/perl

run_tests: pass OPENSSL_ROOT_DIR into perl tests

Merge pull request #107 from vt-alt/travis

travis: speed up Travis build

Merge pull request #106 from vt-alt/test_sign

Sign/Verify and tests for all curves

test_params: Avoid calls to fill_GOST_EC_params and bind_gost

Use external API to generate key with parameters instead of calling
fill_GOST_EC_params(). Use OPENSSL_CONF trick like in test_sign to avoid
calling bind_gost(). Reverting 1716316de77 ("Export bind_gost to allow
local engine init") as not needed anymore.

Enable benchmark for all curves