Skip to content

fix client cookies delete allocator#115

Open
Dmc0125 wants to merge 1 commit into
laytan:mainfrom
Dmc0125:fix/cookies-delete-allocator
Open

fix client cookies delete allocator#115
Dmc0125 wants to merge 1 commit into
laytan:mainfrom
Dmc0125:fix/cookies-delete-allocator

Conversation

@Dmc0125
Copy link
Copy Markdown

@Dmc0125 Dmc0125 commented May 14, 2026

response_destroy was calling delete(cookie._raw) without specifying an allocator, causing it to fall back to context.allocator instead of the allocator used when the response was allocated. This results in an access violation when the response was allocated with a different allocator (e.g. context.temp_allocator).

Fix by passing res.cookies.allocator explicitly to delete(cookie._raw).

Fixes #109

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.

Client's response_destroy uses wrong allocator

1 participant