Merge branch 'minimal-check' into 'master'

Run tests under minimal configuration

See merge request gnutls/gnutls!1122

tests: make tests pass with disabled GOST  algorithms

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

gitlab-ci: enable running make check on minimal build

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

Merge branch 'wip-cross-compilation' into 'master'

Fix cross-compilation of the Guile bindings

See merge request gnutls/gnutls!1120

Merge branch 'fix-coverity' into 'master'

vko: fix possible unitilized scalar access

See merge request gnutls/gnutls!1118

guile: Silence auto-compilation warning for 'guild'.

Reported by Helmut Grohne <helmut@subdivi.de>
and Andreas Metzler <ametzler@bebt.de>
at <https://bugs.debian.org/943905>.

* guile/Makefile.am (%.go): Pass "GUILE_AUTO_COMPILE=0" to avoid
warnings about 'guild' needing to be compiled.

Signed-off-by: Ludovic Courtès <ludo@gnu.org>

vko: fix possible unitilized scalar access

Fix error path in _gnutls_gost_keytrans_decrypt. If
_asn1_strict_der_decode() fails, cleanup path will try to
gnutls_pk_params_release(&pub), which will access unitialized pub
variable. Fix by deleting asn1 sctructure directly.

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

guile: Do not attempt to load shared object when cross-compiling.

Reported by Helmut Grohne <helmut@subdivi.de>
and Andreas Metzler <ametzler@bebt.de>
at <https://bugs.debian.org/943905>.

* configure.ac: Add 'CROSS_COMPILING' conditional.
* guile/Makefile.am (CROSS_COMPILING_VARIABLE): New variable.
(%.go): Use it.
* guile/modules/gnutls.in <top level>: Do not call 'load-extension'
when "GNUTLS_GUILE_CROSS_COMPILING" is defined.

Signed-off-by: Ludovic Courtès <ludo@gnu.org>

Merge branch 'gost-split-2' into 'master'

GOST key exchange support

See merge request gnutls/gnutls!1097

Merge branch 'tmp-cfb8-fixes' into 'master'

nettle: backport fixes to cfb8_decrypt

See merge request gnutls/gnutls!1117

Add support for VKO GOST key exchange

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

.gitlab-ci.yml: bump configure cache version

Signed-off-by: Daiki Ueno <dueno@redhat.com>

crypto-selftests: test CFB8 ciphers with different chunksizes

Signed-off-by: Guenther Deschner <gd@samba.org>
Signed-off-by: Daiki Ueno <dueno@redhat.com>

nettle: use included CFB8 implementation if nettle is 3.5

Signed-off-by: Daiki Ueno <dueno@redhat.com>

Merge branch 'prf-crash' into 'master'

prf: don't crash when called before handshake completion

See merge request gnutls/gnutls!1116

groups: add function to return group by curve

Two GOST groups will have two curves attached. Add function to retrieve
group by curve, rather than by group id.

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

ecc: define curve->group relationship

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

Declare groups corresponding to GOST curves

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

Add GOST key transport support

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

nettle: add support for GOST key derivation

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

_gnutls_pk_derive: add argument for nonce

GOST VKO key derivation needs another opaque argument (called UKM).
Add an argument to _gnutls_pk_derive to accomodate that keying material.

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

nettle/gost: add support for GOST VKO algorithm

GOST VKO is a variant of ECDHE algorithm.

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

nettle/gost: provide GOST keywrapping support

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

prf: don't crash when called before handshake completion

If a gnutls_prf*() function is called before the handshake is completed,
return GNUTLS_E_INVALID_REQUEST instead of crashing.

Signed-off-by: Miroslav Lichvar <mlichvar@redhat.com>

nettle: backport fixes to cfb8_decrypt

cfb8: don't truncate output IV if input is shorter than block size:
https://git.lysator.liu.se/nettle/nettle/commit/f4a9c842621baf5d71aa9cc3989851f44dc46861

Signed-off-by: Daiki Ueno <dueno@redhat.com>

gnutls_privkey_sign_data2: removed unnecessary text [ci skip]

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>

Merge branch 'crt-vrfy-final' into 'master'

tls-sig: reverse bytes in TLS signatures for GOST signatures

See merge request gnutls/gnutls!1114

Merge branch 'tmp-update-ci-to-f31' into 'master'

Update CI to F31

See merge request gnutls/gnutls!1113

.gitlab-ci.yml: do not inline strcmp in valgrind build

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>

.gitlab-ci.yml: removed unnecessary use of --enable-valgrind-tests

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>

Merge branch 'master' into 'master'

nettle: Support sysctl(KERN_ARND) for random number generation on NetBSD.

See merge request gnutls/gnutls!1109

Add NEWS entry for the NetBSD KERN_ARND change.

Signed-off-by: Nia Alarie <nia@NetBSD.org>

tls-sig: reverse bytes in TLS signatures for GOST signatures

GOST TLS suites have one peculiarity: CertificateVerify message uses
byte order opposite to the rest of GOST signature usage (BE instead of
LE). So, reverse byte order in signatures in TLS code. For now this
applies only to TLS 1.2 code. GOST TLS 1.3 ciphersuites will also follow
this approach. Legacy TLS 1.0 ciphersuites also had this peculiarity.

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

Merge branch 'logfile-doc-improvement' into 'master'

doc: describe how to make gnutls-cli quiet for pipe usage

Closes #845

See merge request gnutls/gnutls!1108

Merge branch 'remove-uint24' into 'master'

lib: simplify uint24 handling

See merge request gnutls/gnutls!1107

.gitlab-ci.yml: updated CI environment to F31

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>

tests: include config.h in rawpk-api.c

This seems to have impacted windows compilation.

Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>

tests: global-init-override do not run in windows

It cannot be compiled in f30.

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>

updated to libopts 5.18.16

This fixes compilation in Fedora 30 which ships with this
version of autogen.

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>

Merge branch 'fix-gost-curves' into 'master'

ecc: fix curve sizes for TC26-256 gost curves

See merge request gnutls/gnutls!1110

Merge branch 'serv-fix-table' into 'master'

serv: move closing TABLE tag after actual table end

See merge request gnutls/gnutls!1111

serv: move closing TABLE tag after actual table end

Move closing TABLE tag after printing information on cipher and MAC.

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

ecc: fix curve sizes for TC26-256 gost curves

Fix curve size being incorrectly set to 64 instead of 32 for several
GOST curves.

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

nettle: Support sysctl(KERN_ARND) for RNG on NetBSD.

This system call will never block and does not require a file
descriptor to be opened. It provides an endless stream of random
numbers from the kernel's ChaCha20-based random number generator.

Signed-off-by: Nia Alarie <nia@NetBSD.org>

doc: describe how to make gnutls-cli quiet for pipe usage

Signed-off-by: Bjoern Jacke <bjacke@samba.org>

Merge branch 'remove-guint64' into 'master'

lib: drop gnutls_uint64 usage as sequence number

See merge request gnutls/gnutls!1102

lib: simplify uint24 handling

Drop separate uint24 type and functions to convert between it and
uint32_t. This makes _gnutls_read/_write_uint24 simpler and easier to
understand. And with faster assembly code.

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

lib: drop gnutls_uint64 usage as sequence number

GnuTLS is depending already on uint64_t being a properly defined type.
So there is no need to have a special byte-array type for 8-byte
integers. Use uint64_t instead, thus simplifying a code quite heavily.

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

Merge branch 'tls-continuous-mac' into 'master'

Support GOST cipher suite MAC calculation

See merge request gnutls/gnutls!1098

Merge branch 'new-crt-vrfy' into 'master'

Split CertVerify code. Switch sign_entry_st to use flags

See merge request gnutls/gnutls!1103

Merge branch 'key-compat' into 'master'

lib: pubkey vs TLS signature compatibility for GOST algorithms

See merge request gnutls/gnutls!1101

sign: convert tls13_ok to flags field

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

tls-sig: split TLS 1.0/1.1 CertificateVerify code

For the symmetry split the TLS 1.[01] CertificateVerify code, so that
main functions work as pure multiplexors.

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

mac: mark GOST28147-TC26Z-IMIT as using CONTINUOUS_MAC

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

Support GOST cipher suite MAC calculation

GOST ciphersuites require that MAC is calculated over _all_ packets,
rather than just current packet. Add flag to auth_cipher_hd_st
controlling this behaviour.

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

mac: change preimage_insecure to be a flag

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

cipher: replace several bools with single flags instance

Replace bools in cipher_entry_st with flags field.

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

Merge branch 'fix-noreturn' into 'master'

src: fix noreturn-related warning

See merge request gnutls/gnutls!1100

lib: pubkey vs TLS signature compatibility for GOST algorithms

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

src: fix noreturn-related warning

Recent autogen started adding '#include <stdnoreturn.h>' into -args.h
files. However in GnuTLS tools code this results in the following
warnings, because stdnoreturn.h unconditionally redefines 'noreturn' to
_Noreturn:

warning: '_Noreturn' attribute directive ignored

Use __noreturn__ attribute instead as does Gnulib.

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

Merge branch 'stream-iv' into 'master'

Allow using implicit IV for stream ciphers with TLS

See merge request gnutls/gnutls!1099

Allow using implicit IV for stream ciphers with TLS

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

Merge branch 'gost-prf' into 'master'

prf: add Streebog PRF support

See merge request gnutls/gnutls!1088

Merge branch 'tmp-fix-coverity' into 'master'

Fix coverity in lib/

See merge request gnutls/gnutls!1092

prf: add Streebog (GOST R 34.11-2012) PRF support

Add support and tests for PRF generated using both Streebog versions.
This is necessary for adding GOST TLS ciphersuites support.

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

Add const to several read-only packet sequence params

Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>

tests/buffer.c: Add unit test for _gnutls_buffer_unescape()

Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>

lib/x509/x509.c: Check before pointer dereference in get_alt_name()

Fixes Coverity issue 1361513

Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>

cipher: Let _gnutls_auth_cipher_setiv() return int

Fixes Coverity issue 1454646

Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>

lib/record.c: Use assignment instead of memcpy()

Fixes Coverity issue 1454647

Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>

lib/sslv2_compat.c: Check return value of _gnutls_generate_session_id()

Fixes Coverity issue 1454649

Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>

lib/x509/output.c: Remove unneeded NULL check in print_crt_pubkey()

Fixes Coverity issue 1454670

Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>

lib/auth/srp_passwd.c: Fix NULL dereference in _gnutls_srp_pwd_read_entry()

Fixes Coverity issue 1454652

Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>

lib/str.c: Replace sscanf() in _gnutls_buffer_unescape()

Fixes Coverity issue 1454651

Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>

lib/handshake.c: Check return value of _gnutls_version_max()

Fixes Coverity issue 1454674
Fixes Coverity issue 1454658

Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>

Remove trailing spaces in several files

Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>

Merge branch 'tmp-remove-coverity' into 'master'

.gitlab-ci.yml: removed coverity build [ci skip]

See merge request gnutls/gnutls!1095

.gitlab-ci.yml: removed coverity build [ci skip]

The coverity run is subject to several restrictions by the service,
and thus it is not really useful in the main CI runs as it cannot reasonably
be run on MRs or master. As such we simplify the main CI file by moving the
coverity to the coverage sub-project and running it weekly.

The new location is at:
https://gitlab.com/gnutls/coverage

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>

Merge branch 'tmp-ext-mandatory' into 'master'

session tickets: parse extension during session resumption

Closes #841

See merge request gnutls/gnutls!1087

Merge branch 'doc-lscpu' into 'master'

README.md: document lscpu/util-linux dependency for make check

Closes #764

See merge request gnutls/gnutls!1093

Merge branch 'gost-pkcs11' into 'master'

P11tool improvements

See merge request gnutls/gnutls!1071

Merge branch 'psk-file-fix' into 'master'

tests/psk-file: fix heizenbug in last test

See merge request gnutls/gnutls!1090

Merge branch 'groups-non-ec' into 'master'

ext/supported_groups: don't consider non-EC groups for EC

See merge request gnutls/gnutls!1089

Merge branch 'tmp-fix-doc3' into 'master'

documentation updates

Closes #842

See merge request gnutls/gnutls!1069

crq APIs: fix typos [ci skip]

Resolves: #842

Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>

document limitations of gnutls_record_discard_queued() [ci skip]

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>

README.md: document lscpu/util-linux dependency for make check

Closes #764

Signed-off-by: Ricardo M. Correia <rcorreia@wizy.org>

testpkcs11.sh: test that we output mechanism flags correctly

Verify some of PKCS#11 mechanism flags.

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

p11tool: print mechanism info in list-mechanisms

Print key size range and flags in mechanisms list.

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

Merge branch 'fix-cert-keys' into 'master'

tests: correct gost server certificates

See merge request gnutls/gnutls!1080

tests/psk-file: fix heizenbug in last test

Currently last test case in psk-file expects that the server will
terminate connection with fatal error and close connection. Client will
receive GNUTLS_E_PUSH_ERROR error. However on slow boxes (or under qemu)
client is able to receive server's fatal alert thus returning unexpected
error. To make this behaviour predictable make server wait for client to
read all data and actually close connection on it's own.

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

session tickets: parse extension during session resumption on client side

It is possible for a server to send a new session ticket during
TLS1.2 resumption. To be able to parse it as client we need to
check the extension during resumption as well.

Resolves: #841

Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>

ext/supported_groups: don't consider non-EC groups for EC

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

tests: correct gost server certificates

Correct GOST server certificates:
 - use only Digital Signature Key Usage,
 - use new format for 512-bit curve key and certificate.

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

Merge branch 'new-gost-x509' into 'master'

Implement new requirements for GOST PublicKeyParameters

See merge request gnutls/gnutls!1070

.gitlab-ci.yml: only run coverity task on 3_6_x tags [ci skip]

Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>

cert-tests/gost: add certificate with new GOSTParameters struct

Add certificate example using simplified (new) GOSTParameters structure.

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

.gitlab-ci.yml: include an automated coverity build on tags

Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>

lib: implement support for updated GOST PublicKeyParameters

Recomendation for standardization R 1323565.1.023-2018 has made changes
to PublicKeyParameters for GOST R 34.10-2012 keys. It has removed
encryptionParamSet (since now S-BOX is basically fixed as TC26-Z) and
made digestParamSet OPTIONAL (as it can be concluded from public key
OID). Implement these requirements.

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

nettle/pk: add support for "new" TC26 256 B curve

TC26 likes aliases. Thus "new" TC26 256 B curve is the same as old
CryptoPro-256-A curve (but with limitation to use GOST R 34.10-2012).

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