Skip to content

x509asn1: fix to return error in an error case from encodeOID() - #20991

Closed
vszakats wants to merge 1 commit into
curl:masterfrom
vszakats:x509retcode
Closed

vszakats wants to merge 1 commit into
curl:masterfrom
vszakats:x509retcode

Conversation

@vszakats

Copy link
Copy Markdown
Member

Found by Codex Security

Follow-up to d8b0318 #3582

@github-actions github-actions Bot added the TLS label Mar 18, 2026
@bagder

bagder commented Mar 18, 2026

Copy link
Copy Markdown
Member

Seems this is a function we should write a unit test for!

@bagder

bagder commented Mar 18, 2026

Copy link
Copy Markdown
Member

@vszakats do you want me to have a go at it?

@vszakats

Copy link
Copy Markdown
Member Author

@vszakats do you want me to have a go at it?

Yes!

@bagder

bagder commented Mar 19, 2026

Copy link
Copy Markdown
Member

Will do. You can merge this and I'll work on a test separately.

@vszakats vszakats closed this in 86c25c0 Mar 19, 2026
@vszakats
vszakats deleted the x509retcode branch March 19, 2026 10:17
outcast36 pushed a commit to greearb/curl that referenced this pull request Jun 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants