Skip to content

OSS-Fuzz: Add new fuzzer targets lb operations - #3680

Open
arthurscchan wants to merge 1 commit into
apache:masterfrom
arthurscchan:brpc-lb-fuzzer
Open

arthurscchan wants to merge 1 commit into
apache:masterfrom
arthurscchan:brpc-lb-fuzzer

Conversation

@arthurscchan

Copy link
Copy Markdown
Contributor

This PR adds a new OSS-Fuzz fuzzer targets lb operations, aiming to improve the overall fuzzing coverage and effectiveness of project brpc.

Signed-off-by: Arthur Chan <arthur.chan@adalogics.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

Static review identified no blocking issues in the bounded fuzz harness or its build integration.

No open issues 🎉

What changed in this PR

This PR expands bRPC’s OSS-Fuzz coverage with a dedicated consistent-hashing load-balancer fuzzer.

Changes:

  • Adds input-driven server-ring construction, server selection, and add/remove operations using a fixed socket pool.
  • Registers fuzz_lb in the OSS-Fuzz build script and CMake fuzz targets.
File Description
test/​fuzzing/​oss-fuzz.sh Includes fuzz_lb in the OSS-Fuzz build.
test/​fuzzing/​fuzz_lb.cpp Adds the consistent-hashing load-balancer fuzz harness.
test/​CMakeLists.txt Registers fuzz_lb for CMake fuzz builds.

Review effort: Balanced

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