Skip to content

Fix stack overflow detection during ASAN build.#4642

Open
vchigrin wants to merge 1 commit intobytecodealliance:mainfrom
vchigrin:fix-stack-overflow-detection
Open

Fix stack overflow detection during ASAN build.#4642
vchigrin wants to merge 1 commit intobytecodealliance:mainfrom
vchigrin:fix-stack-overflow-detection

Conversation

@vchigrin
Copy link
Contributor

See discussion in issue #4638.

@yamt
Copy link
Contributor

yamt commented Oct 1, 2025

lum1n0us
lum1n0us previously approved these changes Mar 9, 2026
Copy link
Contributor

@TianlongLiang TianlongLiang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM, but code style CI check is failing

@vchigrin vchigrin force-pushed the fix-stack-overflow-detection branch from 88d9dfe to 75896c5 Compare March 10, 2026 13:24
@vchigrin
Copy link
Contributor Author

Thank you very much, fixed style issues and rebased on more recent main branch commit.

@vchigrin vchigrin requested a review from TianlongLiang March 10, 2026 14:23
Copy link
Contributor

@lum1n0us lum1n0us left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

wasm_runtime_detect_native_stack_overflow works incorrectly under ASAN

4 participants