Skip to content

Update client-http.c#2893

Draft
wreuven wants to merge 5 commits intowarmcat:mainfrom
wreuven:v4.3-stable-fixed
Draft

Update client-http.c#2893
wreuven wants to merge 5 commits intowarmcat:mainfrom
wreuven:v4.3-stable-fixed

Conversation

@wreuven
Copy link
Copy Markdown

@wreuven wreuven commented May 30, 2023

This bug was triggered for example when the connection closes (e.g. timeout) just as we attempt a new reuse of the connection.

Returning 0, means the callers will make use of a now invalid WSI. This often results in a double free of the wsi memory.

Need to return -1

wreuven and others added 5 commits May 30, 2023 17:28
This bug was triggered for example when the connection closes (e.g. timeout) just as we attempt a new reuse of the connection.

Returning 0, means the callers will make use of a now invalid WSI. This often results in a double free of the wsi memory.

Need to return -1
@lws-team lws-team force-pushed the main branch 10 times, most recently from 4a18968 to c199714 Compare June 19, 2025 05:18
@lws-team lws-team force-pushed the main branch 9 times, most recently from aca2548 to 20263de Compare July 9, 2025 18:51
@lws-team lws-team force-pushed the main branch 6 times, most recently from e028532 to 5867044 Compare August 1, 2025 12:39
@lws-team lws-team force-pushed the main branch 3 times, most recently from 555e80f to 36ef2d6 Compare August 11, 2025 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants