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.

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

Allocate hiredis_reader_state_t inside hiredis_connection_t #224

Merged
merged 1 commit into from
Jan 12, 2025

Conversation

byroot
Copy link
Member

@byroot byroot commented Jan 8, 2025

Fix: #221

Avoid that pointer becoming invalid when YJIT kicks in.

@byroot byroot mentioned this pull request Jan 8, 2025
@byroot byroot force-pushed the hiredis-no-stack-alloc branch from 686077d to 95af2b3 Compare January 12, 2025 11:18
Fix: #221

Avoid that pointer becoming invalid when YJIT kicks in.

Co-Authored-By: Rian McGuire <[email protected]>
@byroot byroot force-pushed the hiredis-no-stack-alloc branch from 95af2b3 to 66c4a2c Compare January 12, 2025 11:19
@byroot byroot merged commit aedba07 into master Jan 12, 2025
26 of 28 checks passed
@byroot byroot deleted the hiredis-no-stack-alloc branch January 12, 2025 11:21
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.

SIGSEGV in reply_append
1 participant