diff options
| -rw-r--r-- | meta/recipes-support/curl/curl/CVE-2024-8096.patch | 207 | ||||
| -rw-r--r-- | meta/recipes-support/curl/curl_8.7.1.bb | 1 |
2 files changed, 208 insertions, 0 deletions
diff --git a/meta/recipes-support/curl/curl/CVE-2024-8096.patch b/meta/recipes-support/curl/curl/CVE-2024-8096.patch new file mode 100644 index 0000000000..a26a6253c9 --- /dev/null +++ b/meta/recipes-support/curl/curl/CVE-2024-8096.patch | |||
| @@ -0,0 +1,207 @@ | |||
| 1 | From aeb1a281cab13c7ba791cb104e556b20e713941f Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Daniel Stenberg <daniel@haxx.se> | ||
| 3 | Date: Tue, 20 Aug 2024 16:14:39 +0200 | ||
| 4 | Subject: [PATCH] gtls: fix OCSP stapling management | ||
| 5 | |||
| 6 | Reported-by: Hiroki Kurosawa | ||
| 7 | Closes #14642 | ||
| 8 | |||
| 9 | Upstream-Status: Backport [https://github.com/curl/curl/commit/aeb1a281cab13c7ba791cb104e556b20e713941f] | ||
| 10 | CVE: CVE-2024-8096 | ||
| 11 | Signed-off-by: Hitendra Prajapati <hprajapati@mvista.com> | ||
| 12 | --- | ||
| 13 | lib/vtls/gtls.c | 146 ++++++++++++++++++++++++------------------------ | ||
| 14 | 1 file changed, 73 insertions(+), 73 deletions(-) | ||
| 15 | |||
| 16 | diff --git a/lib/vtls/gtls.c b/lib/vtls/gtls.c | ||
| 17 | index 6eaa6a8..7dd7df8 100644 | ||
| 18 | --- a/lib/vtls/gtls.c | ||
| 19 | +++ b/lib/vtls/gtls.c | ||
| 20 | @@ -538,6 +538,13 @@ CURLcode gtls_client_init(struct Curl_easy *data, | ||
| 21 | init_flags |= GNUTLS_NO_TICKETS; | ||
| 22 | #endif | ||
| 23 | |||
| 24 | +#if defined(GNUTLS_NO_STATUS_REQUEST) | ||
| 25 | + if(!config->verifystatus) | ||
| 26 | + /* Disable the "status_request" TLS extension, enabled by default since | ||
| 27 | + GnuTLS 3.8.0. */ | ||
| 28 | + init_flags |= GNUTLS_NO_STATUS_REQUEST; | ||
| 29 | +#endif | ||
| 30 | + | ||
| 31 | rc = gnutls_init(>ls->session, init_flags); | ||
| 32 | if(rc != GNUTLS_E_SUCCESS) { | ||
| 33 | failf(data, "gnutls_init() failed: %d", rc); | ||
| 34 | @@ -923,104 +930,97 @@ Curl_gtls_verifyserver(struct Curl_easy *data, | ||
| 35 | infof(data, " server certificate verification SKIPPED"); | ||
| 36 | |||
| 37 | if(config->verifystatus) { | ||
| 38 | - if(gnutls_ocsp_status_request_is_checked(session, 0) == 0) { | ||
| 39 | - gnutls_datum_t status_request; | ||
| 40 | - gnutls_ocsp_resp_t ocsp_resp; | ||
| 41 | + gnutls_datum_t status_request; | ||
| 42 | + gnutls_ocsp_resp_t ocsp_resp; | ||
| 43 | + gnutls_ocsp_cert_status_t status; | ||
| 44 | + gnutls_x509_crl_reason_t reason; | ||
| 45 | |||
| 46 | - gnutls_ocsp_cert_status_t status; | ||
| 47 | - gnutls_x509_crl_reason_t reason; | ||
| 48 | + rc = gnutls_ocsp_status_request_get(session, &status_request); | ||
| 49 | |||
| 50 | - rc = gnutls_ocsp_status_request_get(session, &status_request); | ||
| 51 | + if(rc == GNUTLS_E_REQUESTED_DATA_NOT_AVAILABLE) { | ||
| 52 | + failf(data, "No OCSP response received"); | ||
| 53 | + return CURLE_SSL_INVALIDCERTSTATUS; | ||
| 54 | + } | ||
| 55 | |||
| 56 | - infof(data, " server certificate status verification FAILED"); | ||
| 57 | + if(rc < 0) { | ||
| 58 | + failf(data, "Invalid OCSP response received"); | ||
| 59 | + return CURLE_SSL_INVALIDCERTSTATUS; | ||
| 60 | + } | ||
| 61 | |||
| 62 | - if(rc == GNUTLS_E_REQUESTED_DATA_NOT_AVAILABLE) { | ||
| 63 | - failf(data, "No OCSP response received"); | ||
| 64 | - return CURLE_SSL_INVALIDCERTSTATUS; | ||
| 65 | - } | ||
| 66 | + gnutls_ocsp_resp_init(&ocsp_resp); | ||
| 67 | |||
| 68 | - if(rc < 0) { | ||
| 69 | - failf(data, "Invalid OCSP response received"); | ||
| 70 | - return CURLE_SSL_INVALIDCERTSTATUS; | ||
| 71 | - } | ||
| 72 | + rc = gnutls_ocsp_resp_import(ocsp_resp, &status_request); | ||
| 73 | + if(rc < 0) { | ||
| 74 | + failf(data, "Invalid OCSP response received"); | ||
| 75 | + return CURLE_SSL_INVALIDCERTSTATUS; | ||
| 76 | + } | ||
| 77 | |||
| 78 | - gnutls_ocsp_resp_init(&ocsp_resp); | ||
| 79 | + (void)gnutls_ocsp_resp_get_single(ocsp_resp, 0, NULL, NULL, NULL, NULL, | ||
| 80 | + &status, NULL, NULL, NULL, &reason); | ||
| 81 | |||
| 82 | - rc = gnutls_ocsp_resp_import(ocsp_resp, &status_request); | ||
| 83 | - if(rc < 0) { | ||
| 84 | - failf(data, "Invalid OCSP response received"); | ||
| 85 | - return CURLE_SSL_INVALIDCERTSTATUS; | ||
| 86 | - } | ||
| 87 | + switch(status) { | ||
| 88 | + case GNUTLS_OCSP_CERT_GOOD: | ||
| 89 | + break; | ||
| 90 | |||
| 91 | - (void)gnutls_ocsp_resp_get_single(ocsp_resp, 0, NULL, NULL, NULL, NULL, | ||
| 92 | - &status, NULL, NULL, NULL, &reason); | ||
| 93 | + case GNUTLS_OCSP_CERT_REVOKED: { | ||
| 94 | + const char *crl_reason; | ||
| 95 | |||
| 96 | - switch(status) { | ||
| 97 | - case GNUTLS_OCSP_CERT_GOOD: | ||
| 98 | + switch(reason) { | ||
| 99 | + default: | ||
| 100 | + case GNUTLS_X509_CRLREASON_UNSPECIFIED: | ||
| 101 | + crl_reason = "unspecified reason"; | ||
| 102 | break; | ||
| 103 | |||
| 104 | - case GNUTLS_OCSP_CERT_REVOKED: { | ||
| 105 | - const char *crl_reason; | ||
| 106 | - | ||
| 107 | - switch(reason) { | ||
| 108 | - default: | ||
| 109 | - case GNUTLS_X509_CRLREASON_UNSPECIFIED: | ||
| 110 | - crl_reason = "unspecified reason"; | ||
| 111 | - break; | ||
| 112 | - | ||
| 113 | - case GNUTLS_X509_CRLREASON_KEYCOMPROMISE: | ||
| 114 | - crl_reason = "private key compromised"; | ||
| 115 | - break; | ||
| 116 | - | ||
| 117 | - case GNUTLS_X509_CRLREASON_CACOMPROMISE: | ||
| 118 | - crl_reason = "CA compromised"; | ||
| 119 | - break; | ||
| 120 | - | ||
| 121 | - case GNUTLS_X509_CRLREASON_AFFILIATIONCHANGED: | ||
| 122 | - crl_reason = "affiliation has changed"; | ||
| 123 | - break; | ||
| 124 | + case GNUTLS_X509_CRLREASON_KEYCOMPROMISE: | ||
| 125 | + crl_reason = "private key compromised"; | ||
| 126 | + break; | ||
| 127 | |||
| 128 | - case GNUTLS_X509_CRLREASON_SUPERSEDED: | ||
| 129 | - crl_reason = "certificate superseded"; | ||
| 130 | - break; | ||
| 131 | + case GNUTLS_X509_CRLREASON_CACOMPROMISE: | ||
| 132 | + crl_reason = "CA compromised"; | ||
| 133 | + break; | ||
| 134 | |||
| 135 | - case GNUTLS_X509_CRLREASON_CESSATIONOFOPERATION: | ||
| 136 | - crl_reason = "operation has ceased"; | ||
| 137 | - break; | ||
| 138 | + case GNUTLS_X509_CRLREASON_AFFILIATIONCHANGED: | ||
| 139 | + crl_reason = "affiliation has changed"; | ||
| 140 | + break; | ||
| 141 | |||
| 142 | - case GNUTLS_X509_CRLREASON_CERTIFICATEHOLD: | ||
| 143 | - crl_reason = "certificate is on hold"; | ||
| 144 | - break; | ||
| 145 | + case GNUTLS_X509_CRLREASON_SUPERSEDED: | ||
| 146 | + crl_reason = "certificate superseded"; | ||
| 147 | + break; | ||
| 148 | |||
| 149 | - case GNUTLS_X509_CRLREASON_REMOVEFROMCRL: | ||
| 150 | - crl_reason = "will be removed from delta CRL"; | ||
| 151 | - break; | ||
| 152 | + case GNUTLS_X509_CRLREASON_CESSATIONOFOPERATION: | ||
| 153 | + crl_reason = "operation has ceased"; | ||
| 154 | + break; | ||
| 155 | |||
| 156 | - case GNUTLS_X509_CRLREASON_PRIVILEGEWITHDRAWN: | ||
| 157 | - crl_reason = "privilege withdrawn"; | ||
| 158 | - break; | ||
| 159 | + case GNUTLS_X509_CRLREASON_CERTIFICATEHOLD: | ||
| 160 | + crl_reason = "certificate is on hold"; | ||
| 161 | + break; | ||
| 162 | |||
| 163 | - case GNUTLS_X509_CRLREASON_AACOMPROMISE: | ||
| 164 | - crl_reason = "AA compromised"; | ||
| 165 | - break; | ||
| 166 | - } | ||
| 167 | + case GNUTLS_X509_CRLREASON_REMOVEFROMCRL: | ||
| 168 | + crl_reason = "will be removed from delta CRL"; | ||
| 169 | + break; | ||
| 170 | |||
| 171 | - failf(data, "Server certificate was revoked: %s", crl_reason); | ||
| 172 | + case GNUTLS_X509_CRLREASON_PRIVILEGEWITHDRAWN: | ||
| 173 | + crl_reason = "privilege withdrawn"; | ||
| 174 | break; | ||
| 175 | - } | ||
| 176 | |||
| 177 | - default: | ||
| 178 | - case GNUTLS_OCSP_CERT_UNKNOWN: | ||
| 179 | - failf(data, "Server certificate status is unknown"); | ||
| 180 | + case GNUTLS_X509_CRLREASON_AACOMPROMISE: | ||
| 181 | + crl_reason = "AA compromised"; | ||
| 182 | break; | ||
| 183 | } | ||
| 184 | |||
| 185 | - gnutls_ocsp_resp_deinit(ocsp_resp); | ||
| 186 | + failf(data, "Server certificate was revoked: %s", crl_reason); | ||
| 187 | + break; | ||
| 188 | + } | ||
| 189 | + | ||
| 190 | + default: | ||
| 191 | + case GNUTLS_OCSP_CERT_UNKNOWN: | ||
| 192 | + failf(data, "Server certificate status is unknown"); | ||
| 193 | + break; | ||
| 194 | + } | ||
| 195 | |||
| 196 | + gnutls_ocsp_resp_deinit(ocsp_resp); | ||
| 197 | + if(status != GNUTLS_OCSP_CERT_GOOD) | ||
| 198 | return CURLE_SSL_INVALIDCERTSTATUS; | ||
| 199 | - } | ||
| 200 | - else | ||
| 201 | - infof(data, " server certificate status verification OK"); | ||
| 202 | } | ||
| 203 | else | ||
| 204 | infof(data, " server certificate status verification SKIPPED"); | ||
| 205 | -- | ||
| 206 | 2.25.1 | ||
| 207 | |||
diff --git a/meta/recipes-support/curl/curl_8.7.1.bb b/meta/recipes-support/curl/curl_8.7.1.bb index 5442d8d4fd..d094604ea1 100644 --- a/meta/recipes-support/curl/curl_8.7.1.bb +++ b/meta/recipes-support/curl/curl_8.7.1.bb | |||
| @@ -18,6 +18,7 @@ SRC_URI = " \ | |||
| 18 | file://CVE-2024-6197.patch \ | 18 | file://CVE-2024-6197.patch \ |
| 19 | file://CVE-2024-7264-1.patch \ | 19 | file://CVE-2024-7264-1.patch \ |
| 20 | file://CVE-2024-7264-2.patch \ | 20 | file://CVE-2024-7264-2.patch \ |
| 21 | file://CVE-2024-8096.patch \ | ||
| 21 | " | 22 | " |
| 22 | SRC_URI[sha256sum] = "6fea2aac6a4610fbd0400afb0bcddbe7258a64c63f1f68e5855ebc0c659710cd" | 23 | SRC_URI[sha256sum] = "6fea2aac6a4610fbd0400afb0bcddbe7258a64c63f1f68e5855ebc0c659710cd" |
| 23 | 24 | ||
