summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* tests: added CRL verification teststmp-crlNikos Mavrogiannopoulos2018-09-166-1/+92
| | | | | | | | | This tests CRL verification with certtool --verify-crl on correct and incorrect cases. Relates #564 Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* certtool: updates in documentation in messages for CRL generationNikos Mavrogiannopoulos2018-09-163-7/+12
| | | | | | | | This fixes the messages printed for the generation of a CRL, and makes the return code of the CRL verification depending on the verification result. Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* Fix variable used in reallocationNikos Mavrogiannopoulos2018-09-141-1/+1
| | | | | | | | | | This corrects the variable name used in the sizeof argument for realloc. This does not alter the actual allocation size, but rather it fixes a logic error. Relates: #554 Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* .gitignore: updatedNikos Mavrogiannopoulos2018-09-141-1/+16
| | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* dtls: recover when a NewSessionTicket message is lostNikos Mavrogiannopoulos2018-09-143-7/+264
| | | | | | | | | | | When the server's NewSessionTicket gets lost while the ChangeCipherSpec goes through, the client did not request retransmission by retransmitting his last flight, and the handshake was blocked. This commit addresses the issue and adds a reproducer. Resolves #543 Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* tlsfuzzer: remove duplicate tests and sort them alphabeticallyDaiki Ueno2018-09-141-28/+20
| | | | Signed-off-by: Daiki Ueno <dueno@redhat.com>
* doc: fix reference to invocation nodesAndreas Schwab2018-09-141-6/+6
| | | | Signed-off-by: Andreas Schwab <schwab@suse.de>
* priority: be backwards compatible with priority strings starting with NONENikos Mavrogiannopoulos2018-09-126-15/+39
| | | | | | | | | | | That is, we allow priority strings which do not enable any groups to work, by disabling TLS1.3. For example 'NONE:+VERS-TLS-ALL:+MAC-ALL:+RSA:+AES-128-GCM:+SIGN-ALL:+COMP-NULL' is still operational, but no TLS1.3 is enabled when specified. Resolves: #549 Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* Merge branch 'tmp-strdup' into 'master'Tim Rühsen2018-08-303-9/+3
|\ | | | | | | | | Use gnutls_strdup() in library code See merge request gnutls/gnutls!742
| * Use gnutls_strdup() instead of strdup() in library codetmp-strdupTim Rühsen2018-08-242-3/+3
| | | | | | | | Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>
| * Remove gnulib work-around '#undef strdup'Tim Rühsen2018-08-241-6/+0
|/ | | | | | | The 'issue' should be fixed already. Even if not, it has to addressed in gnulib. Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>
* Merge branch 'tmp-psk-ext-refactor' into 'master'Daiki Ueno2018-08-239-139/+150
|\ | | | | | | | | tls13/psk_ext_parser: simplify the iterator interface See merge request gnutls/gnutls!736
| * ext/pre_shared_key: use consistent name for regitration entryDaiki Ueno2018-08-235-6/+6
| | | | | | | | Signed-off-by: Daiki Ueno <dueno@redhat.com>
| * ext/pre_shared_key: make ticket age calculation consistentDaiki Ueno2018-08-231-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously we used a pattern like this: uint32_t obfuscated_ticket_age, ticket_age_add; time_t ticket_age; ticket_age = obfuscated_ticket_age - ticket_age_add; if (ticket_age < 0) { ... } This always evaluates to false, because subtraction between unsigned integers yields an unsigned integer. Let's do the comparison before subtraction and also use correct types for representing time: uint32_t for protocol time and time_t for system time. Signed-off-by: Daiki Ueno <dueno@redhat.com>
| * tls13/psk_ext_parser: simplify the iterator interfaceDaiki Ueno2018-08-235-123/+134
|/ | | | | | | | Previously it was unclear whether psk_ext_parser_st is stateful or not. This change introduces the simpler API to iterate over the immutable data (psk_ext_parser_st), following the iterator pattern. Signed-off-by: Daiki Ueno <dueno@redhat.com>
* gnutls-cli-debug: mention RFC8446 for TLS1.3 and RFC8422 for X25519Nikos Mavrogiannopoulos2018-08-223-4/+3
| | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* Merge branch 'tmp-gnulib-use-git' into 'master'Nikos Mavrogiannopoulos2018-08-221-17/+17
|\ | | | | | | | | Remove --no-git from ./bootstrap See merge request gnutls/gnutls!739
| * Remove --no-git from ./bootstrap [ci skip]tmp-gnulib-use-gitTim Rühsen2018-08-211-17/+17
|/ | | | | | | | This removes the --no-git option as bootstrap itself does not use the remote repository for cloning. At least as long $GNULIB_SRCDIR points to a recent enough local gnulib git repo. Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>
* Merge branch 'tmp-update-version-numbers' into 'master'Nikos Mavrogiannopoulos2018-08-2117-80/+65
|\ | | | | | | | | | | | | Update library to use the final RFC8446 version numbers Closes #542 and #359 See merge request gnutls/gnutls!730
| * handshake: do not negotiate TLS1.3 using the legacy version fieldtmp-update-version-numbersNikos Mavrogiannopoulos2018-08-204-8/+25
| | | | | | | | | | | | | | | | | | Previously we could end-up with a TLS1.3 connection if the TLS1.3 ID was seen on the wire. We now explicitly fallback to TLS1.2 when we see a protocol with TLS1.3 semantics in an SSL2.0 or in the legacy version of the client hello. Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
| * handshake: simplified protocol version checking functionsNikos Mavrogiannopoulos2018-08-206-49/+39
| | | | | | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
| * tlsfuzzer: modify to use the final code pointsNikos Mavrogiannopoulos2018-08-201-0/+2
| | | | | | | | | | | | Relates #542 Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
| * fuzz: updated traces for final TLS1.3 versionNikos Mavrogiannopoulos2018-08-206-0/+0
| | | | | | | | | | | | Resolves #359 Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
| * protocols: bumped TLS1.3 version number to RFC8446 valueNikos Mavrogiannopoulos2018-08-205-30/+6
|/ | | | | | | | This adds support of the final RFC numbers. Resolves #542 Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* Merge branch 'rfc7250_cert-types' into 'master'Nikos Mavrogiannopoulos2018-08-2034-110/+1895
|\ | | | | | | | | RFC7250 certificate type negotiation See merge request gnutls/gnutls!498
| * Implemented RFC7250 certificate type negotiation extensions.Tom Vrancken2018-08-2034-110/+1895
|/ | | | Signed-off-by: Tom Vrancken <dev@tomvrancken.nl>
* Merge branch 'tmp-record-size-limit' into 'master'Daiki Ueno2018-08-2011-91/+623
|\ | | | | | | | | | | | | record_size_limit extension Closes #524 See merge request gnutls/gnutls!733
| * ext/record_size_limit: new extensionDaiki Ueno2018-08-2011-31/+619
| | | | | | | | | | | | | | | | | | | | | | This implements the record_size_limit extension as defined in RFC 8449. Although it obsoletes the max_record_size extension, for compatibility reasons GnuTLS still sends it on certain occasions. For example, when the new size is representable as the codepoint defined for max_record_size. Signed-off-by: Daiki Ueno <dueno@redhat.com>
| * ext/max_record: remove use of extension private dataDaiki Ueno2018-08-201-67/+11
|/ | | | | | | | As the extension data is always stored in session->security_parameters.max_record_send_size, it shouldn't be necessary to track it with the private data. Signed-off-by: Daiki Ueno <dueno@redhat.com>
* Merge branch 'tmp-fix-resumption-requested' into 'master'Nikos Mavrogiannopoulos2018-08-203-1/+12
|\ | | | | | | | | | | | | Fix gnutls_session_resumption_requested Closes #546 See merge request gnutls/gnutls!735
| * gnutls_session_resumption_requested: fixed behavior under TLS1.3Nikos Mavrogiannopoulos2018-08-203-1/+12
|/ | | | | | | | | This makes gnutls_session_resumption_requested() functional under TLS1.3 and introduces a unit test of the function. Resolves #546 Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* Merge branch 'tmp-no-git-in-bootstrap' into 'master'Tim Rühsen2018-08-201-18/+21
|\ | | | | | | | | | | | | .gitlab-ci.yml: use --no-git to bootstrap Closes #547 See merge request gnutls/gnutls!737
| * .gitlab-ci.yml: use --no-git to bootstraptmp-no-git-in-bootstrapNikos Mavrogiannopoulos2018-08-191-18/+21
|/ | | | | | | | | That is, to reduce CI time, and avoid failures due to non-availability of the gnulib git repo. Resolves #547 Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* hello_ext: removed bogus commentNikos Mavrogiannopoulos2018-08-191-2/+0
| | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* .gitmodules: gnulib submodule is now synced from libidn's mirrorNikos Mavrogiannopoulos2018-08-191-1/+1
| | | | | | | | | This mirror is updated hourly and is hosted on gitlab, meaning less dependency on external sites downtime. Resolves: #547 Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* Merge branch 'tmp-ametzler-typo-guarranteed_overriden' into 'master'Tim Rühsen2018-08-1813-33/+33
|\ | | | | | | | | Fix two typos (overriden/guarranteed) See merge request gnutls/gnutls!734
| * Fix two typos (overriden/guarranteed)Andreas Metzler2018-08-1613-33/+33
|/ | | | Signed-off-by: Andreas Metzler <ametzler@bebt.de>
* Merge branch 'tmp-document-none' into 'master'Nikos Mavrogiannopoulos2018-08-151-1/+8
|\ | | | | | | | | doc: document the non-portability of NONE priority string See merge request gnutls/gnutls!731
| * doc: document the non-portability of NONE priority stringNikos Mavrogiannopoulos2018-08-151-1/+8
|/ | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* Merge branch 'tmp-fixes-cov' into 'master'Nikos Mavrogiannopoulos2018-08-1411-28/+69
|\ | | | | | | | | | | | | Fixes for issues identified by static analyzers Closes #518 See merge request gnutls/gnutls!729
| * tools: check output of called functionsNikos Mavrogiannopoulos2018-08-136-14/+58
| | | | | | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
| * write_oid_and_params: moved nullity check of params earlierNikos Mavrogiannopoulos2018-08-131-6/+5
| | | | | | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
| * gnutls_session_set_premaster: corrected error checksNikos Mavrogiannopoulos2018-08-131-5/+4
| | | | | | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
| * pubkey_verify_hashed_data: apply algorithm checksNikos Mavrogiannopoulos2018-08-131-2/+0
| | | | | | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
| * privkey_sign_raw_data: use assert to mark code which always succeedsNikos Mavrogiannopoulos2018-08-131-0/+1
| | | | | | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
| * _gnutls_send_change_cipher_spec: removed unnecessary testNikos Mavrogiannopoulos2018-08-131-1/+1
|/ | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* .travis.yml: do not run brew upgradeNikos Mavrogiannopoulos2018-08-131-2/+6
| | | | | | This addresses issue with travis compilation on MacOSX. Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* Merge branch 'tmp-use-explicit-bzero' into 'master'Nikos Mavrogiannopoulos2018-08-084-12/+20
|\ | | | | | | | | | | | | gnutls_memset: use explicit_bzero Closes #230 See merge request gnutls/gnutls!728
| * gnutls_memset: use explicit_bzeroNikos Mavrogiannopoulos2018-08-084-12/+20
|/ | | | | | | | | That is, use the glibc function when available and the second parameter is zero. Resolves #230 Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* Merge branch 'tmp-consistent-falltrough' into 'master'Nikos Mavrogiannopoulos2018-08-0814-99/+144
|\ | | | | | | | | | | | | use a consistent method to mark fall-through in switch cases Closes #306 See merge request gnutls/gnutls!726