Skip to content

GH-49803: [C++][CI] Avoid aborting when fuzz-mutated IPC file has less batches#49804

Merged
pitrou merged 1 commit intoapache:mainfrom
pitrou:gh49803-ipc-file-differential-fuzz
Apr 21, 2026
Merged

GH-49803: [C++][CI] Avoid aborting when fuzz-mutated IPC file has less batches#49804
pitrou merged 1 commit intoapache:mainfrom
pitrou:gh49803-ipc-file-differential-fuzz

Conversation

@pitrou
Copy link
Copy Markdown
Member

@pitrou pitrou commented Apr 20, 2026

Rationale for this change

The IPC file fuzzer can remove batches from the IPC file footer while the batches do exist physically in the embedded IPC stream.

This causes problem in differential fuzzing, because comparing the results of the IPC stream reader against the IPC file reader then fails.

This issue was found by OSS-Fuzz: https://issues.oss-fuzz.com/issues/503357759

What changes are included in this PR?

Skip differential fuzzing against the IPC stream reader if the number of batches read from the IPC file footer is not equal.

Are these changes tested?

Yes, by existing fuzz regression file.

Are there any user-facing changes?

No.

@pitrou pitrou changed the title GH-49803: [C++] Avoid aborting when fuzz-mutated IPC file has less batches GH-49803: [C++][CI] Avoid aborting when fuzz-mutated IPC file has less batches Apr 20, 2026
@pitrou
Copy link
Copy Markdown
Member Author

pitrou commented Apr 20, 2026

@github-actions crossbow submit -g cpp

@github-actions
Copy link
Copy Markdown

Revision: 2f7386e

Submitted crossbow builds: ursacomputing/crossbow @ actions-cebef960c7

Task Status
example-cpp-minimal-build-static GitHub Actions
example-cpp-minimal-build-static-system-dependency GitHub Actions
example-cpp-tutorial GitHub Actions
test-build-cpp-fuzz GitHub Actions
test-conda-cpp GitHub Actions
test-conda-cpp-valgrind GitHub Actions
test-debian-13-cpp-amd64 GitHub Actions
test-debian-13-cpp-i386 GitHub Actions
test-debian-experimental-cpp-gcc-15 GitHub Actions
test-fedora-42-cpp GitHub Actions
test-ubuntu-22.04-cpp GitHub Actions
test-ubuntu-22.04-cpp-20 GitHub Actions
test-ubuntu-22.04-cpp-bundled GitHub Actions
test-ubuntu-22.04-cpp-emscripten GitHub Actions
test-ubuntu-22.04-cpp-no-threading GitHub Actions
test-ubuntu-24.04-cpp GitHub Actions
test-ubuntu-24.04-cpp-bundled-offline GitHub Actions
test-ubuntu-24.04-cpp-gcc-13-bundled GitHub Actions
test-ubuntu-24.04-cpp-gcc-14 GitHub Actions
test-ubuntu-24.04-cpp-minimal-with-formats GitHub Actions
test-ubuntu-24.04-cpp-thread-sanitizer GitHub Actions

@pitrou pitrou marked this pull request as ready for review April 20, 2026 13:25
@pitrou pitrou requested a review from lidavidm April 20, 2026 16:14
@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting review Awaiting review labels Apr 20, 2026
@pitrou pitrou merged commit adecb17 into apache:main Apr 21, 2026
53 of 56 checks passed
@pitrou pitrou removed the awaiting merge Awaiting merge label Apr 21, 2026
@pitrou pitrou deleted the gh49803-ipc-file-differential-fuzz branch April 21, 2026 07:01
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.

2 participants