Skip to content

config2setopts: acknowledge OOM error from CURLOPT_MIMEPOST#20375

Closed
bagder wants to merge 1 commit intomasterfrom
bagder/mimepost-oom
Closed

config2setopts: acknowledge OOM error from CURLOPT_MIMEPOST#20375
bagder wants to merge 1 commit intomasterfrom
bagder/mimepost-oom

Conversation

@bagder
Copy link
Member

@bagder bagder commented Jan 20, 2026

It should make curl exit. Found with "strict torture".

It should make curl exit. Found with "strict torture".
@bagder bagder marked this pull request as ready for review January 20, 2026 15:28
@bagder bagder requested a review from Copilot January 20, 2026 15:28
@bagder
Copy link
Member Author

bagder commented Jan 20, 2026

I vote we exclude RELEASE-NOTES from language checks...

RELEASE-NOTES:109: Buss ==> Bus

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a bug where an Out Of Memory (OOM) error from CURLOPT_MIMEPOST was not being acknowledged, preventing curl from exiting properly when such errors occur. The issue was found through "strict torture" testing.

Changes:

  • Modified line 856 to capture the return value from my_setopt_mimepost into the result variable

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bagder bagder closed this in f876984 Jan 20, 2026
@bagder bagder deleted the bagder/mimepost-oom branch February 11, 2026 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants