Skip to content

Commit

Permalink
Update and Fix NULL-dereference when loading PKCS7
Browse files Browse the repository at this point in the history
Calling `load_pem_pkcs7_certificates` or `load_der_pkcs7_certificates` could lead to a NULL-pointer dereference and segfault.

**PoC**
Here is a Python code that triggers the issue:
```python
from cryptography.hazmat.primitives.serialization.pkcs7 import load_der_pkcs7_certificates, load_pem_pkcs7_certificates

pem_p7 = b"""
-----BEGIN PKCS7-----
MAsGCSqGSIb3DQEHAg==
-----END PKCS7-----
"""

der_p7 = b"\x30\x0B\x06\x09\x2A\x86\x48\x86\xF7\x0D\x01\x07\x02"

load_pem_pkcs7_certificates(pem_p7)
load_der_pkcs7_certificates(der_p7)
```
Exploitation of this vulnerability poses a serious risk of Denial of Service (DoS) for any application attempting to deserialize a PKCS7 blob/certificate. The consequences extend to potential disruptions in system availability and stability.
  • Loading branch information
wica-sufatmawati authored Dec 1, 2023
1 parent c536c9c commit 1ea50fc
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions scripts/tools/telink/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
cryptography==41.0.4
cryptography==41.0.6
cffi==1.15.0
future==0.18.3
pycparser==2.21
pypng==0.0.21
PyQRCode==1.2.1
PyQRCode==1.2.1

0 comments on commit 1ea50fc

Please sign in to comment.