Skip to content

getinfo: initialize PureInfo field used_proxy - #21020

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

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

Conversation

@vszakats

@vszakats vszakats commented Mar 20, 2026 •

Copy link
Copy Markdown
Member

Found by Codex Security

Follow-up to cc04c73 #12719

@vszakats vszakats changed the title getinfo: initalize PureInfo field used_proxy getinfo: initialize PureInfo field used_proxy Mar 20, 2026
@vszakats
vszakats requested a review from Copilot March 20, 2026 01:58

Copilot AI left a comment

Copy link
Copy Markdown

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 ensures the PureInfo.used_proxy field is properly reset at the start of a transfer/session, preventing stale state from being returned via CURLINFO_USED_PROXY.

Changes:

  • Initialize data->info.used_proxy in Curl_initinfo().

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

@vszakats vszakats closed this in 2cdb220 Mar 20, 2026
@vszakats
vszakats deleted the init_used_proxy branch March 20, 2026 12:47
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

Development

Successfully merging this pull request may close these issues.

2 participants