Prefix gcc attributes with 'attr_'

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

gnutls_prf_early: corrected Since version [ci skip]

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

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

Fix warn_unused_result for clang < 4

See merge request gnutls/gnutls!1016

Fix warn_unused_result for clang < 4

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

Merge branch 'tmp-fedora30' into 'master'

.gitlab-ci.yml: switched fedora to latest version

See merge request gnutls/gnutls!1015

.gitlab-ci.yml: switched fedora to latest version

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

Merge branch 'tmp-releases' into 'master'

RELEASES.md: document the releases policy

See merge request gnutls/gnutls!1011

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

Do not regenerate autogen files if --enable-local-libopts is given

Closes #772

See merge request gnutls/gnutls!1010

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

Makefile.am: do not create files when it shouldn't

See merge request gnutls/gnutls!1014

Makefile.am: do not create files when it shouldn't

If a pdf or html file is not distributed, previously `make dist`
would create a file called '*.pdf' which did not make sense. This
addresses this problem.

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

Do not regenerate autogen files if --enable-local-libopts is given

This addresses issue on installed systems which have autogen but
use --enable-local-libopts. In these systems if the installed autogen
would not match the local libopts library version compilation would
fail because the auto-generated files depend on the corresponding to
autogen version libopts internals.

Resolves: #772

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

Merge branch 'tmp-datum-cleanup' into 'master'

Datum.c cleanup

See merge request gnutls/gnutls!1002

Remove malloc from gnutls_srp_set_server_fake_salt_seed()

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

gnutls_session_set_data(): Check for allocation error

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

_gnutls_set_[str]datum: Cleanup, add function attributes

_gnutls_set_datum(): Do not change output 'dat' on error
_gnutls_set_strdatum: Likewise, cleanup code

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

Add nonnull, nonnull_all and warn_unusd_result attributes in lib/gnutls_int.h

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

doc: do not distribute pdf files

It compicates the 'make dist' phase and does not add much
value as the files are available from the web site.

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

released 3.6.8

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

minor updates in the latex version of the manual [ci skip]

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

doc update [ci skip]

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

RELEASES.md: document the releases policy [ci skip]

This adds a file to document the policy on releases based on
the discussions taken place in the last face to face meeting.

https://gitlab.com/gnutls/gnutls/wikis/face2face-meeting-fosdem2019

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

Merge branch 'tmp-avoid-libs-in-libsvar' into 'master'

Do not add libraries in the global LIBS in configure

Closes #735

See merge request gnutls/gnutls!1008

.gitlab-ci.yml: ensure that the LIBS variable is empty after a configure run

We do not use this variable as it is global and applies to all of
tests, applications and library, and when it is set it is usually due to
bugs in configure.ac.

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

Do not add libraries in the global LIBS in configure

This ensures that libraries are linked with the programs
requiring them.

Resolves: #735

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

Merge branch 'tmp-version-override' into 'master'

tests: prf-early fixes the global version

See merge request gnutls/gnutls!1009

bumped version

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

tests: prf-early fixes the global version

This allows having fixed data in the hello message involved.
That required exposing the variable holding the global gnutls
version number for testing.

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

certtool: corrected typo in manual [ci skip]

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

Revert "bumped version"

In order to make the CI functional again. The version number update
seems to conflict with tests/tls13/prf-early.sh

This reverts commit d34d93b8713cf10235ce7016fd69b6932b0752c0.

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

tests: prf-early.sh: use the static flag of datefudge

This eliminates unexpected failures of the test in slower systems.

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

tlsfuzzer: reverted accidental move to incorrect version

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

NEWS: doc update

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

bumped version

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

Pass down Q for FFDHE in al pre TLS1.3 as well

Signed-off-by: Simo Sorce <simo@redhat.com>

Check Q for FFDHE primes in prime-check

These are mersenne primes so q = (p - 1) / 2
We check that p = (q * 2) + 1

Signed-off-by: Simo Sorce <simo@redhat.com>

Always pass in and check Q in TLS 1.3

In FIPS mode do an extra check that we did have Q, but it is always
passed into the tls13 derive function from the callers.

Signed-off-by: Simo Sorce <simo@redhat.com>

Add plumbing to handle Q parameter in DH exchanges

Signed-off-by: Simo Sorce <simo@redhat.com>

Add test to ensure ECDH exchange behaves correctly

This test ensures that public keys are properly tested for validity
before a ECDH exchange is computed.

Signed-off-by: Simo Sorce <simo@redhat.com>

Add test to ensure DH exchange behaves correctly

This test ensures that public keys are properly tested for validity
before a DH exchange is computed.

Signed-off-by: Simo Sorce <simo@redhat.com>

Add Full Public Key Check for DH

This is for NIST SP800-56A requirements and FIPS CAVS testing.
GnuTLS never passes in a non-empty Q for normal operations, but tests will
and if Q is passed in it needs to be checked.

Signed-off-by: Simo Sorce <simo@redhat.com>

Fix Copy&Paste error

Signed-off-by: Simo Sorce <simo@redhat.com>

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

Fix handling of malformed KeyUpdate messages

Closes #699

See merge request gnutls/gnutls!1005

Merge branch 'tmp-small-records' into 'master'

priority: add new option to allow small records (>= 64)

See merge request gnutls/gnutls!1006

Merge branch 'tmp-remove-unused-flag' into 'master'

pubkey: remove deprecated OLD_PUBKEY_VERIFY_FLAG_TLS1_RSA

Closes #754

See merge request gnutls/gnutls!1004

tls13/key_update: increase handling limit from 1 to 8

The limit was too small when testing the capability of handling
multiple KeyUpdate messages with tlsfuzzer.

This requires a change in the rate limit logic, as previously it
doesn't count the KeyUpdate messages despite the name of
KEY_UPDATES_PER_SEC.

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

Merge branch 'tmp-privkey-tls13' into 'master'

server auth: disable TLS 1.3 if no signature algorithm is usable

Closes #731

See merge request gnutls/gnutls!987

tlsfuzzer: use %ALLOW_SMALL_RECORDS for testing

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

priority: add new option to allow small records (>= 64)

There is a mismatch in the lower limit of record sizes in RFC
8449 (64) and our default (512).  If the server advertises a smaller
limit than our default, the client has no way to keep communicating
with the server.

This patch adds a new priority string option %ALLOW_SMALL_RECORDS to
set the limit to 64.

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

record_add_to_buffers: check if there is an incomplete handshake header

The function checks if a Handshake message is interleaved with an
Application Data, but the check was insuffient because it assumed that
a complete header is received in the buffer.

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

algorithms: eliminate the FFDHE alert desc requirement

This implements the errata for RFC 7919 eliminating the requirement to
reply with an insufficient_security alert when we have negotiated an
FFDHE group, but cannot find common ciphersuite:
https://www.rfc-editor.org/errata/eid4908

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

Merge branch 'mark_const' into 'master'

Mark second argument of function gnutls_x509_crt_equals2 as const

See merge request gnutls/gnutls!1000

Mark second argument of function gnutls_x509_crt_equals2 as const

This will allow using this function with certificates
returned by function gnutls_certificate_get_peers
without casts dropping const qualifier or
making temporary copies out of retrieved data.

Signed-off-by: Aleksei Nikiforov <darktemplar@basealt.ru>

tests: verify functionality of GNUTLS_VERIFY_DISABLE_CA_SIGN flag

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

pubkey: remove deprecated TLS1_RSA flag check

The gnutls_certificate_verify_flags comparisons against
OLD_PUBKEY_VERIFY_FLAG_TLS1_RSA conflicts with
GNUTLS_VERIFY_DISABLE_CA_SIGN and no longer seems to be used in calls to
both gnutls_pubkey_verify_data2 and gnutls_pubkey_verify_hash2 as it
seems to have been fully replaced by GNUTLS_VERIFY_USE_TLS1_RSA.

Resolves: #754

Signed-off-by: Kenneth J. Miller <ken@miller.ec>

x509.h: corrected typo in newly introduced definition

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

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

Few minor bug fixes for the next release

Closes #770 and #767

See merge request gnutls/gnutls!1003

x509.h: removed stray '%'

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

certtool: CA certificates will contain the digital signature key usage flag

This change ensures that all certificates will contain the digital
signature key usage flag if that's specified in the template.

Resolves: #767

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

Added profile to correspond to the future security parameter

It seems that the FUTURE security level parameter was added
without a corresponding verification profile. This patch address
the issue by introducing it.

Resolves: #770

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

tests: added unit tests of utc and generalTime convertor

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

server auth: disable TLS 1.3 if no signature algorithm is usable

This is a server side counterpart of
005a4d04145707daad9588acedfdb5f6cd97c80c.

Instead of signalling an error when no algorithm is usable in TLS 1.3,
it downgrades the session to TLS 1.2 with a warning.

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

algorithms/secparams.c: fixed indentation

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

gnutls-serv: GERR macro will output in stderr

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

Merge branch 'tmp-fix-evil-idna' into 'master'

Apply STD3 ASCII rules in gnutls_idna_map()

Closes #720

See merge request gnutls/gnutls!1001

Apply STD3 ASCII rules in gnutls_idna_map()

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

Merge branch 'tmp-fix-gcc4.4' into 'master'

Tmp fix gcc4.4

See merge request gnutls/gnutls!996

Fix _Thread_local for C99 installed in C11 environments

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

Remove redundant typedef of Tspi_Context_GetTpmObject_func()

Gcc 4.4 errors out on this.

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

Merge branch 'tmp-update-gnulib' into 'master'

Update gnulib for gcc-9 manywarnings

Closes #768

See merge request gnutls/gnutls!999

Update gnulib for gcc-9 manywarnings

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

Merge branch 'tmp-check-allocations' into 'master'

Check all memory allocation in examples and certtool

Closes #739

See merge request gnutls/gnutls!998

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

Fix endless looping GETPORT in tests/scripts/common.sh

See merge request gnutls/gnutls!997

Check all memory allocation in examples and certtool

Resolves: #739

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

Merge branch 'tmp-record-sizes' into 'master'

ext/record_size_limit: distinguish sending and receiving limits

See merge request gnutls/gnutls!985

ext/record_size_limit: distinguish sending and receiving limits

The previous behavior was that both sending and receiving limits are
negotiated to be the same value.  It was problematic when:

- client sends a record_size_limit with a large value in CH
- server sends a record_size_limit with a smaller value in EE
- client updates the limit for both sending and receiving, upon
  receiving EE
- server sends a Certificate message larger than the limit

With this patch, each peer maintains the sending / receiving limits
separately so not to confuse with the contradicting settings.

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

Merge branch 'fix-streebog' into 'master'

lib/nettle: fix carry flag in Streebog code

See merge request gnutls/gnutls!992

lib/nettle: fix carry flag in Streebog code

Fix carry flag being calculated incorrectly in Streebog code.

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

Merge branch 'tmp-lgtm-suppress-ctime' into 'master'

tools: suppress ctime() error from lgtm warnings

See merge request gnutls/gnutls!994

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

_gnutls_srp_entry_free: follow consistent behavior in freeing data

Closes #761

See merge request gnutls/gnutls!995

Fix endless looping GETPORT in tests/scripts/common.sh

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

_gnutls_srp_entry_free: follow consistent behavior in freeing data

_gnutls_srp_entry_free would previously not free any parameters that
were known to gnutls to account for documented behavior of
gnutls_srp_set_server_credentials_function(). This was not updated
when the newly added 8192 parameter was added to the library.

This introduces a safety check for generator parameters, even though
in practice they are the same pointer.

Resolves: #761

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

dane.h: added multiple inclusion header guard

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

tools: suppress ctime() error from static analysers

This function is not thread safe and can be easily misused
even in single threaded scenarios (one such minor bug fixed).

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

Merge branch 'fix-guile-option' into 'master'

guile: Properly format guile configure options

See merge request gnutls/gnutls!991

accelerated: added header guards

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

Merge branch 'tmp-header-guards' into 'master'

Add or clean header guards in lib/

Closes #728

See merge request gnutls/gnutls!954

Add or clean header guards in lib/includes/gnutls/

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

Add or clean header guards in lib/nettle

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

Add or clean header guards in tests/

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

Add or clean header guards in src/

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

Add or clean header guards in lib/x509/

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

Add or clean header guards in lib/tls13/

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

Add or clean header guards in lib/extras/

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

Add or clean header guards in lib/ext/

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

Add or clean header guards in lib/auth/

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

Add or clean header guards in lib/

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

Add 'Header guards' section in CONTRIBUTING.md

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

guile: Properly format guile configure options

Without the square brackets autoconf turns hyphens into underscores,
which is not what we want or what the help says.

Signed-off-by: Daniel Schaefer <git@danielschaefer.me>

gnutls_sign_list: document the non-thread-safeness

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

crypto: add private API to retrieve internal IV

For FIPS validation purposes, this adds a new function
_gnutls_cipher_get_iv() that exposes internal IV after encryption and
decryption.  The function is not generally useful because the IV value
can be easily calculated from the initial IV and the subsequent
ciphertext but for FIPS validation purposes.

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