summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* CONTRIBUTING.md: document unit testing method of internal functions [ci skip]tmp-include-unit-testing-docNikos Mavrogiannopoulos2019-04-071-3/+5
| | | | | | Resolves: #749 Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* Merge branch 'tmp-client-auth-decline' into 'master'Daiki Ueno2019-04-078-29/+629
|\ | | | | | | | | | | | | cert auth: reject auth if no signature algorithm is usable in TLS 1.3 Closes #730 See merge request gnutls/gnutls!967
| * tests: add post-handshake auth test using PKCS#11 tokenDaiki Ueno2019-04-073-0/+468
| | | | | | | | | | | | | | This adds a test that exercise the client's auth rejection logic, using the RSA-PSS disabled PKCS #11 token. Signed-off-by: Daiki Ueno <dueno@redhat.com>
| * tests: add mock PKCS#11 module disabling RSA-PSSDaiki Ueno2019-04-072-0/+114
| | | | | | | | | | | | | | | | | | This adds libpkcs11mock2.so, which wraps SoftHSM but filters out the use of the CKM_RSA_PKCS_PSS mechanism. That way we can simulate the situation where the certificate is RSA while the private key cannot be used for RSA-PSS. Signed-off-by: Daiki Ueno <dueno@redhat.com>
| * cert auth: reject auth if no signature algorithm is usable in TLS 1.3Daiki Ueno2019-04-043-23/+42
| | | | | | | | | | | | | | | | | | | | Previously, when there is no overlap between usable signature algorithms and the "signature_algorithms" extension in Certificate Request, the client failed in sending Certificate Verify, followed by a connection close. In TLS 1.3, it is possible to keep the connection but reject the authentication by not sending Certificate Verify. Signed-off-by: Daiki Ueno <dueno@redhat.com>
| * handshake: remove unnecessary HSK_CRT_SENT flagDaiki Ueno2019-04-032-6/+5
| | | | | | | | | | | | | | Previously, while the flag HSK_CRT_SENT was checked in _gnutls13_send_certificate_verify, the flag was never set anywhere. Signed-off-by: Daiki Ueno <dueno@redhat.com>
* | Merge branch 'tmp-cmac-cfb8-fix' into 'master'Tim Rühsen2019-04-063-1/+9
|\ \ | | | | | | | | | | | | nettle: include config.h before checking for definitions See merge request gnutls/gnutls!970
| * | nettle: include config.h before checking for definitionstmp-cmac-cfb8-fixNikos Mavrogiannopoulos2019-04-063-1/+9
|/ / | | | | | | | | | | | | | | | | This makes sure that we don't include the internal backport if compiled with a version of nettle that includes that code. We also exclude nettle/backport from the static analyzer's list as it contains files outside our control (from nettle project). Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* | Merge branch 'master' into 'master'Tim Rühsen2019-04-051-0/+3
|\ \ | | | | | | | | | | | | [OSCP] Fix : null pointer resp See merge request gnutls/gnutls!969
| * | [OSCP] Fix : null pointer respElta Koepp2019-04-051-0/+3
| | | | | | | | | | | | Signed-off-by: Elta Koepp <elta_koepp@gmail.com>
* | | gnutls_memset(): calling explicit_bzero() is enough to zero-fill a bufferMaciej S. Szmigiero2019-04-051-0/+1
|/ / | | | | | | | | | | | | If we use explicit_bzero() to zero-fill a buffer in gnutls_memset() we don't need to zero it again via a volatile trick later in this function. Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>
* | .gitlab-ci.yml: do not run commit-check on master branchNikos Mavrogiannopoulos2019-04-031-0/+2
| | | | | | | | | | | | That is, because there are no diffs to check. Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* | Merge branch 'check_if_signed' into 'master'Nikos Mavrogiannopoulos2019-04-021-10/+15
|\ \ | |/ |/| | | | | Fix check_if_signed See merge request gnutls/gnutls!964
| * Fix check_if_signedSimo Sorce2019-04-011-10/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | Fix the target branch we check against by adding upstream as remote. Drop the use of set -e as this causes the shell to immediately exit on errors instead of allowing the code to check the failure and report what it faled about. Also print which commits are being checked and what information was found so that a CI failure can be better diagnosed. Signed-off-by: Simo Sorce <simo@redhat.com>
* | doc update [ci skip]Nikos Mavrogiannopoulos2019-03-301-0/+8
| | | | | | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* | Vendor in XTS functionality from NettleSimo Sorce2019-03-2910-0/+536
|/ | | | | | | | If nettle's XTS is not available, use a vendored in version from master. This is necessary as long as we need to link against 3.4 for ABI compatibility reasons. Signed-off-by: Simo Sorce <simo@redhat.com>
* fuzz: improvements in gnutls_x509_verify_fuzzer [ci skip]Nikos Mavrogiannopoulos2019-03-29431-4/+68
| | | | | | | Added a larger set of corpus (generated with afl-fuzz), and made sure that the fuzzer application crashes if verification succeeds. Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* Merge branch 'tmp-fail-sigcheck' into 'master'Tim Rühsen2019-03-282-3/+5
|\ | | | | | | | | Let check_if_signed fail if git fails See merge request gnutls/gnutls!962
| * Let check_if_signed fail if git failstmp-fail-sigcheckTim Rühsen2019-03-282-3/+5
|/ | | | Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>
* Merge branch 'master' into 'master'Tim Rühsen2019-03-271-0/+3
|\ | | | | | | | | Detect malloc failure. See merge request gnutls/gnutls!960
| * Update ocsptool-common.cElta Koepp2019-03-271-2/+1
| |
| * Detect malloc failure.Elta Koepp2019-03-271-0/+4
|/ | | malloc(data.size + 1) maybe returns NULL on failure.
* released 3.6.7gnutls_3_6_7Nikos Mavrogiannopoulos2019-03-271-8/+15
| | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* handshake: add missing initialization of local variableDaiki Ueno2019-03-271-0/+2
| | | | | | | Resolves: #704 Signed-off-by: Daiki Ueno <dueno@redhat.com> Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* fuzz: added fuzzer for certificate verificationNikos Mavrogiannopoulos2019-03-278-2/+286
| | | | | | | | This also adds a reproducer for CVE-2019-3829. Resolves: #694 Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* Merge branch 'sha3_selftests' into 'master'Nikos Mavrogiannopoulos2019-03-261-0/+25
|\ | | | | | | | | fips140: Perform SHA-3 self tests See merge request gnutls/gnutls!958
| * fips140: Perform SHA-3 self testsAnderson Toshiyuki Sasaki2019-03-261-0/+25
| | | | | | | | | | | | | | It is required to perform the self tests to validate SHA-3 implementation. Signed-off-by: Anderson Toshiyuki Sasaki <ansasaki@redhat.com>
* | bumped versionNikos Mavrogiannopoulos2019-03-262-2/+2
| | | | | | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* | Merge branch 'tmp-increase-nr-of-tickets' into 'master'Nikos Mavrogiannopoulos2019-03-269-35/+67
|\ \ | |/ |/| | | | | | | | | handshake: increase the default number of tickets we send to 2 Closes #596 See merge request gnutls/gnutls!942
| * handshake: increase the default number of tickets we send to 2tmp-increase-nr-of-ticketsNikos Mavrogiannopoulos2019-03-209-35/+67
| | | | | | | | | | | | | | | | | | | | | | This makes it easier for clients which perform multiple connections to the server to use the tickets sent by a default server. That's because 2 tickets allow for 2 new connections (if one is using each ticket once as recommended), which in turn lead to 4 new and so on. Resolves: #596 Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* | Merge branch 'tmp-security-update' into 'master'Dmitry Eremin-Solenikov2019-03-261-5/+5
|\ \ | | | | | | | | | | | | SECURITY.md: updated to reflect the current practice See merge request gnutls/gnutls!951
| * | SECURITY.md: updated to reflect the current practice [ci skip]tmp-security-updateNikos Mavrogiannopoulos2019-03-091-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change updates the SECURITY guidelines to reflect the current practice (no special security releases), and thus refer directly to the upcoming or following release. Furthermore, it removes any mention of absolute time, as the release cadence is already fixed to bi-monthly. Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* | | Merge branch 'guile' into 'master'Nikos Mavrogiannopoulos2019-03-241-22/+1
|\ \ \ | | | | | | | | | | | | | | | | configure.ac: remove --with-guile-site-dir See merge request gnutls/gnutls!957
| * | | configure.ac: remove --with-guile-site-dirAlon Bar-Lev2019-03-231-22/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The hack of distcheck is not known and should not be the default as the GUILE_SITE_DIR macro is the default expected behavior. There is little value in specifying any other location of the site-dir as it is out of the guile configuration so best to remove. Signed-off-by: Alon Bar-Lev <alon.barlev@gmail.com>
* | | | tools: removed unused codeNikos Mavrogiannopoulos2019-03-242-58/+0
| | | | | | | | | | | | | | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* | | | gnutls-cli: Fix output with option "--logfile"Ke Zhao2019-03-242-10/+64
|/ / / | | | | | | | | | | | | | | | | | | | | | The X.509 connection would still print informational message to the stdout by default. Move that output to logfile and add x509 functionality test in the test suite. Signed-off-by: Ke Zhao <kzhao@redhat.com>
* | | Merge branch 'tmp-improve-session-resumption' into 'master'Nikos Mavrogiannopoulos2019-03-227-11/+139
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Improved estimation of wait in gnutls_session_get_data2 Closes #706 See merge request gnutls/gnutls!936
| * | | Improved estimation of wait in gnutls_session_get_data2tmp-improve-session-resumptionNikos Mavrogiannopoulos2019-03-197-11/+139
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously we would wait an arbitrary value of 50ms for the server to send session tickets. This change makes the client wait for the estimated single trip time + 60 ms for the server to calculate the session tickets. This improves the chance to obtain tickets from internet servers during the call of gnutls_session_get_data2(). Resolves: #706 Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* | | | _x509_en/decode_provable_seed: clarified purpose of functions [ci skip]Nikos Mavrogiannopoulos2019-03-201-0/+6
| |_|/ |/| | | | | | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* | | Merge branch 'tmp-fix-pkcs11-so' into 'master'Nikos Mavrogiannopoulos2019-03-193-1/+25
|\ \ \ | |/ / |/| | | | | | | | | | | | | | pkcs11: security officer login implies writable session Closes #721 See merge request gnutls/gnutls!953
| * | pkcs11: security officer login implies writable sessiontmp-fix-pkcs11-soNikos Mavrogiannopoulos2019-03-153-1/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | According to the PKCS#11 v2.30, 6.7.1 there are no read-only Security Officer sessions. Resolves: #721 Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
* | | Merge branch 'tmp-remove-fixme' into 'master'Tim Rühsen2019-03-1613-35/+14
|\ \ \ | | | | | | | | | | | | | | | | Removed all FIXME comments in code See merge request gnutls/gnutls!955
| * | | Removed all FIXME comments in code [ci skip]Nikos Mavrogiannopoulos2019-03-1613-35/+14
| |/ / | | | | | | | | | | | | | | | | | | | | | We expand informational comments on limitations, but with removing FIXME (keyword didn't help fixing these), and remove completely unhelpful comments, obsolete ones, or comments about ideas. Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* | | doc updateNikos Mavrogiannopoulos2019-03-161-0/+2
| | | | | | | | | | | | Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
* | | gnutls-cli: Add option "--logfile" to redirect information message outputKe Zhao2019-03-167-187/+346
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | First, add an option "--logfile" so user could choose a specific file to store all the informational messages. In some cases, informational messages may cause unexpected result if the output is standard output. With this option, user could redirect these messages to a specific file. This will be helpful in testing and tracking. Second, replace printf() function with log_msg() function This log_msg() function is used when "--logfile" is enabled. Third, add a functionality test for "--logfile" option Add a test script to test if "--logfile" option works as it should be. Signed-off-by: Ke Zhao <kzhao@redhat.com>
* | Merge branch 'tmp-use-https' into 'master'Tim Rühsen2019-03-13781-1066/+1065
|\ \ | | | | | | | | | | | | Change HTTP:// references to HTTPs:// (generally) See merge request gnutls/gnutls!910
| * | Update the GNU Free Documentation License (FDL)Tim Rühsen2019-03-131-8/+7
| | | | | | | | | | | | Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>
| * | Fix URL of ABI compliance checkerTim Rühsen2019-03-131-1/+1
| | | | | | | | | | | | Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>
| * | Fix URLs of p11-kitTim Rühsen2019-03-132-2/+2
| | | | | | | | | | | | Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>
| * | Use https:// in lib/, src/, and m4/Tim Rühsen2019-03-1318-29/+29
| | | | | | | | | | | | Signed-off-by: Tim Rühsen <tim.ruehsen@gmx.de>