Skip to content
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

Buffer overflow (IDFGH-9027) #10436

Closed
3 tasks done
chaenel opened this issue Dec 23, 2022 · 0 comments
Closed
3 tasks done

Buffer overflow (IDFGH-9027) #10436

chaenel opened this issue Dec 23, 2022 · 0 comments
Assignees
@hmalpani
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally

Comments

@chaenel
Copy link

chaenel commented Dec 23, 2022

Answers checklist.

  • I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
  • I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report

In esp_http_client_example.c which is used as a starting point to implement http_client applications, definition MAX_HTTP_OUTPUT_BUFFER is used to define the output buffer size.
But on the HTTP_EVENT_ON_DATA event in the _http_event_handler, data are copied into output buffer without limitation to MAX_HTTP_OUTPUT_BUFFER size.
@espressif-bot espressif-bot added the Status: Opened Issue is new label Dec 23, 2022
@github-actions github-actions bot changed the title Buffer overflow Buffer overflow (IDFGH-9027) Dec 23, 2022
@espressif-bot espressif-bot added Status: In Progress Work is in progress and removed Status: Opened Issue is new labels Jan 3, 2023
@espressif-bot espressif-bot added Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: In Progress Work is in progress Resolution: NA Issue resolution is unavailable labels Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hmalpani hmalpani

Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@chaenel @espressif-bot @hmalpani