Skip to content

Add nullptr check in logCertVerificationFailure - #93

Merged
thisisnsh merged 2 commits into
mainfrom
fix-cert-verification-null-check
Aug 29, 2026
Merged

Add nullptr check in logCertVerificationFailure#93
thisisnsh merged 2 commits into
mainfrom
fix-cert-verification-null-check

Conversation

@thisisnsh

Copy link
Copy Markdown
Contributor

Problem statement

  • In logCertVerificationFailure following pointers were not null checked before accessing them:
    • X509_STORE_CTX cert context
    • X509 cert
    • X509_NAME subject name

Proposed changes

  • In logCertVerificationFailure check pointers are not null before accessing them
  • Add tests
  • Add error code, message and depth to log for better debugging

Remaining work

  • Unit Tests
  • Integration Tests
  • Documentation

@thisisnsh
thisisnsh enabled auto-merge (squash) August 29, 2026 00:45
@thisisnsh
thisisnsh disabled auto-merge August 29, 2026 02:36
@thisisnsh
thisisnsh merged commit a80a507 into main Aug 29, 2026
7 checks passed
@thisisnsh
thisisnsh deleted the fix-cert-verification-null-check branch August 29, 2026 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants