Skip to content

examples: fix two issues found by CodeQL #18605

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
vszakats wants to merge 3 commits into from
Closed

examples: fix two issues found by CodeQL #18605

vszakats wants to merge 3 commits into from

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Sep 18, 2025 •
edited
Loading

  • http2-upload: use fstat() to query file length to fix TOCTOU.

  • ftpuploadresume: fix checking sscanf() return value.

Follow-up to b4922b1 #18564

@vszakats vszakats closed this in f13250e Sep 18, 2025
@vszakats vszakats deleted the cqlfix1 branch September 18, 2025 18:14
vszakats added a commit to vszakats/curl that referenced this pull request Sep 30, 2025
@vszakats
examples: fix two more cases of stat() TOCTOU
dd4e951 
Also:
- ftpupload: drop intermediate variable for file size.

Follow-up to f13250e curl#18605
@vszakats vszakats mentioned this pull request Sep 30, 2025
Closed
vszakats added a commit that referenced this pull request Sep 30, 2025
@vszakats
examples: fix two more cases of stat() TOCTOU
c478c7e 
Also:
- ftpupload: bump an intermediate variable size.

Follow-up to f13250e #18605

Closes #18778
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

documentation

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vszakats