Skip to content

ftp+file, use stack buffer for resume handling#12789

Closed
icing wants to merge 5 commits intocurl:masterfrom
icing:file-statebuf
Closed

ftp+file, use stack buffer for resume handling#12789
icing wants to merge 5 commits intocurl:masterfrom
icing:file-statebuf

Conversation

@icing
Copy link
Copy Markdown
Contributor

@icing icing commented Jan 25, 2024

instead of data->state.buffer

@icing icing changed the title file, use stack scratch buffer for resume handling ftp, use stack scratch buffer for resume handling Jan 25, 2024
@icing icing changed the title ftp, use stack scratch buffer for resume handling ftp+file, use stack buffer for resume handling Jan 25, 2024
@bagder
Copy link
Copy Markdown
Member

bagder commented Jan 25, 2024

file.c still uses the state.buffer for downloads in file_do, did you mean to leave that part out?

@icing
Copy link
Copy Markdown
Contributor Author

icing commented Jan 26, 2024

file.c still uses the state.buffer for downloads in file_do, did you mean to leave that part out?

Added now.

@bagder bagder closed this in a6c9a33 Jan 26, 2024
bagder added a commit that referenced this pull request Jan 26, 2024
This is malloc'ed memory and it does not more. Test 742 helps us verify
this.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants