saymrwulf/cryptography
1
0
Fork 0
mirror of https://github.com/saymrwulf/cryptography.git synced 2026-05-14 20:37:55 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Avoid allocating a Vec -- directly create a list (#10217)

Browse source
This commit is contained in:
Alex Gaynor 2024-01-21 05:40:11 -05:00 committed by GitHub
parent 39e3011170
commit 8d3b4b57bf
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/rust/src/x509/verify.rs
View file

@ -211,12 +211,12 @@ impl PyServerVerifier {
self.as_policy().max_chain_depth
}
fn verify(
fn verify<'p>(
&self,
py: pyo3::Python<'_>,
py: pyo3::Python<'p>,
leaf: pyo3::Py<PyCertificate>,
intermediates: Vec<pyo3::Py<PyCertificate>>,
) -> CryptographyResult<Vec<pyo3::Py<PyCertificate>>> {
) -> CryptographyResult<&'p pyo3::types::PyList> {
let policy = self.as_policy();
let store = self.store.get();
@ -236,7 +236,11 @@ impl PyServerVerifier {
)
.map_err(|e| VerificationError::new_err(format!("validation failed: {e:?}")))?;
Ok(chain.iter().map(|c| c.extra().clone_ref(py)).collect())
let result = pyo3::types::PyList::empty(py);
for c in chain {
result.append(c.extra())?;
}
Ok(result)
}
}