Skip to content

Fixing fuzzer test crash in route_fuzz_test.cc.#32987

Merged
ggreenway merged 2 commits intoenvoyproxy:mainfrom
yanjunxiang-google:router_fuzzer_test_crash
Mar 20, 2024
Merged

Fixing fuzzer test crash in route_fuzz_test.cc.#32987
ggreenway merged 2 commits intoenvoyproxy:mainfrom
yanjunxiang-google:router_fuzzer_test_crash

Conversation

@yanjunxiang-google
Copy link
Copy Markdown
Contributor

@yanjunxiang-google yanjunxiang-google commented Mar 19, 2024

It looks like with the commit of #32908, fuzzer test under route_fuzz_test.cc are crashing due to some singletons are not initialized.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Yanjun Xiang <yanjunxiang@google.com>
@yanjunxiang-google
Copy link
Copy Markdown
Contributor Author

Fuzzer test crash detail report is here: https://oss-fuzz.com/testcase-detail/6160522024124416

@yanjunxiang-google
Copy link
Copy Markdown
Contributor Author

/assign @ggreenway

@ggreenway ggreenway enabled auto-merge (squash) March 19, 2024 21:37
@yanjunxiang-google
Copy link
Copy Markdown
Contributor Author

/retest

Signed-off-by: Yanjun Xiang <yanjunxiang@google.com>
@ggreenway ggreenway merged commit 1cfe7c9 into envoyproxy:main Mar 20, 2024
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